Package: debhelper
Version: 13.7.1
Severity: wishlist
Dear Maintainer,
The roundcube packages ships a temporary directory which is cleaned via
cronjobs and which I'd like to define via tmpfiles.d(5) instead.
debhelper 13.7.1 adds the following snippet to the postinst script:
# Automatically added by dh_installtmpfiles/13.7.1
if [ "$1" = "configure" ] || [ "$1" = "abort-upgrade" ] || [ "$1" = "abort-deconfigure" ] || [ "$1" = "abort-remove" ] ; then
# In case this system is running systemd, we need to ensure that all
# necessary tmpfiles (if any) are created before starting.
if [ -z "${DPKG_ROOT:-}" ] && [ -d /run/systemd/system ] ; then
systemd-tmpfiles --create roundcube.conf >/dev/null || true
fi
fi
# End automatically added section
So this is a no-op unless PID1 is systemd, which is indeed documented in
dh_installtmpfiles(1):
debian/package.tmpfiles
[…] Note that the "tmpfiles.d" mechanism is currently only used by systemd.
I personally don't really mind the ‘Depends: systemd’, but this breaks
DEP-8 tests in shallow isolation levels.
It seems the snippet guard could be relaxed somewhat: systemd ≥249.3-2
provides systemd-standalone-tmpfiles which ships /bin/systemd-tmpfiles,
and with ‘Depends: systemd-standalone-tmpfiles | systemd’ I was able to
fix the autopkgtest by unconditionally calling `systemd-tmpfiles --create roundcube.conf >/dev/null || true`
in the postinst.
Wouldn't it make sense to replace the guard with `command -v systemd-tmpfiles`
so the command is run whenever systemd-tmpfiles(8) exists? This appears
to work regardless of whether it's shipped by systemd or systemd-standalone-tmpfiles,
even when /run/systemd/system doesn't exist (such as in a minimal chroot).
Thanks for maintaining debhelper!
We believe that the bug you reported is fixed in the latest version of
debhelper, which is due to be installed in the Debian FTP archive.
A summary of the changes between this version and the previous one is
attached.
Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to 1013969@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Niels Thykier <niels@thykier.net> (supplier of updated debhelper package)
(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)
Format: 1.8
Date: Sun, 03 Jul 2022 09:04:23 +0000
Source: debhelper
Architecture: source
Version: 13.8
Distribution: unstable
Urgency: medium
Maintainer: Debhelper Maintainers <debhelper@packages.debian.org>
Changed-By: Niels Thykier <niels@thykier.net>
Closes: 997961 1006805 1010591 1010594 1011099 1011256 1011404 1012777 1013969
Changes:
debhelper (13.8) unstable; urgency=medium
.
[ Niels Thykier ]
* dh_perl: Fix incorrect binary versioning caused by a regression
in 13.3. Thanks to Niko Tyni for reporting the issue.
(Closes: #997961)
* d/control: Bump Standards-Version to 4.6.1 - no changes required.
* root_sequence.pm: Run `dh_installsysusers` before
`dh_installtmpfiles` (only affects compat 14 or later). Thanks
to Nicholas Brown for the suggestion. (Closes: #1011099)
* autoscripts/postinst-init-tmpfiles: Remove runtime check on the
system having systemd as init system.
* dh_installtmpfiles: Replace runtime check on the system running
under systemd in generated `postinst` script with a dependency
on `systemd | systemd-tmpfiles` added to `${misc:Depends}`.
Thanks to Guilhem Moulin for the suggestion. (Closes: #1013969)
* dh_installsystemd: Ditto.
* dh_installinit: Ditto.
* debhelper-compat-upgrade-checklist.pod: Correct description for
the addition of dh_installsysusers in compat 14.
* meson.pm: Use `meson install` instead of `ninja install` for
dh_auto_install in compat 14. Thanks to Andrea Pappacoda for
the suggestion. (Closes: #1006805)
.
[ Max-Julian Pogner ]
* Dh_Lib.pm: Remove double semi-colon. (Closes: #1010591)
* Dh_Lib.pm: _strip_spaces now explicitly returns undef on undef
argument.
(Closes: #1010594)
.
[ Andrea Pappacoda ]
* cmake.pm: Set FETCHCONTENT_FULLY_DISCONNECTED to true. This
ensures that the `FetchContent` module does not attempt to
use internet connection.
.
[ Vagrant Cascadian ]
* debhelper-compat-upgrade-checklist.pod: Update documentation
for compat 14 change. (Closes: #1011404)
.
[ Ferenc Wágner ]
* debhelper.pod: Correct link to the new upgrade checklist.
(Closes: #1012777)
.
[ Luca Boccassi ]
* dh_installman: Avoid false positive language detection on
shared libs.
.
[ Translations ]
* Update Portuguese translation (Américo Monteiro)
(Closes: #1011256)
Checksums-Sha1:
ab83b353a662b126d7da2026ac0ad86baa4817c2 1781 debhelper_13.8.dsc
490b5e4c26597e05ea9b023ba245467aeb173d82 571980 debhelper_13.8.tar.xz
f8ec73d3d3c49891a82040fc2ec2f26284a6ff0b 5172 debhelper_13.8_source.buildinfo
Checksums-Sha256:
0deea560ceff3f0e0bc310e8e69b1d868cae74099e4ca3ddaa9bf3a5078f6959 1781 debhelper_13.8.dsc
3d22db5662a54e3bbe67de699d1e7fd88f920fb1fc1db58a1c45c164f4fb96ea 571980 debhelper_13.8.tar.xz
3dea3d634f304086e0da639c0a16efd0fb591e5cd600db3bdfc08fab83efcfd5 5172 debhelper_13.8_source.buildinfo
Files:
856c26b5f503ec889ceb4f3613826997 1781 devel optional debhelper_13.8.dsc
504e6ddd34f8163c091622e510eb4254 571980 devel optional debhelper_13.8.tar.xz
02353b46898a8c129a3d6dea14331157 5172 devel optional debhelper_13.8_source.buildinfo
-----BEGIN PGP SIGNATURE-----
iQJGBAEBCgAwFiEE8f9dDX4ALfD+VfsMplt42+Z8eqwFAmLBXJkSHG5pZWxzQHRo
eWtpZXIubmV0AAoJEKZbeNvmfHqsB1AP/jEqU/ia8TMTRW+bEDWVGq+WWqzYs3R+
7OJW6Vx4ZRIGLu97r5S0TG/U6xlVn3evLSWclz617+CLYLqThzpX7gv0Jf1E2lTw
LGbvjieMmGFCR/rX6i6tFHMs6dPqbzDCUHR2ES15cHqSmCt+U+m7vLLt0H83f94p
Fnfbjly8AXLmPv6UikLpViyT3RNQTMkwpHBknaH/8LCwu1MWEev8HRQI3WZoZ7dJ
tysZDTk6YN74RCrIy82uA9qzpNzUW4UxkuzEZ2y28bVtsXGZV0KOMB/8S5uX1sXy
FboQfuePazknAtZrqQVoe5uqog/mYhi4ogLOnmiy2xwS5sNu729wJGDvsa1Lg3WT
HuToCc1bHUIe+aJHVz0vRx0lU0oBk+16mZE6YX/tGuvJc9faIPFOoYTyCSWNos0v
knRYe8wbNFsOynF0uXgWCU3yyCVbWITDtzPV55AziADyakkgqmKUHRpz8F45Mb84
SM7QN35GcKQhnoBic/dMCedHlFhsSYZTO3SmB1XROf3STxR46q5/U7+VgNH6ZfO7
zUkAY5B2/BtQk3lc83ld/JEStW+KiZi0p6CTaj6fMxT/X1gqyyBKQIx+Q1KiANKo
V61mrR968Io3s9uveAURCLLPcZICqn2zXAdlGy7xC4CjKkdy+5CvUgzEBaZoH/Zq
doBEMIudGdZK
=SvjG
-----END PGP SIGNATURE-----