[U-Boot] [PATCH 2/2] Makefile: Accept target names in all lower case to ease matching
Henrik Nordström
henrik at henriknordstrom.net
Sun Nov 25 19:08:19 CET 2012
sön 2012-11-25 klockan 18:45 +0100 skrev Wolfgang Denk:
> Dear Henrik Nordström,
>
> In message <1353842544.17518.11.camel at home.hno.se> you wrote:
> > ---
> > Makefile | 1 +
> > 1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/Makefile b/Makefile
> > index 1a17be9..5a98745 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -752,6 +752,7 @@ unconfig:
> > sinclude $(obj).boards.depend
> > $(obj).boards.depend: boards.cfg
> > @awk '(NF && $$1 !~ /^#/) { print $$1 ": " $$1 "_config; $$(MAKE)" }' $< > $@
> > + @awk '(NF && $$1 !~ /^#/ && tolower($$1) != $$1) { print tolower($$1) ": " $$1 "_config; $$(MAKE)" }' $< > $@
>
> NAK. This is not DOS. Names are case sensitive.
Ok, then I'll swing the patch in the other direction and make target
names actually case sensitive. Because today they are a bit confused
partially case-insensitive.
Note that it's not about board config file names or directory names,
only about the first column in boards.cfg.
Regards
Henrik
---
From: Henrik Nordstrom <henrik at henriknordstrom.net>
Date: Sun, 25 Nov 2012 19:00:12 +0100
Subject: [PATCH] mkconfig: Make board name matching case-sensitive
As per mail discussion board names are supposed to be case-sensitive
---
mkconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/mkconfig b/mkconfig
index 7c9aa74..324703f 100755
--- a/mkconfig
+++ b/mkconfig
@@ -21,7 +21,7 @@ options=""
if [ \( $# -eq 2 \) -a \( "$1" = "-A" \) ] ; then
# Automatic mode
- line=`egrep -i "^[[:space:]]*${2}[[:space:]]" boards.cfg` || {
+ line=`egrep "^[[:space:]]*${2}[[:space:]]" boards.cfg` || {
echo "make: *** No rule to make target \`$2_config'. Stop." >&2
exit 1
}
--
1.7.7.6
More information about the U-Boot
mailing list