Hey Dan! Thank you for your review. I understand that needrestart as is now has no testing story, which is quite unfortunate. Do you have some testing scenario that you would consider as a minimum requirement for the package to get accepted into main? needrestart is a project which we are not upstream for, meaning that adding really useful tests would ultimately require help from upstream. Realistically, the only test I can tinker up before 21.04 is ready would be a very very simple autopkgtest that would be running the needrestart command and checking its output. Even then, well, we'd be introducing a delta against Debian - as I'm not confident that Debian would be willing to accept something as simple as that.
This is something that has not been tackled since long. Debian has a wishlist bug open for it since 2016: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=810584 Other than that, the source includes no testing as well, it's a very specific fruitcake. It's unfortunate, as I guess the perl module might have had some unit testing specified, but adding that without upstream approval would be weird. Seeing that it's late in the cycle and so far the regression history of the package is rather low, could we settle on us reaching out both upstream and to Debian regarding test-story improvements and in the meantime accepting the package into main as is? Or, if that's in any way helpful, additionally creating such an 'easy' sanity-test autopkgtest (+ sending it to Debian to get rid of the delta)? Cheers! ** Bug watch added: Debian Bug tracker #810584 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=810584 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1907422 Title: [MIR] needrestart + dependencies To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/libintl-perl/+bug/1907422/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
