On Mon, Feb 13, 2012 at 10:43 PM, Mike Frysinger <vap...@gentoo.org> wrote: > On Monday 13 February 2012 18:02:02 Tom Rini wrote: >> isnt possible with /dev/null. > > isn't > >> --- a/config.mk >> +++ b/config.mk >> >> -cc-option-sys = $(shell if $(CC) $(CFLAGS) $(1) -S -o /dev/null -xc >> /dev/null \ >> - > /dev/null 2>&1; then \ >> +cc-option-sys = $(shell mkdir -p $(dir $(CC_TEST_FILE)); \ >> + touch $(CC_TEST_FILE); \ >> + if $(CC) $(CFLAGS) $(1) -S -o $(CC_TEST_FILE) \ >> + -xc $(CC_TEST_FILE) > /dev/null 2>&1; then \ > > why do you need to touch the file first ? why do you source this test file > instead of continuing to source /dev/null ? don't you run into problems if > you use cc-option-sys more than once and a previous run wrote something to > that file ? then cc-option-sys could fail on later runs ... > > seems like you should drop the `touch` and keep the -xc /dev/null and only > change the -o to $(CC_TEST_FILE) ...
I've also changed the variable to CC_TEST_OFILE and that works, thanks. -- Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot