Hi,

I'm sorry to tell you that I still got the duplicated diversions wrong.
The full story is at
https://lists.debian.org/debian-devel/2025/01/msg00125.html. In any
case, I managed to reproduce bad scenarios for bfh-container. The main
issue here is that --rename may be specified for the canonicalized
diversion when the actual file is aliased and then it may be renamed in
a bad way making dpkg unhappy. Additionally, the order of diversions was
wrong as diversion targets must not overlap. Adding the canonicalized
diversion was failing as the aliased diversion had not yet changed its
target.

I'm attaching a patch that fixes both and also attaching my test cases
(that are all passing now). I added more comments such that the mess
becomes easier to understand by my future self and others.

Helmut

This never ends. :-(

Evidently, I missed a relevant test case (reinstalling and removing the
package) and of course it would fail. Here is a replacement patch and a
new test case.

This time, I am completely avoiding --rename and handling all the cases
explicitly doing the renames where needed. Hope this one passes the test
of time.

Helmut

We believe that the bug you reported is fixed in the latest version of
bfh-metapackages, 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 1092955@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Daniel Baumann <daniel.baumann@progress-linux.org> (supplier of updated bfh-metapackages 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: Thu, 16 Jan 2025 06:47:00 +0100
Source: bfh-metapackages
Architecture: source
Version: 20211009-32
Distribution: sid
Urgency: medium
Maintainer: Daniel Baumann <daniel.baumann@progress-linux.org>
Changed-By: Daniel Baumann <daniel.baumann@progress-linux.org>
Closes: 1092955
Changes:
 bfh-metapackages (20211009-32) sid; urgency=medium
 .
   * Uploading to sid.
   * Applying patch from Helmut Grohne <helmut@subdivi.de> to fix --rename
     flags on duplicated diversions (DEP17 M18, Closes: #1092955).
Checksums-Sha1:
 7e71d6f81078edee0afad7869136b3d5e00bbf2a 1816 bfh-metapackages_20211009-32.dsc
 e072d3bc6220fec155de3044804c50b05cb9be47 6520 bfh-metapackages_20211009-32.debian.tar.xz
 14d9851ff3f18faf92f041734cd76f6efe86d53d 7378 bfh-metapackages_20211009-32_amd64.buildinfo
Checksums-Sha256:
 6ef61a1e1f999c320801f8cca9779d59934f215c735322ce3179af985015b71b 1816 bfh-metapackages_20211009-32.dsc
 e842054a692af29632737e9b5a17636a54ee6d1d66cb6cdddd0a2abe00a2d2aa 6520 bfh-metapackages_20211009-32.debian.tar.xz
 e8504154562a28585f5663f17e894c06bcc6791b224913befffcf2f8d9cc2a2f 7378 bfh-metapackages_20211009-32_amd64.buildinfo
Files:
 cd1b2c9282c4080018d4244da48c558c 1816 metapackages optional bfh-metapackages_20211009-32.dsc
 a71a00a93df786acb637871ff290cb52 6520 metapackages optional bfh-metapackages_20211009-32.debian.tar.xz
 ec2a9eb116ed1b1eaa643d7f66e4999f 7378 metapackages optional bfh-metapackages_20211009-32_amd64.buildinfo
-----BEGIN PGP SIGNATURE-----

iHQEARYIAB0WIQQmmGg4gLaoSj0ERgL7tPDoCoAiLwUCZ4iddQAKCRD7tPDoCoAi
L69RAPiVGhz2sqGhzajJLWQupN7q02v0fmIcMc9XI1PtcdrgAP9Zqns+2O4O29HB
n1yckRIBDBuTLwfE68BGXT+Ix/tHDw==
=vRBi
-----END PGP SIGNATURE-----