diff mbox

[U-Boot,v8,15/31] autoconfig.mk: Make it possible to define configs from other configs

Message ID 1362139841-16540-15-git-send-email-benoit.thebaudeau@advansee.com
State Superseded
Delegated to: Stefano Babic
Headers show

Commit Message

Benoît Thébaudeau March 1, 2013, 12:10 p.m. UTC
Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
---
Changes in v8:
 - New patch.

Changes in v7: None
Changes in v6: None
Changes in v5: None
Changes in v4: None
Changes in v3: None
Changes in v2: None

 tools/scripts/define2mk.sed |    2 ++
 1 file changed, 2 insertions(+)

Comments

Tom Rini March 1, 2013, 9:20 p.m. UTC | #1
On Fri, Mar 01, 2013 at 01:10:24PM +0100, Beno??t Th??baudeau wrote:

> Signed-off-by: Beno??t Th??baudeau <benoit.thebaudeau@advansee.com>
> ---
> Changes in v8:
>  - New patch.
> 
> Changes in v7: None
> Changes in v6: None
> Changes in v5: None
> Changes in v4: None
> Changes in v3: None
> Changes in v2: None
> 
>  tools/scripts/define2mk.sed |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tools/scripts/define2mk.sed b/tools/scripts/define2mk.sed
> index 13e2845..c641edf 100644
> --- a/tools/scripts/define2mk.sed
> +++ b/tools/scripts/define2mk.sed
> @@ -24,6 +24,8 @@
>  	s/="\([0-9][0-9]*\)"/=\1/;
>  	# ... and from hex numbers
>  	s/="\(0[Xx][0-9a-fA-F][0-9a-fA-F]*\)"/=\1/;
> +	# ... and from configs defined from other configs
> +	s/="\(CONFIG_[A-Za-z0-9_][A-Za-z0-9_]*\)"/=$(\1)/;
>  	# Change '1' and empty values to "y" (not perfect, but
>  	# supports conditional compilation in the makefiles
>  	s/=$/=y/;

OK, please add a little more to the commit message, such as an example
of how this works or gives us now?  Thanks.
Benoît Thébaudeau March 1, 2013, 9:30 p.m. UTC | #2
Hi Tom,

On Friday, March 1, 2013 10:20:18 PM, Tom Rini wrote:
> On Fri, Mar 01, 2013 at 01:10:24PM +0100, Beno??t Th??baudeau wrote:
> 
> > Signed-off-by: Beno??t Th??baudeau <benoit.thebaudeau@advansee.com>
> > ---
> > Changes in v8:
> >  - New patch.
> > 
> > Changes in v7: None
> > Changes in v6: None
> > Changes in v5: None
> > Changes in v4: None
> > Changes in v3: None
> > Changes in v2: None
> > 
> >  tools/scripts/define2mk.sed |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/tools/scripts/define2mk.sed b/tools/scripts/define2mk.sed
> > index 13e2845..c641edf 100644
> > --- a/tools/scripts/define2mk.sed
> > +++ b/tools/scripts/define2mk.sed
> > @@ -24,6 +24,8 @@
> >  	s/="\([0-9][0-9]*\)"/=\1/;
> >  	# ... and from hex numbers
> >  	s/="\(0[Xx][0-9a-fA-F][0-9a-fA-F]*\)"/=\1/;
> > +	# ... and from configs defined from other configs
> > +	s/="\(CONFIG_[A-Za-z0-9_][A-Za-z0-9_]*\)"/=$(\1)/;
> >  	# Change '1' and empty values to "y" (not perfect, but
> >  	# supports conditional compilation in the makefiles
> >  	s/=$/=y/;
> 
> OK, please add a little more to the commit message, such as an example
> of how this works or gives us now?  Thanks.

OK, I'll do that in v9.

Thanks for the reviews.

Best regards,
Benoît
diff mbox

Patch

diff --git a/tools/scripts/define2mk.sed b/tools/scripts/define2mk.sed
index 13e2845..c641edf 100644
--- a/tools/scripts/define2mk.sed
+++ b/tools/scripts/define2mk.sed
@@ -24,6 +24,8 @@ 
 	s/="\([0-9][0-9]*\)"/=\1/;
 	# ... and from hex numbers
 	s/="\(0[Xx][0-9a-fA-F][0-9a-fA-F]*\)"/=\1/;
+	# ... and from configs defined from other configs
+	s/="\(CONFIG_[A-Za-z0-9_][A-Za-z0-9_]*\)"/=$(\1)/;
 	# Change '1' and empty values to "y" (not perfect, but
 	# supports conditional compilation in the makefiles
 	s/=$/=y/;