Your message dated Fri, 22 Mar 2024 14:52:53 +0000
with message-id <[email protected]>
and subject line Bug#1065742: fixed in debhelper 13.15
has caused the Debian Bug report #1065742,
regarding dh_install: cannot use empty lines or comments in substitution
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.)


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

Hello,

supporting build-profiles like nodoc with dh_install seems to cumbersome.

If the not-always-built files were listed in debian/foo.install the
build would fail when the build profile was active (because they were
not built/available).

As second plan I tried to conditionally install them directly using
command-line arguments to dh_install like this:

override_dh_install:
ifneq ($(filter nodoc,$(DEB_BUILD_PROFILES)),)
        dh_install
else
        dh_install -Ngnupg-l10n
        dh_install -pgnupg-l10n --autodest \
                'usr/share/gnupg/help.*.txt' \
                usr/share/gnupg/help.txt
endif
This also failed, since for a --build=any build debhelper decides that
"dh_install -pgnupg-l10n ..." is a noop and does not add the files to
debian/.debhelper/generated/gnupg-l10n/installed-by-dh_install, in turn
causing a build-error by dh_missing.

The third plan was to make evaluation of debian/gnupg-l10n.install dynamic
by using substitutions. I tried these in the install files
* ${env:LISTOFSTUFF} (with LISTOFSTUFF being set to a file or directory or
  an empty string (nodoc-case)
*  "${env:DISABLE}usr/share/foo" and setting DISABLE='# ' for the nodoc
   case.
Neither of these worked, comment and empty-line processing seems to be
disabled for expanded stuff.

The whole thing was a little bit frustrating by being too complicated
and I think that plan #3 should have worked according to the docs. Which
is why this bug-report is about this case.

cu Andreas

PS: I ended up using an executable debian/gnupg-l10n.install, which
works, but I' d rather keep all nodoc processing in one place in
debian/rules.

--- End Message ---
--- Begin Message ---
Source: debhelper
Source-Version: 13.15
Done: Niels Thykier <[email protected]>

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: Fri, 22 Mar 2024 15:25:39 +0100
Source: debhelper
Architecture: source
Version: 13.15
Distribution: unstable
Urgency: medium
Maintainer: Debhelper Maintainers <[email protected]>
Changed-By: Niels Thykier <[email protected]>
Closes: 1001403 1014537 1021877 1030011 1036760 1062508 1063126 1065742
Changes:
 debhelper (13.15) unstable; urgency=medium
 .
   * d/changelog: Fix typo of Felix Geyer
   * dh_makeshlibs: Ensure t64:Provides is always provided (but
     sometimes empty)
   * Replace `d/compat` with a `X-DH-Compat` field in compat 14+.
     (Closes: #1021877)
   * Ensure values to --dbg-package and --link-doc are known packages
   * PROGRAMMING.md: Add a note about `dh_assistant`
   * Make it easier to re-bootstrap without `dwz` (and also `man-db`).
     This is a new build profile that build `debhelper` with fewer
     dependencies. Note that this profile causes `debhelper` to produce
     a different result in some cases. Packages built with this minimal
     version of debhelper should be rebuilt when bootstrapping is
     complete.
   * dh_shlibdeps: Use ${shlibs:Pre-Depends} for Essential packages
     in compat 14+.
   * dh_gencontrol: Automatically apply relationship substvars in
     compat 14+
   * Clearly document how to do filtering (which is via dh-exec)
     (Closes: #1065742)
   * Reject debhelper config files without a package prefix for
     multi-binary packages in compat 15. Starting in compat 14, it will
     now trigger a warning to have `debian/install` instead of
     `debian/package.install` when `debian/control` contains 2 or more
     binary packages. In compat 15, this is promoted to an error.  The
     main exceptions are files like `debian/changelog`, `debian/NEWS`,
     and `debian/copyright`, which are applied to all packages and can
     still be used without a prefix. (Closes: #1014537)
   * cmake.pm: Pass ASFLAGS as ASMFLAGS to cmake [c14] (Closes: #1036760)
   * meson.pm: Pass --auto-features=enabled in compat 14+
     (Closes: #1030011, #1062508)
   * Dh_Lib.pm: Simplify reset_perm_and_owner so it only works with int
     now. The only misbehaving call-sites where in debhelper itself
     according to codesearch.debian.net and were fixed as well.
   * Dh_Lib.pm: Bump perl version requirement to 5.28 (Closes: #1001403)
   * dh_makeshlibs: Skip directories known not to contain SONAMEs
     (Closes: #1063126)
   * Restructure debhelper upgrade compat checklist for compat 14 in an
     attempt to make it easier for consumers.
Checksums-Sha1:
 427af2800eec90f73f0e8eddc525d08093597a63 1462 debhelper_13.15.dsc
 ffc6783ba068c5e61b77c9a1ba312014ca0f24d8 602412 debhelper_13.15.tar.xz
 1da4a3e2307ea0f604a02e83055000c439216312 5049 debhelper_13.15_source.buildinfo
Checksums-Sha256:
 b29dcec20ca1f546fbbe77644f8429a4b11cbdea0c8ffd781a9804b61620c2a0 1462 
debhelper_13.15.dsc
 752202483eb6c5eae9599b2ee0e80090165d221e20cdeda75f377ad67f4fc84b 602412 
debhelper_13.15.tar.xz
 4b299c5c6fef4e39d1bda32c4585e73bb87d89bf7c958223c667d31e445ef259 5049 
debhelper_13.15_source.buildinfo
Files:
 c8fea2deceec16a304e47a1f23d9c207 1462 devel optional debhelper_13.15.dsc
 20607bc64567ddf155d968653353669a 602412 devel optional debhelper_13.15.tar.xz
 778267813d27c125ec561d47fda541f5 5049 devel optional 
debhelper_13.15_source.buildinfo

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

iQFGBAEBCgAwFiEE9ecZmu9eXGflVYc/dA1oiINl0okFAmX9ldISHG5pZWxzQHRo
eWtpZXIubmV0AAoJEHQNaIiDZdKJJPkH/2kNXgHiGR1MkA3szMg79FkF5l7v67Ui
0Wqoyhk1o+qYUnTVS8JD2gELySkMZBxz6LaklnmPnMTI/MuOw8wym0BZ6GfvLPbA
RCvqnnPDJBAzESijTYdIkcH0oTSBpGFhluZWPs9JuSLTzhu13SW34Qpbdn6uO4Gd
fvgB8zRKH3+DLU9PGzxZuKuZAqJbl6aiTcxHw4VKvAFVBov4AqcBLuBqOo6xwqyM
32gnd4/OPeSLbAcuse9hGga3tReFGIktyADgXfwd/kEgzOkTqKvW7EQ88KK9FMHT
lJcSmBwbxRYceZrjVpxA7c5NJPdgc43XDBhO0aK5bdfIevXSE3lWDI8=
=CKm5
-----END PGP SIGNATURE-----

Attachment: pgpdSnCaQlG4E.pgp
Description: PGP signature


--- End Message ---

Reply via email to