Bug#895659: gutenprint: please stop using --ignore debhelper option
Source: gutenprint
Version: 5.3.0~pre1-2
Severity: wishlist
Tags: patch
Hello.
Debhelper is considering deprecating the --ignore option.
It is used once in gutenprint.
Commit 5b3f43f1 stops installing the usb directory on non Linux architectures.
Among other changes, it moves "usr/share/cups/usb" from
debian/printer-driver-gutenprint.install to the freshly created
debian/printer-driver-gutenprint.install.linux.
The intent was probably that both files are used when
DEB_HOST_ARCH_OS=linux, else only the first one.
This does not work because debhelper only considers the most specific file.
Commit 49ffeb1f fixes this in debian/rules by adding:
ifeq ($(DEB_BUILD_ARCH_OS),linux)
# Install common files
dh_install -pprinter-driver-gutenprint --ignore=debian/printer-driver-gutenprint.install.linux
endif
right before
dh_install -a
The effect is that
* on linux, the first dh_install run installs normal files,
and the second run installs the usb directory.
* on other architectures, dh_install is only run once and installs the
normal files.
If all this is correct, the attached patch should have the same effect
in a more readable way, and without --ignore.
Thanks.
--- a/debian/printer-driver-gutenprint.install.linux
+++ /dev/null
@@ -1 +0,0 @@
-usr/share/cups/usb
--- a/debian/rules
+++ b/debian/rules
@@ -38,10 +38,9 @@ override_dh_clean:
override_dh_install-arch:
ifeq ($(DEB_BUILD_ARCH_OS),linux)
- # Install common files
- dh_install -pprinter-driver-gutenprint --ignore=debian/printer-driver-gutenprint.install.linux
+ dh_install -pprinter-driver-gutenprint usr/share/cups/usb
endif
- dh_install -a
+ dh_install -a --remaining-packages
# Remove rpath
find debian/printer-driver-gutenprint/usr/lib -type f -perm /0111 -print0 | xargs -0 chrpath --delete
# Install ppd-updater file
Reply to: