public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
To: openembedded-core@lists.openembedded.org
Subject: sysroot flags in $CC (was: [PATCH 3/3] ncurses: upgrade to 6.0+20160625)
Date: Mon, 22 Aug 2016 13:15:45 +0200	[thread overview]
Message-ID: <lywpj9hx5q.fsf_-_@ensc-virt.intern.sigma-chemnitz.de> (raw)
In-Reply-To: <CAJTo0LY1L1zadYdw+AH=hVD6xu5r9sRap_G7+iALhf9q41Excw@mail.gmail.com> (Ross Burton's message of "Mon, 22 Aug 2016 11:18:22 +0100")

"Burton, Ross" <ross.burton-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
writes:

>> This breaks the build with OE because it tries to split our $CC.
>> E.g. with
>>
>> | CC = ccache arm-linux-gnueabi-gcc -march=armv7ve ... --sysroot=...
>>
>> configure will warn
>>
>> | your environment misuses the CC variable to hold CFLAGS/CPPFLAGS options
>>
>> and messes up CFLAGS completely so that 'arm-linux-gnuebi-gcc' becomes
>> part of it :(
>>
>
> I've long wondered about changing our abuse of CC to hold the sysroot,

I do not think, that this is such a big abuse.  There are no flags
where these options can/should be placed instead of.  CPPFLAGS would
work for automake projects but lot of other buildsystems do not know
this variable.

Writing them into CFLAGS (and CXXFLAGS and FFLAGS and ...) would still
require them in CPPFLAGS too (for 'gcc -E' cases).  They will appear
twice (CPPFLAGS + xFLAGS) in automake project which is not a big deal
but a little bit ugly.


Enrico
-- 
SIGMA Chemnitz GmbH       Registergericht:   Amtsgericht Chemnitz HRB 1750
Am Erlenwald 13           Geschaeftsfuehrer: Grit Freitag, Frank Pyritz
09128 Chemnitz


  reply	other threads:[~2016-08-22 11:16 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-15  7:35 [PATCH 0/3] packages upgrade: man-pages/gnupg/ncurses Hongxu Jia
2016-08-15  7:35 ` [PATCH 1/3] man-pages: upgrade to 4.07 Hongxu Jia
2016-08-15  7:35 ` [PATCH 2/3] gnupg: upgrade to 2.1.14 Hongxu Jia
2016-08-15  7:35 ` [PATCH 3/3] ncurses: upgrade to 6.0+20160625 Hongxu Jia
2016-08-22  9:40   ` Enrico Scholz
2016-08-22 10:18     ` Burton, Ross
2016-08-22 11:15       ` Enrico Scholz [this message]
2016-08-22 11:21       ` Enrico Scholz

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=lywpj9hx5q.fsf_-_@ensc-virt.intern.sigma-chemnitz.de \
    --to=enrico.scholz@sigma-chemnitz.de \
    --cc=openembedded-core@lists.openembedded.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