Alessandro Ghedini wrote: > > Dear mentors, > > I am looking for a sponsor for my package "ulatencyd".
Hi Alessandro, here is my (promised) review, with some delay; please forgive me for that; life took over… > ulatencyd - Daemon to minimize latency on a linux system using cgroups > > The package appears to be lintian clean. Confirmed. > The upload would fix these bugs: 610495 (ITP) Now some questions: * Why don't you ship the systemd service file? With systemd around the corner, you will certainly end up adding it in the future. And why are you stripping it away with a patch (where you could dh_auto_install to debian/tmp and have a "ulatencyd.install" file to opt files _in_) ? I would just correct the path in this install file and be done with it. * Your debian/init.d isn't named correctly (IMHO). man dh_installinit tells us that it should be named debian/ulatency.init (or debian/init, but I very much prefer being explicit). As for the names, it's the same for logrotate, manpages and docs (but don't worry, it's mostly a matter of taste). * Deactivation of the tests: why do you disable the tests ? Build tests should be run and they should not fail (obviously…). You should either comment your debian/rules explaining the reasons or (preferably) convince upstream to patch (or patch yourself) the tests in order to be able to run within the buildd environment. * debian/gbp.conf should not be in the source package; having a debian/source/local-options to filter it out sounds nice. Otherwise it sounds nice. Cheers, OdyX -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/iklrjp$k1i$1...@dough.gmane.org