Your message dated Mon, 27 Jan 2020 21:19:19 +0000
with message-id <[email protected]>
and subject line Bug#885994: fixed in debhelper 12.9
has caused the Debian Bug report #885994,
regarding New override type to run additional commands: after_dh_*
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
885994: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=885994
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: debhelper
Version: 11
Severity: wishlist

Hi,

the existing dh override mechanism is perfect to do things like:

override_dh_strip:
        dh_strip -Xfoo

It is less optimal for adding (rather than overriding) commands:

override_dh_fixperms:
        dh_fixperms
        chmod 4755 debian/foo/usr/bin/foo

This has several problems:
* it's ugly because it repeats "dh_fixperms", and it's also
  error-prone because the explicit dh_fixperms might be forgotten
  (#884419, #884815, #884817),
* it doesn't use the global dh options passed to dh $@ unless they are
  repeated here,
* it defeats parallelization (#881175),
* it might or might not work well with override_dh_*-(arch|indep).

(Examples directly from dh(1)).

I'd like to see after_dh_* as alternative to override_dh_*:

after_dh_fixperms:
        chmod 4755 debian/foo/usr/bin/foo

This looks much cleaner to me. (In fact, the reason I'm filing this
wishlist bug is that I'm always twitching a bit when I have to repeat
dh_foo in the override_dh_foo incantation.)

Thanks for considering,
Christoph

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message ---
Source: debhelper
Source-Version: 12.9

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 [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Niels Thykier <[email protected]> (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 [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Mon, 27 Jan 2020 20:45:05 +0000
Source: debhelper
Architecture: source
Version: 12.9
Distribution: unstable
Urgency: medium
Maintainer: Debhelper Maintainers <[email protected]>
Changed-By: Niels Thykier <[email protected]>
Closes: 568897 885994 949396 949538 949635
Changes:
 debhelper (12.9) unstable; urgency=medium
 .
   [ Niels Thykier ]
   * debhelper.7: Clarify how DH_OPTIONS works and that some
     third-party tools may ignore it if they do not support
     it.  Thanks to Andrej Shadura for reporting the issue
     over IRC.
   * doc/PROGRAMMING: Introduce a section on the NOOP PROMISE
     comment.
   * dh: In compat 13, skip calls to hook and override targets
     for dh_auto_test, dh_dwz and dh_strip.  Thanks to
     Mike Hommey for suggesting it.  (Closes: #568897)
   * Dh_Getopt.pm: Remove support for "--ignore".  No package
     uses it any more.  For most parts, it has been replaced
     by the "3.0 (quilt)" source format.
   * debian/changelog: Clarify that substitutions change in
     12.8 only apply to the maintscript file (and not
     maintainer scripts) for dh_installdeb.
   * debhelper.7: Improve text in compat 13 section.
   * autoconf.pm: Only consider "configure" as an autoconf
     script if it is a file (and not a directory).  Thanks to
     Andrius Merkys for reporting the bug and providing a
     patch.  (Closes: #949396)
   * debian/control: Bump Standards-Version to 4.5.0 - no
     changes required.
   * dh_shlibdeps: Pass "--" in front of filenames passed to
     file(1).  Thanks to Christoph Biedl for the suggestion.
     (Closes: #949538)
   * dh_strip: Ditto
   * dh.1: Improve documentation for override targets and
     include a section on the new hook target.
     (Closes: #885994)
   * Dh_Lib.pm: Make it possible to disable the command
     logging by passing "inhibit_log => 1" to the init sub.
     Furthermore, the logging is now only enabled when init
     is called (and not disabled by that option or
     inhibit_log()).  This prevents Dh_Lib from triggering
     a log write just by being loaded.
   * dh: Refactor addon loading.  As a side-effect, dh no
     longer leaks as many functions into the scope of addons.
     Addons depending on the previous behaviour may now fail
     due to calls to undefined subroutintes and will need to
     be fixed.
   * dh_installinitramfs.1: Correct the name of localized
     versions of dh_installinitramfs so that man can find
     them.
 .
   [ Thorsten Glaser ]
   * dh_auto_*: Make it possible to disable build systems by
     passing --buildsystem=none.  Previously this would
     require overriding each and every dh_auto_* tool.
     (Closes: #949635)
Checksums-Sha1:
 d6760cfaea25b56d0b3d235d33e64db07e9e1729 1835 debhelper_12.9.dsc
 f6deb32e6c3983371baa4bdda0e327506b4ba5a8 521116 debhelper_12.9.tar.xz
 f325d2834f397c56da86205c9d29f0b04ca8ca00 4613 debhelper_12.9_source.buildinfo
Checksums-Sha256:
 7db1ce578c248d92824af25638fea8c0c02595ecfb4be1445ac18bd76cee5f54 1835 
debhelper_12.9.dsc
 d627404c45a9f4012624fd38f1dac22cb66bbae6c0d158abf01a9785d2154a97 521116 
debhelper_12.9.tar.xz
 497fa5cd6034bc5c392b0938681ad94ec305a161f78f3ebeded5bb101e60d223 4613 
debhelper_12.9_source.buildinfo
Files:
 566fc7cc7cd59c36073234b29ff29a20 1835 devel optional debhelper_12.9.dsc
 b3cf3a7d68a98299f9a11bc4370d43f7 521116 devel optional debhelper_12.9.tar.xz
 6963248dc97bf4db79d1b0682e825271 4613 devel optional 
debhelper_12.9_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJGBAEBCgAwFiEE8f9dDX4ALfD+VfsMplt42+Z8eqwFAl4vUB0SHG5pZWxzQHRo
eWtpZXIubmV0AAoJEKZbeNvmfHqsrRUQAJDPIBWPKKSLow81PR7LKbYGmoSkMMlx
wNi9U9ntqcfw90bH/7Q19WGv2g0wd1IfE4/NanQymDVK19/5zZTvoBvGmcGuZb/m
HTPt4PwCgWqTt7bVhcTm/XydGIpRPzn0MyO+A82u7g1rrQYQodDVMXIVhaOgonmu
7mQDfnCKQ85jW4dvh60/66ggdRlNjl9q9ZxxNH6P2OwL5rVGV1ouJwGCkx+8Ya6y
mLMNtaICdAtDZ+4eCtngdLbrslcuLoQ8k55qWKfu7JGeFBQysdtxL5WpKdBYOtSj
i6+7lgYtrIM8idrDKKvCy3wPxisDnnVkS1dSCpmQs3bnNNJBNIntpz/eRvUBVNrC
3vagb8ojzkwQcbSt0XKaBsoadDab4kHtOayvAWovkEKDxZcMFZNftQneirAMneSV
JwbSsmzc3aGOxFq0ke0cjOvm8vuhaWK9uGq5LF06eBPng5JgLBfs4nCpIREFp4lX
FqQOO2RPLD/Ig9CJLw6Tkhe9jFnRcckbm9AmyK5GSOJKS1hPSj7L7x+fPvCR8KGm
lpVH2ayYfEq7kKndoZk0NFDVjCXXOYYxnbV+o3vTSGtMoSs7ITZ2gTvIpExVG2We
kc+6NTgXyd7d/5rFXz2EDPHiEUaS7HNCcE9cc+oys6oH1+Yjg2xbO3MK/KRPmzRZ
ImqkPwi2a7cc
=4IQZ
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to