Hi, after today update to amavisd-new/amavisd-new-2.12.0-r2 amavisd-new stopped working and killing my Mailserver with it. It was because of dev-perl/Archive-Zip missing. Seems like Commit https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fe4d2530ec228792fac96fd3c04599f181056dfd introduced that. Please readd the dep. Reproducible: Always
I was asked to remove the dev-perl/Archive-Zip by the reviewer [1], so that is what I did. I will only put it back in if given the OK. [1] https://github.com/gentoo/gentoo/pull/14656#discussion_r380494273
(In reply to Tetja Rediske from comment #0) > after today update to amavisd-new/amavisd-new-2.12.0-r2 amavisd-new > stopped working and killing my Mailserver with it. I missed mentioning this earlier: A reduced dependency set does not mean existing packages will be removed by executing "emerge -u". It definitely did not happen on our mail servers.
Sure, a -u won't, but a "emerge -c" will. Since I had no other package using it, it happily removed it.
Sure, I should have paid better attention. In the Discussion above they are removed because of "always satisfied", which make no sense to me. Why would even clamav always be satisfied? (next thing i will check on my machine i guess) Regarding to the dev-perl/Archive-Zip nothing else is pulling it in and it does not look optional to me. Even if all of those are optional, maybe a emerge message would be a good way to tell, especially after that change took place.
(In reply to Tetja Rediske from comment #3) > Sure, a -u won't, but a "emerge -c" will. Well, the emerge manual page clearly states: WARNING: Inexperienced users are advised to use --pretend or --ask with this option in order to see a preview of which packages will be uninstalled. Always study the list of packages to be cleaned for any obvious mistakes. This is not meant to troll or disrespect you. emerge -c is always a step to be taken very carefully. So the actual issue is that Archive-Zip has no reverse dependencies anymore: $ equery depends dev-perl/Archive-Zip * These packages depend on dev-perl/Archive-Zip: Juippis, can you please chime in on how you want this handled?
Looks like there was a misunderstanding. Let's wait for juippis response. I guess he just meant ">=", not the complete dep.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7a335f0bf2b81890fc4331c17f88aad03e5ba831 commit 7a335f0bf2b81890fc4331c17f88aad03e5ba831 Author: Joonas Niilola <juippis@gentoo.org> AuthorDate: 2020-02-28 03:52:04 +0000 Commit: Joonas Niilola <juippis@gentoo.org> CommitDate: 2020-02-28 03:52:04 +0000 mail-filter/amavisd-new: add missing dep for dev-perl/Archive-Zip Closes: https://bugs.gentoo.org/710982 Signed-off-by: Joonas Niilola <juippis@gentoo.org> .../{amavisd-new-2.12.0-r2.ebuild => amavisd-new-2.12.0-r3.ebuild} | 1 + 1 file changed, 1 insertion(+)
(In reply to Thomas Deutschmann from comment #6) > Looks like there was a misunderstanding. Let's wait for juippis response. I > guess he just meant ">=", not the complete dep. Indeed that's how we prefer it to improve readability and in some ways portability to ebuilds. So "virtual/perl-IO-Compress" instead of ">=virtual/perl-IO-Compress-1.35" whenever <1.35 hasn't existed in tree for a while. This also flags a CI note after a while in verbose mode. I think my failure in here was however building the PR before waiting an update to it, instead of waiting an udpate and THEN building it. I should've caught that because I use an empty stage3 image to test these. Seems like candrews saved most part of it, https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=45ffd7c8407d4f8c51b513947fd178dd70cec9cf Should be done now, thanks!
