© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Review for Package: libswitch-perl
[Summary]
MIR team NACK
There seem to be better, more modern ways to do this without adding a very
old dependency to main. This would need a very good explanation why it would
be impossible to use perl builtins of the this decade to solve the same.
[Duplication]
This looks wrong to be added to main in 2023. I mean this package clearly
states "This is an obsolete module provided for compatibility since it is being
removed from the core. For perl 5.10 and above the "given/when" builtins
are much preferred."
Perl 5.10 was in Ubuntu since intrepid, that is 13 years ago.
New code should just use [1]
So if this is only for compat with very very old code, why is it suddenly
needed in devscripts? I have found [2] to be the reason which on a glance
does look wrong to force the use of this perl module.
First I thought it might be due to the use of regex, but even that seems to
be known strange bugs in switch and workign fine with given/when [3].
Looking forward to 24.04 as the next LTS it feels wrong to add this for
another decade. Did I mention there was no update since 2014?
Instead please consider to recode this new code, propose it to Debian and
get rid of the dependency.
[1]: https:/
[2]: https:/
[3]: https:/