Code review comment for ~michal-maloszewski99/ubuntu/+source/amavisd-new:merge-1.2.13.0-3_mantic

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Hah, I do know ...

We do have this delta "New package amavisd-new-postfix".

But this implies a change, quoted from [1]:
"For earlier compat levels then unless --destdir option is specified, the files are installed into debian/package/ if there is only one binary package. In the multiple binary package case, the files are instead installed into debian/tmp/, and should be moved from there to the appropriate package build directory using dh_install(1) or similar tools."

So the existence of amavisd-new-postfix is what changes the behavior.
From there some Debian changes do not apply well to the Ubuntu build anymore.

That amavisd-new-postfix is part of an "custom ubuntu mail stack" which to admit hasn't got much attention in years and mostly carried forward as-is. In this case it is actually mostly an empty package that does its changes to postfix in the postinst maintainer script.
So it does - itself - not need to install anything of the build in dh_auto_install.

Therefore we should be able to set --destdir

[1]: https://manpages.debian.org/testing/debhelper/dh_auto_install.1.de.html

« Back to merge proposal