]> xenbits.xensource.com Git - people/vhanquez/xen.git/commitdiff
build: fix grep invocation in cc-options
authorIan Campbell <ian.campbell@citrix.com>
Mon, 3 Oct 2011 15:36:09 +0000 (16:36 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Mon, 3 Oct 2011 15:36:09 +0000 (16:36 +0100)
Currently the build produces lots of
        Usage: grep [OPTION]... PATTERN [FILE]...
        Try `grep --help' for more information.

This is due to the "grep -- $(2)" in cc-options. It seems that the
default of reading stdin is disabled when using "--". I don't know if
this is a bug in grep or how it is supposed to be but we can work
around it by explicitly passing in "-"

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Committed-by: Keir Fraser <keir@xen.org>
xen-unstable changeset:   23898:3d1664cc9e45
xen-unstable date:        Fri Sep 30 21:17:47 2011 +0100

Config.mk

index ccab44742e4f979170e392690b3d4f56370438a2..e2ede7430aac504817cf2ab2dcca3f1d2ef06cbd 100644 (file)
--- a/Config.mk
+++ b/Config.mk
@@ -57,7 +57,7 @@ PYTHON_PREFIX_ARG ?= --prefix="$(PREFIX)"
 #
 # Usage: cflags-y += $(call cc-option,$(CC),-march=winchip-c6,-march=i586)
 cc-option = $(shell if test -z "`echo 'void*p=1;' | \
-              $(1) $(2) -S -o /dev/null -xc - 2>&1 | grep -- $(2)`"; \
+              $(1) $(2) -S -o /dev/null -xc - 2>&1 | grep -- $(2) -`"; \
               then echo "$(2)"; else echo "$(3)"; fi ;)
 
 # cc-option-add: Add an option to compilation flags, but only if supported.