Bug#934640: apache2: fails to install, a2enconf missing
Hi,
thanks for fixing it.
I see you've removed the '-i' passed to dh_link in override_dh_link.
This is inferior to my suggested patch of replacing override_dh_link
with override_dh_link_indep for the following reasons:
1. it overrides dh_link in all cases (arch-dep and indep) and thus
(tries to) executes jdupes even when building arch-dep only
(dpkg-buildpackage -B as commonly done on buildds), leading to this
message during build:
---snip---
dh_perl -a
debian/rules override_dh_link
make[1]: Entering directory '/build/apache2-2.4.39-2'
dh_link
jdupes -rl debian/apache2-doc/usr/share/doc
could not stat dir debian/apache2-doc/usr/share/doc
No duplicates found.
make[1]: Leaving directory '/build/apache2-2.4.39-2'
dh_strip_nondeterminism -a
---snap---
you got lucky that jdupes does not exit with 0 on non-existing paths,
otherwise your build would have failed. I suggest not to leave this to
chance that nobody ever fixes jdupes to exit with 1 in these cases, as
it would be properly doing so in the first place.
2. creates *all* symlinks in debian/$package for all arch-dep packages
too when build indep only. While a purely cosmetical thing (as the
"speed" gain of not creating the symlinks is negligible, it clutters
debian/* during build unecessarily and makes finding errors/debugging
build things a tad more ugly.
Therefore, please consider using override_dh_link_indep and pass '-i'
again to dh_link, like my original patch:
https://git.progress-linux.org/distributions/engywuck-backports/packages/apache2/commit/?id=80cecebdf64409d493baaa8cec4de5cf801934eb
Regards,
Daniel
Reply to: