build: avoid an extra shell out in quiet variable function
authorJani Nikula <jani@nikula.org>
Tue, 29 Aug 2017 18:27:07 +0000 (21:27 +0300)
committerDavid Bremner <david@tethera.net>
Wed, 30 Aug 2017 01:05:37 +0000 (22:05 -0300)
$(word 1, $1) yields the same result as the more complicated
$(shell echo $1 | sed -e s'/ .*//')

Makefile.local

index af12ca7f2ef88353648fbbf584422cd613bdb830..c51b9afb8fe6549b67f1c7c116fe1399aec8913a 100644 (file)
@@ -182,14 +182,14 @@ verify-newer:
 # user how to enable verbose compiles.
 ifeq ($(V),)
 quiet_DOC := "Use \"$(MAKE) V=1\" to see the verbose compile lines.\n"
-quiet = @printf $(quiet_DOC)$(eval quiet_DOC:=)"$1 $@\n"; $($(shell echo $1 | sed -e s'/ .*//'))
+quiet = @printf $(quiet_DOC)$(eval quiet_DOC:=)"$1 $@\n"; $($(word 1, $(1)))
 endif
 # The user has explicitly enabled quiet compilation.
 ifeq ($(V),0)
-quiet = @printf "$1 $@\n"; $($(shell echo $1 | sed -e s'/ .*//'))
+quiet = @printf "$1 $@\n"; $($(word 1, $(1)))
 endif
 # Otherwise, print the full command line.
-quiet ?= $($(shell echo $1 | sed -e s'/ .*//'))
+quiet ?= $($(word 1, $(1)))
 
 %.o: %.cc $(global_deps)
        @mkdir -p $(patsubst %/.,%,.deps/$(@D))