From: Nikolai Merinov <n.merinov@inango-systems.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: openembedded-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCHv3] cmake.bbclass: pass mandatory compiler flags through CMAKE_<LANG>_COMPILER_ARG1
Date: Tue, 25 Jun 2019 15:39:36 +0300 (IDT) [thread overview]
Message-ID: <1029454405.2067.1561466376798.JavaMail.zimbra@inango-systems.com> (raw)
In-Reply-To: <24b339d3b414d7f32a9525e2480d2c7c4ffa0aa3.camel@linuxfoundation.org>
Hi Richard,
Can you help me to understand how I can reproduce this issue on local build? I want to try to debug it.
Regards,
Nikolai
----- Original Message -----
From: "Richard Purdie" <richard.purdie@linuxfoundation.org>
To: "n merinov" <n.merinov@inango-systems.com>
Cc: "openembedded-core" <openembedded-core@lists.openembedded.org>
Sent: Tuesday, June 25, 2019 1:03:11 PM
Subject: Re: [OE-core] [PATCHv3] cmake.bbclass: pass mandatory compiler flags through CMAKE_<LANG>_COMPILER_ARG1
On Mon, 2019-06-24 at 13:24 +0300, Nikolai Merinov via Openembedded-
core wrote:
> Patch updated with "Signed-off-by" field as required by automatic
> checks.
>
> ----- Original Message -----
> From: "openembedded-core" <openembedded-core@lists.openembedded.org>
> To: "Khem Raj" <raj.khem@gmail.com>
> Cc: "openembedded-core" <openembedded-core@lists.openembedded.org>
> Sent: Thursday, June 20, 2019 7:16:07 PM
> Subject: [OE-core] [PATCHv2] cmake.bbclass: pass mandatory compiler
> flags through CMAKE_<LANG>_COMPILER_ARG1
>
> Hi,
>
> Updated patch for the current "master" branch is attached.
>
> > have you also tried building SDK and see if these changes are
> > reflected in SDK as well ?
>
> I checked that the changes reflected in the Extensible SDK for the
> "core-image-sato" image of the poky.
>
> Regards,
> Nikolai
>
> ----- Original Message -----
> From: "Khem Raj" <raj.khem@gmail.com>
> To: "n merinov" <n.merinov@inango-systems.com>, "openembedded-core" <
> openembedded-core@lists.openembedded.org>
> Sent: Tuesday, May 21, 2019 1:41:46 AM
> Subject: Re: [OE-core] [PATCH] cmake.bbclass: pass mandatory compiler
> flags through CMAKE_<LANG>_COMPILER_ARG1
>
> On 5/14/19 8:04 AM, Nikolai Merinov wrote:
> > The CMake takes mandatory compiler arguments from the following
> > variables:
> > - CMAKE_SYSROOT -- path to sysroot that should be passed to
> > compiler.
> > - CMAKE_<LANG>_COMPILER_TARGET -- target architecture, used for
> > compilers
> > that supports several targets through command line options.
> > e.g. "clang --target ${CMAKE_C_COMPILER_TARGET}".
> > - CMAKE_<LANG>_COMPILER_EXTERNAL_TOOLCHAIN -- path to external
> > toolchain,
> > used for compilers that support build with external toolchain.
> > e.g. "clang --gcc-toolchain
> > ${CMAKE_C_COMPILER_EXTERNAL_TOOLCHAIN}".
> > - CMAKE_<LANG>_COMPILER_ARG1 -- other mandatory arguments to a
> > compiler
> > command.
> >
> > CMAKE_<LANG>_COMPILER_ARG1 is the most suitable variable to pass
> > mandatory
> > arguments, that belongs to CC variable with other build systems, to
> > a
> > compiler.
> >
> > Additionally usage of CMAKE_<LANG>_COMPILER_ARG1 instead of
> > CMAKE_<LANG>_FLAGS reduce the risk that a variable can be overrided
> > by
> > CMakeLists.txt files.
>
I'm afraid this caused a number of build failures and selftest issues:
https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/236
(most of the selftest failures there for example are from this, as is
https://autobuilder.yoctoproject.org/typhoon/#/builders/69/builds/746/steps/7/logs/step7b
)
Cheers,
Richard
next prev parent reply other threads:[~2019-06-25 12:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-14 15:04 [PATCH] cmake.bbclass: pass mandatory compiler flags through CMAKE_<LANG>_COMPILER_ARG1 Nikolai Merinov
2019-05-14 15:49 ` Bach, Pascal
2019-05-15 9:19 ` Nikolai Merinov
2019-05-20 20:41 ` Khem Raj
2019-06-20 14:16 ` [PATCHv2] " Nikolai Merinov
2019-06-24 10:24 ` [PATCHv3] " Nikolai Merinov
2019-06-25 8:03 ` Richard Purdie
2019-06-25 12:39 ` Nikolai Merinov [this message]
2019-06-20 14:30 ` ✗ patchtest: failure for cmake.bbclass: pass mandatory compiler flags through CMAKE_<LANG>_COMPILER_ARG1 (rev2) Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1029454405.2067.1561466376798.JavaMail.zimbra@inango-systems.com \
--to=n.merinov@inango-systems.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=richard.purdie@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox