[Buildroot] [PATCH v5 1/4] Rename the variable "QUIET" to "BR_QUIET" for conformance

[Fabio Porcedda]

On Tue, Dec 30, 2014 at 11:59 AM, Fabio Porcedda
<fabio.porcedda at gmail.com> wrote:
> On Sun, Dec 28, 2014 at 10:57 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
>> Fabio, All,
>>
>> On 2014-12-28 17:24 +0100, Fabio Porcedda spake thusly:
>>> On Tue, Dec 23, 2014 at 11:51 AM, Fabio Porcedda
>>> <fabio.porcedda at gmail.com> wrote:
>>> > On Tue, Dec 23, 2014 at 11:17 AM, Thomas Petazzoni
>>> > <thomas.petazzoni at free-electrons.com> wrote:
>>> >> Hello,
>>> >>
>>> >> On Tue, 23 Dec 2014 09:54:58 +0100, Fabio Porcedda wrote:
>>> >>> Rename the variable "QUIET" to "BR_QUIET" accordingly to the convention
>>> >>> that internal variables should be named BR_XXX.
>>> >>>
>>> >>> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
>>> >>
>>> >> Do we really want all internal variables to be named BR_* ? Like
>>> >> HOST_CONFIGURE_OPTS, TARGET_CONFIGURE_OPTS, TARGET_MAKE_ENV and al. ?
>>> >> I'm not sure we want to do this, do we?
>>> >
>>> > On Wed, Nov 26, 2014 at 10:35 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
>>> > <snip>
>>> >> Also, in retrospect, QUIET was not a really good name for this variable
>>> >> in the Makefile. It would have been better if it were named BR_QUIET.
>>> >> But that was introduced way before we decided on a variable naming
>>> >> convention... :-/
>>> >
>>> > So which naming conventions Yann was referring?
>>>
>>> Hi Yann,
>>> Can you please explain further your opinion?
>>
>> Oh, sorry, it seems I was not explicit enough in my mail...
>>
>> What I said was "QUIET was not a really good name [...] that was
>> introduced way before we decided on a variable naming convention..."
>>
>> I did not want to make it sound like we should rename it, sorry.
>>
>> I agree with Thomas here: we should keep QUIET as-is.
>>
>> Sorry, sorry for the confusion... :-(
>
> Ok, I will sent an updated patch set without the renaming patch.

I've sent an updated patch set:
http://lists.busybox.net/pipermail/buildroot/2014-December/116079.html

BR
-- 
Fabio Porcedda