[oe] [meta-clang][PATCH] clang: make sure python3 is used for native builds

[Khem Raj]

On Fri, Sep 13, 2019 at 7:49 AM Martin Jansa <martin.jansa at gmail.com> wrote:
>
> OK, will do, README should be updated as well as it says to send individual patches to ML :).

right. Send another patch to update that

>
> On Fri, Sep 13, 2019 at 3:59 PM Khem Raj <raj.khem at gmail.com> wrote:
>>
>> On Fri, Sep 13, 2019 at 3:53 AM Martin Jansa <martin.jansa at gmail.com> wrote:
>> >
>> > Please ignore this one.
>> >
>> > Now I've noticed that it was like this before "clang: Let cmake figure out
>> > python version for native" so I'll have to find why it doesn't work for me
>> > instead of reverting your recent changes.
>> >
>>
>> this change is fine, there is another problem especially when doing stage2 build
>> where it does not respect python settings from env so I was hoping
>> that it will help to
>> let it detect python from hosttools which it does but I think this
>> patch makes things better
>> I would certainly think lets keep it. I am also mulling to disable
>> stage2, it will mean that
>> we dont use LTO to build the compiler itself it might become a bit
>> slow to compile programs
>>
>> can you send this as github pull please since CI is triggered on pulls there ?
>>
>> > On Fri, Sep 13, 2019 at 12:26 PM Martin Jansa <martin.jansa at gmail.com>
>> > wrote:
>> >
>> > > * with /usr/bin/python -> python2.7 on the host, I was seeing:
>> > >   CMake Error at
>> > > work-shared/llvm-project-source-9.0.0-r0/git/lldb/cmake/modules/LLDBConfig.cmake:229
>> > > (message):
>> > >     Found incompatible Python interpreter (2.7) and Python libraries (3.7)
>> > >   Call Stack (most recent call first):
>> > >     work-shared/llvm-project-source-9.0.0-r0/git/lldb/CMakeLists.txt:20
>> > > (include)
>> > >
>> > > Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
>> > > ---
>> > >  recipes-devtools/clang/clang_git.bb | 3 ++-
>> > >  1 file changed, 2 insertions(+), 1 deletion(-)
>> > >
>> > > diff --git a/recipes-devtools/clang/clang_git.bb b/recipes-devtools/clang/
>> > > clang_git.bb
>> > > index 3906ef1..880307f 100644
>> > > --- a/recipes-devtools/clang/clang_git.bb
>> > > +++ b/recipes-devtools/clang/clang_git.bb
>> > > @@ -17,7 +17,7 @@ BUILD_RANLIB_class-nativesdk = "llvm-ranlib"
>> > >  BUILD_NM_class-nativesdk = "llvm-nm"
>> > >  LDFLAGS_append_class-nativesdk = " -fuse-ld=gold"
>> > >
>> > > -inherit cmake cmake-native python3-dir
>> > > +inherit cmake cmake-native python3-dir python3native
>> > >
>> > >  OECMAKE_FIND_ROOT_PATH_MODE_PROGRAM = "BOTH"
>> > >
>> > > @@ -102,6 +102,7 @@ EXTRA_OECMAKE_append_class-native = "\
>> > >                    -DBOOTSTRAP_LLVM_ENABLE_LLD=ON \
>> > >                    -DLLVM_TARGETS_TO_BUILD='${LLVM_TARGETS_TO_BUILD}' \
>> > >
>> > >  -DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD='${LLVM_EXPERIMENTAL_TARGETS_TO_BUILD}'
>> > > \
>> > > +                  -DPYTHON_EXECUTABLE='${PYTHON}' \
>> > >  "
>> > >  EXTRA_OECMAKE_append_class-nativesdk = "\
>> > >                    -DCMAKE_CROSSCOMPILING:BOOL=ON \
>> > > --
>> > > 2.17.1
>> > >
>> > >
>> > --
>> > _______________________________________________
>> > Openembedded-devel mailing list
>> > Openembedded-devel at lists.openembedded.org
>> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel