[Buildroot] [PATCH 00/12] toolchain: warn for unsafe library/header paths
arnout at mind.be
Tue Aug 26 20:14:06 UTC 2014
[/me is back from too long holidays, facing 2000 BR mails :-)]
On 08/24/14 13:22, Yann E. MORIN wrote:
> Thomas², All,
> On 2014-08-22 21:29 +0200, Thomas De Schampheleire spake thusly:
>> "Yann E. MORIN" <yann.morin.1998 at free.fr> schreef:
>>> Thomas, All,
>>> On 2014-08-21 23:33 +0200, Thomas Petazzoni spake thusly:
>>>> On Thu, 21 Aug 2014 22:30:29 +0200, Romain Naour wrote:
>>>>> Also, BR_COMPILER_PARANOID_UNSAFE_PATH needs to be renamed to
>>>>> BR2_COMPILER_PARANOID_UNSAFE_PATH to follow the naming scheme ?
>>>> No: BR2_* variables are Config.in options. Environment variables are
>>> Earlier this year in Brussels, we said we were to use BR2_ everywhere :
>> Hmm, that is written in the conclusion, but this is very
>> different than what is in the 'proposed rules'.
>> Was it really the final outcome to use BR2 for all types
>> of variable? I thought we wanted to make a
>> distinction between config variables and others...
> Yes, I was a bit surprised by the conclusion, too.
> From what I remember, I thought we decided for:
> - user-facing variables: prefix with BR2_
> - internal variables: prefix with BR_
> IIRC, we already have a few changesets implementing those rules.
> For example:
> 35c666e toolchain/wrapper: rename BR_DEBUG_WRAPPER
> (I could not find another reverse such change, unfortunetely...)
Blame the reporter.
The conclusion was:
* BR2_ for all Kconfig variables;
* BR2_ for user-facing environment variables;
* BR_ for internal environment variables.
Other such changes:
4113b3c infra: replace BUILDROOT_CONFIG with BR2_CONFIG
6768021 infra: replace BUILDROOT_DL_DIR with BR2_DL_DIR.
> Yann E. MORIN.
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot