[Buildroot] [PATCH 00/12] toolchain: warn for unsafe library/header paths

[Yann E. MORIN]

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
> >> BR_*.
> >
> >Earlier this year in Brussels, we said we were to use BR2_ everywhere :
> >    http://elinux.org/Buildroot:DeveloperDaysFOSDEM2014
> 
> 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...)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'