All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shinya Kuribayashi <skuribay@ruby.dti.ne.jp>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH 8/9] Generate include/autoconf.mk from board	config files
Date: Wed, 26 Sep 2007 23:24:24 +0900	[thread overview]
Message-ID: <46FA6B98.202@ruby.dti.ne.jp> (raw)
In-Reply-To: <20070925182710.18532.84181.stgit@trillian.cg.shawcable.net>

Grant Likely wrote:
> diff --git a/tools/scripts/define2mk.sed b/tools/scripts/define2mk.sed
> new file mode 100644
> index 0000000..6464627
> --- /dev/null
> +++ b/tools/scripts/define2mk.sed
> @@ -0,0 +1,29 @@
> +#
> +# Sed script to parse CPP macros and generate output usable by make
> +#
> +# It is expected that this script is fed the output of 'gpp -dM'

'cpp -dM'

> +# which preprocesses the common.h header files and outputs the final
> +# list of CPP macros (and whitespace is sanitized)
> +#
> +
> +# Only process values prefixed with #define CONFIG_
> +/^#define CONFIG_[A-Za-z0-9_]\+/ {
> +	# Strip the #define prefix
> +	s/#define *//;
> +	# Change to form CONFIG_*=VALUE
> +	s/ \+/=/;
> +	# Drop trailing spaces
> +	s/ *$//;
> +	# drop quotes around string values
> +	s/="\(.*\)"$/=\1/;
> +	# Concatenate string values
> +	s/" *"//g;
> +	# Wrap non-numeral values with quotes
> +	s/=\(.*\?[^0-9].*\)$/=\"\1\"/;
> +	# Change '1' and empty values to "y" (not perfect, but
> +	# supports conditional compilation in the makefiles
> +	s/=$/=y/;
> +	s/=1$/=y/;
> +	# print the line
> +	p
> +}

It seems scripts works, but I feel like a little bit hard coded.
FWIW how Linux work around this sort of job? I have to check..
Anyway, I don't intend to object this patch. Please go ahead :-)

Thanks,

    Shinya Kuribayashi

P.S.
The remaining patches are also Ok for me.

  reply	other threads:[~2007-09-26 14:24 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-25 18:26 [U-Boot-Users] [PATCH 0/9] First wave of config changes Grant Likely
2007-09-25 18:26 ` [U-Boot-Users] [PATCH 1/9] Build: split COBJS value into multiple lines Grant Likely
2007-09-25 18:26 ` [U-Boot-Users] [PATCH 2/9] Group network drivers in drivers/Makefile Grant Likely
2007-09-25 18:26 ` [U-Boot-Users] [PATCH 3/9] Group console " Grant Likely
2007-09-25 18:26 ` [U-Boot-Users] [PATCH 4/9] Group i2c " Grant Likely
2007-09-25 18:26 ` [U-Boot-Users] [PATCH 5/9] Group USB " Grant Likely
2007-09-25 18:27 ` [U-Boot-Users] [PATCH 6/9] Group block/flash " Grant Likely
2007-09-25 18:27 ` [U-Boot-Users] [PATCH 7/9] Group PCI " Grant Likely
2007-09-25 18:27 ` [U-Boot-Users] [PATCH 8/9] Generate include/autoconf.mk from board config files Grant Likely
2007-09-26 14:24   ` Shinya Kuribayashi [this message]
2007-09-26 14:51     ` Grant Likely
2007-09-25 18:27 ` [U-Boot-Users] [PATCH 9/9] Conditionally compile common/cmd_*.c Grant Likely
2007-09-25 19:39 ` [U-Boot-Users] [PATCH 0/9] First wave of config changes Kim Phillips
2007-09-25 19:47   ` Grant Likely
2007-09-25 21:21     ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=46FA6B98.202@ruby.dti.ne.jp \
    --to=skuribay@ruby.dti.ne.jp \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.