Simplify pkg-config handling
authorIOhannes m zmölnig (Debian/GNU) <umlaeute@debian.org>
Wed, 11 Jan 2023 16:11:33 +0000 (17:11 +0100)
committerIOhannes m zmölnig (Debian/GNU) <umlaeute@debian.org>
Wed, 11 Jan 2023 16:18:46 +0000 (17:18 +0100)
debian/rules

index 58ad929b6344e67a99b16af7a2b648d735a483c1..773f3bdb13ef8b86f0781ed4824ea11167953044 100755 (executable)
@@ -17,11 +17,12 @@ else
 endif
 
 JUCE_VERSION := $(shell dpkg-query --show --showformat='$${source:Version}' juce-modules-source)
+pkglibs = libjpeg libpng libcurl rtaudio
 
 DEB_CPPFLAGS_MAINT_APPEND += -DBUILD_DATE='"$(BUILD_DATE)"' -DTEST_RESOURCES_DIR='"tests/resources/"'
 DEB_CXXFLAGS_MAINT_APPEND += -std=c++17 -Wno-error -Wno-error=format-security
-LIBS=$(shell pkg-config --libs libjpeg libpng libcurl rtaudio)
-DEB_CXXFLAGS_MAINT_APPEND += $(shell pkg-config --cflags libjpeg libpng libcurl rtaudio)
+DEB_CXXFLAGS_MAINT_APPEND += $(shell pkg-config --cflags $(pkglibs))
+DEB_LDFLAGS_MAINT_APPEND  += $(shell pkg-config --libs $(pkglibs))
 
 # JUCE (used by giada) uses some c++11 features requiring atomic_store_8 and
 # atomic_load_8, so we need to link with libatomic on
@@ -33,11 +34,9 @@ DEB_CXXFLAGS_MAINT_APPEND += $(shell pkg-config --cflags libjpeg libpng libcurl
 noatomicarch = $(shell dpkg-architecture -qDEB_HOST_ARCH | egrep -x "(armel|powerpc|powerpcspe|m68k|mips|mipsel|sh4|riscv64)")
 # link with libatomic on architectures without built-in atomic
 ifeq ($(if $(noatomicarch),atomic), atomic)
-       LIBS += -latomic
+       DEB_LDFLAGS_MAINT_APPEND += -latomic
 endif
 
-DEB_LDFLAGS_MAINT_APPEND += $(LIBS)
-
 %:
        dh $@