debian: Build up CONFARGS list rather than individual variables

This avoids having a silly line length for the dh_auto_configure
invocation.
This commit is contained in:
Jessica Clarke 2020-11-16 04:31:18 +00:00 committed by Mario Limonciello
parent 227e281cf9
commit f69931032e

View File

@ -10,10 +10,10 @@ ifeq ($(DEB_HOST_ARCH_BITS),32)
export DEB_CFLAGS_MAINT_APPEND = -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE export DEB_CFLAGS_MAINT_APPEND = -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE
endif endif
CONFARGS =
ifneq ($(CI),) ifneq ($(CI),)
CI_FLAGS=--werror --wrap-mode=default CONFARGS += --werror --wrap-mode=default
else
CI_FLAGS=
endif endif
SB_STYLE := debian SB_STYLE := debian
@ -21,34 +21,36 @@ deb_version := $(shell dpkg-parsechangelog --show-field Version)
ifeq (yes,$(shell dpkg-vendor --derives-from Ubuntu && echo yes)) ifeq (yes,$(shell dpkg-vendor --derives-from Ubuntu && echo yes))
SB_STYLE := ubuntu SB_STYLE := ubuntu
tar_name := fwupd_$(deb_version)_$(DEB_HOST_ARCH).tar.gz tar_name := fwupd_$(deb_version)_$(DEB_HOST_ARCH).tar.gz
FLASHROM=-Dplugin_flashrom=false CONFARGS += -Dplugin_flashrom=false
else else
TMPLDIR := debian/fwupd-$(DEB_HOST_ARCH)-signed-template/usr/share/code-signing/fwupd-$(DEB_HOST_ARCH)-signed-template TMPLDIR := debian/fwupd-$(DEB_HOST_ARCH)-signed-template/usr/share/code-signing/fwupd-$(DEB_HOST_ARCH)-signed-template
ifneq ($(DEB_HOST_ARCH_CPU),ia64) ifneq ($(DEB_HOST_ARCH_CPU),ia64)
FLASHROM=-Dplugin_flashrom=true CONFARGS += -Dplugin_flashrom=true
else else
FLASHROM=-Dplugin_flashrom=false CONFARGS += -Dplugin_flashrom=false
endif endif
endif endif
ifeq (yes,$(shell pkg-config --exists libsmbios_c && echo yes)) ifeq (yes,$(shell pkg-config --exists libsmbios_c && echo yes))
DELL=-Dplugin_dell=true CONFARGS += -Dplugin_dell=true
else else
DELL=-Dplugin_dell=false CONFARGS += -Dplugin_dell=false
endif endif
ifeq (yes,$(shell pkg-config --exists efivar && echo yes)) ifeq (yes,$(shell pkg-config --exists efivar && echo yes))
UEFI=-Dplugin_uefi=true -Dplugin_redfish=true -Dplugin_nvme=true CONFARGS += -Dplugin_uefi=true -Dplugin_redfish=true -Dplugin_nvme=true
else else
UEFI=-Dplugin_uefi=false -Dplugin_redfish=false -Dplugin_nvme=false CONFARGS += -Dplugin_uefi=false -Dplugin_redfish=false -Dplugin_nvme=false
endif endif
ifneq ($(filter $(DEB_HOST_ARCH_CPU),i386 amd64),) ifneq ($(filter $(DEB_HOST_ARCH_CPU),i386 amd64),)
MSR=-Dplugin_msr=true CONFARGS += -Dplugin_msr=true
else else
MSR=-Dplugin_msr=false CONFARGS += -Dplugin_msr=false
endif endif
CONFARGS += -Dplugin_dummy=true -Dgtkdoc=true -Dsupported_build=true
%: %:
dh $@ --with gir dh $@ --with gir
@ -60,7 +62,7 @@ ifeq (ubuntu,$(SB_STYLE))
endif endif
override_dh_auto_configure: override_dh_auto_configure:
dh_auto_configure -- $(UEFI) $(DELL) $(FLASHROM) $(MSR) $(CI_FLAGS) -Dplugin_dummy=true -Dgtkdoc=true -Dsupported_build=true dh_auto_configure -- $(CONFARGS)
override_dh_install: override_dh_install:
find debian/tmp/usr -type f -name "*a" -print | xargs rm -f find debian/tmp/usr -type f -name "*a" -print | xargs rm -f