Makefile: evaluate pkg-config once
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 18 Nov 2009 11:34:54 +0000 (11:34 +0000)
committerCarl Worth <cworth@cworth.org>
Wed, 18 Nov 2009 23:20:20 +0000 (00:20 +0100)
Currently the same `pkg-config ...` is executed for every target, so
just store the results in a variable.

Makefile

index 96aaa7370c558d5ad6aabab5837fc6efefc0d579..023b2ec85fed32059dae867386a189573b5e0e66 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -4,15 +4,16 @@ CFLAGS=-O2
 
 # Additional flags that we will append to whatever the user set.
 # These aren't intended for the user to manipulate.
-extra_cflags = `pkg-config --cflags glib-2.0 gmime-2.4 talloc`
-extra_cxxflags = `xapian-config --cxxflags`
+extra_cflags := $(shell pkg-config --cflags glib-2.0 gmime-2.4 talloc)
+extra_cxxflags := $(shell xapian-config --cxxflags)
 
 # Now smash together user's values with our extra values
 override CFLAGS += $(WARN_FLAGS) $(extra_cflags)
 override CXXFLAGS += $(WARN_FLAGS) $(extra_cflags) $(extra_cxxflags)
 
-override LDFLAGS += `pkg-config --libs glib-2.0 gmime-2.4 talloc` \
-                       `xapian-config --libs`
+override LDFLAGS += \
+       $(shell pkg-config --libs glib-2.0 gmime-2.4 talloc) \
+       $(shell xapian-config --libs)
 
 # Include our local Makefile.local first so that its first target is default
 include Makefile.local