[U-Boot] [PATCH 2/2] Makefile: Accept target names in all lower case to ease matching
Wolfgang Denk
wd at denx.de
Sun Nov 25 21:04:34 CET 2012
Dear Henrik Nordström,
In message <1353866899.28559.57.camel at home.hno.se> you wrote:
>
> > > @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.
Where exactly?
> Note that it's not about board config file names or directory names,
> only about the first column in boards.cfg.
Well, these are make targets, so they are definitely case sensitive.
> if [ \( $# -eq 2 \) -a \( "$1" = "-A" \) ] ; then
> # Automatic mode
> - line=`egrep -i "^[[:space:]]*${2}[[:space:]]" boards.cfg` || {
> + line=`egrep "^[[:space:]]*${2}[[:space:]]" boards.cfg` || {
Did you test what gets broken by this patch?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I'd like to meet the man who invented sex and see what he's working
on now.
More information about the U-Boot
mailing list