Merge ~mirespace/+git/team-subscriptions:mir_libmail-dmarc-perl_lp2023971 into ~canonical-server/+git/team-subscriptions:main

[Miriam España Acebal]

Hi team,

adding here all the requested packages to subscribe to as part of the libmail-dmarc-perl MIR process (bug 2023971).

[Lucas Kanashiro (lucaskanashiro)]

Thanks for the MP Miriam! Maybe is it a good idea to mention in your git commit message that we are subscribing to all those packages because of spamassassin. In a first glance, one cannot see the relation of those perl libraries with the packages we maintain.

And I went through the bugs and some of them are requiring some action from you, this is just a heads-up, you are likely aware of that.

review: Needs Fixing

[Miriam España Acebal (mirespace)]

Hi Lucas and thanks for the review :).

I updated the commit message accordingly, thanks for noting that ("Miriam, why are you working with perl modules?").

Yes, I have this:

* libemail-mime-perl: 2030880 -> ACK with 3 requires TODOs, subscribe team plus security review (sc wip)
   - libemail-messageid-perl: 2030956 -> ACK -> subscribe team
   - libemail-mime-contenttype-perl: 2030962 -> ACK
       + libtext-unidecode-perl: 2031109 -> ACK
   - libemail-mime-encodings-perl: 2031487 -> ACK (fix commited) Done.
   - libemail-simple-perl: 2031491 -> ACK, team subscribe plus security review (sc review)

* libfile-sharedir-perl: 2039566 -> ACK, subscribe, 1 recommend
   + libclass-inspector-perl: 2039569 -> ACK

* libnet-idn-encode-perl: 2038929 -> change by libnet-libidn-perl

* libnet-ip-perl: 2039456 -> ACK: The package does not seem to require a security review at the moment, however requesting team should reach an agreement with security team on how this package will be dealt in terms of security.

* libregexp-common-perl: 2039563 -> ACK, subscribe

I started subscribing them all as a first action (because some of them only need that), but I can wait until after I resolve the issues for you to approve this PM. Then, I'll ping you again if you prefer this.

[Lucas Kanashiro (lucaskanashiro)]

Thanks Miriam. I believe there will be no way back from this, so I think we can already subscribe the team to those packages. Let me know if you want this merged now.

review: Approve

[Miriam España Acebal (mirespace)]

 Hi Lucas. Yes, please, merge it. Thanks!

[Christian Ehrhardt (paelzer)]

ack and merged

diff --git a/subscriptions.yaml b/subscriptions.yaml
index d5ab21c..0f90191 100644
--- a/subscriptions.yaml
+++ b/subscriptions.yaml
@@ -93,25 +93,33 @@ ubuntu-server: !!set
   ? libapache2-reload-perl
   ? libbackuppc-xs-perl
   ? libcacard
+  ? libclass-inspector-perl
   ? libconfig-general-perl
   ? libdbi
   ? libdbi-drivers
   ? libdumbnet
   ? libecap
+  ? libemail-messageid-perl
+  ? libemail-mime-perl
+  ? libemail-mime-contenttype-perl
   ? libemail-mime-encodings-perl
+  ? libemail-simple-perl
   ? libesmtp
   ? libfcgi
   ? libfcgi-perl
+  ? libfile-sharedir-perl
   ? libhx
   ? libiscsi
   ? liblinear
   ? liblockfile
+  ? libmail-dmarc-perl
   ? libmaxminddb
   ? libmemcached
   ? libmime-tools-perl
   ? libmnl
   ? libmspack
   ? libnet
+  ? libnet-ip-perl
   ? libnetfilter-conntrack
   ? libnginx-mod-http-geoip2
   ? libnl3
@@ -123,6 +131,7 @@ ubuntu-server: !!set
   ? libpcap
   ? libqb
   ? librabbitmq
+  ? libregexp-common-perl
   ? librtas
   ? libsdl1.2
   ? libsdl2
@@ -130,6 +139,7 @@ ubuntu-server: !!set
   ? libslirp
   ? libsodium
   ? libstatgrab
+  ? libtext-unidecode-perl
   ? libtime-parsedate-perl
   ? libtirpc
   ? libtommath