]> 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:33:29 +0000 (16:33 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Mon, 3 Oct 2011 15:33:29 +0000 (16:33 +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 07ce7b3fde1698e94b5e6d22550d4e7eb9435875..62d3b0e448436382b0adee8c74aa471d844df452 100644 (file)
--- a/Config.mk
+++ b/Config.mk
@@ -73,7 +73,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.