From mboxrd@z Thu Jan  1 00:00:00 1970
From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= <benoit.thebaudeau@advansee.com>
Date: Thu, 7 Mar 2013 12:38:04 +0100 (CET)
Subject: [U-Boot] [PATCH v9 15/30] autoconfig.mk: Make it possible to
 define configs from other configs
In-Reply-To: <20130307065713.D53D2200520@gemini.denx.de>
References: <1362596377-5827-1-git-send-email-benoit.thebaudeau@advansee.com>
	<1362596377-5827-15-git-send-email-benoit.thebaudeau@advansee.com>
	<20130307065713.D53D2200520@gemini.denx.de>
Message-ID: <1362963274.331312.1362656284373.JavaMail.root@advansee.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Wolfgang Denk,

On Thursday, March 7, 2013 7:57:13 AM, Wolfgang Denk wrote:
> Dear Beno?t Th?baudeau,
> 
> In message <1362596377-5827-15-git-send-email-benoit.thebaudeau@advansee.com>
> you wrote:
> >
> > +	# ... and from configs defined from other configs
> > +	s/="\(CONFIG_[A-Za-z0-9_][A-Za-z0-9_]*\)"/=$(\1)/;
> 
> Should we not remove the lower case letters here?  Such are not
> supposed to be used in macro names.

I don't think so. They're indeed not supposed to be used, but there are perhaps
exceptions somewhere, and it's not the purpose of this script to enforce such a
rule. Also, note that lowercase letters are used in the main line filter of this
script too.

But if this is a strong requirement from you to remove lowercase letters here,
just tell me, and I'll make the change. In that case, should the main line
filter be changed too?

Best regards,
Beno?t