-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 2010-12-11 18:29, Joachim Wiedorn wrote: > Dear mentors, >
Hi, I had a quick look at your package, though there are things I may have missed. > I am looking for a sponsor for the new version 1:22.8-9 > of my package "lilo". > > It builds these binary packages: > lilo - LInux LOader - The Classic OS loader can load Linux and others > lilo-doc - Documentation for LILO (LInux LOader) > > The package appears to be lintian clean. > Too much copy paste from the RFS template :) The binary package has 11 warnings (it also triggers a number of less severe tags e.g. I). Anyhow, considering you are targeting this for Squeeze, lintian-cleanless in itself is not a priority. > The upload would fix these bugs: 398957, 400642, 409285, 420587, 427507 > > [...] > I would be glad if someone uploaded this package for me. > > The CTTE disposed at 1st December that I am allowed to be the new > Maintainer for the LILO package: > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=587886#158 > > This update of the LILO package (based on lilo 22.8-8.3) comes with a > small set of bugfixes to work better within Squeeze. The target is to > enter to testing before Squeeze will be released (I hope to the release > team). > Your priority being what it is; you should aim for fixing critical or highly annoying bugs with the minimal number of changes. If the diff gets to large, the release team may choose to reject it because they cannot keep track of the changes. Have you talked with the release team about the bugs you are fixing? I can understand the feeling of wanting to fix as much as possible, but the release team may feel the bugs are not important enough for the changes required. > > FYI here are the last changelog entry: > > lilo (1:22.8-9) unstable; urgency=low > > * New maintainter. > * debian/patches: > - Remove patch 16_geometry.patch for inaccessible disks and add > information in README.Debian about using of 'inaccessible' option. > (Closes: #409285, #400642) > - Fix script checkit for newer gcc. > - Fix for use LVM volume as root device. (Closes: #398957) > - Fix for better computing vmlinux size, upgrade hints in > README.Debian. (fixing sporadic failures). > * Fix hook scripts: check for /sbin/lilo. > * Add lacking file disk.com for creating test floppy. > * Add universal menu image for debian, remove menu image for sarge. > Update of template, postinst and postrm. (Closes: #420587, #427507) > * Set source format 1.0. Add README.source file. I recommend undoing at least the README.source file; first off, it does not describe what it is supposed too and secondly even if you correct it, it has no functional change. The format part is only 1 line, so it is (probably) not going to disturb the reviewer too much. > * Update watch file to new alioth project area. > * debian/control: > - Remove VCS urls and add new Homepage url. > - Remove IA64 architecture (this is already status quo). > - Remove double lines of Priority and Section. > - Add package conflicts to grub-pc and grub-legacy. > - Remove conflicts to manpages. > - Bump to Standards-Version 3.9.1 (without changes). > > > Kind regards > Joachim Wiedorn I had a look and: - --- lilo-22.8/debian/lilo.postinst +++ lilo-22.8-9/debian/lilo.postinst [...] + # to be compatible to older debian package + if [ ! -e /boot/sarge.bmp ]; then + ln -sf debian.bmp /boot/sarge.bmp fi [...] That "debian.bmp", should that have been a "/boot/debian.bmp"? In this snippet: - --- lilo-22.8/debian/rules +++ lilo-22.8-9/debian/rules @@ -23,6 +23,7 @@ dh_testdir make spotless -rm -f build debian/files debian/lilo.substvars [...] + -rm -f debian/*.debhelper.log debian/lilo-doc.substvars -rm -f doc/*.dvi doc/*.ps doc/*.aux doc/*.toc doc/*.log -rm -rf debian/lilo debian/lilo-doc debian/*.debhelper [...] [...] Most of these lines (incl. the one added) appear to be re-implementing dh_clean. Replacing those lines with dh_clean /might/ be better (because the release team knows what dh_clean does and do not need to verify the handmade version) There are also a lot of po file updates, which are not translation changes. I presume they have been made by some tool (e.g. debconf-updatepo from the clean target). As far as I can tell it has just added a Language header and in some cases modified the translation so that % is with its arguments (e.g. "%"\n"s" becomes \n"%s"). If these changes can sanely be avoided, then that might be a good idea as well. ~Niels -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iQIcBAEBCAAGBQJNBOTPAAoJEAVLu599gGRCQpMP/AggVPyEMLlcCe8SCJAGi02D kGpvLlNeKC7WilH3Vk7AJ0Xvylt/0F1kTCq5b/29QCXesZFtSv2GD8lxxjBx1VSN MJB1a+b4C9+ghFM8MSXGJ4l5K9QRRgeyV0CN6jEVHERa25kP1JSz85re2YbMKno8 nZ7pYPOwIZGWSavP47/BBX0/gOel4X1ftk7Y0THQDofkdTTYgHX0na22yp3Z2F7K MxmkZoG5Z/tkyHkuAMuWrayUe0MbrCbHDC08kt32pVgk5gjfzHJaCNAKKSWwHWWo dpBk+t4qpL+5YgsJ4t35a2KoFDCn+x4Ds14f1cfsYkvUJ+WclyCMz3QbcRCbPnJP UY9YFCw6DsdNlSFJBCmC+RI23jusEFJWItPEZX+a5DdItx/Ru+jC42PDnltqb88G f9WpRYNOgYnZFj9RN0fyhP131qEljb+fo6iD1IFGplXl7yqZHtHOTy3TeSNZ2aRW uWWLMV2BHslLrcl/f/GfjWTpLPgZtUVYiyKxNUbIFtn2Hajgl3GK0DST4eltx16i BGbwG8PnHgr8uxQVqTdn/S1n6qo6qar6pQaEwf+sRXaVoUh4joD11WkkAV5SPM+O 2NynmRCXgOdHFLGrK58IPP5yKzD1xLSaTQN7MrqNe94WHDZXBP4xXdEn+Od7R2FF X0KqejSxYXa7cwhTGNzg =BUD6 -----END PGP SIGNATURE----- -- 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/4d04e4d0.9010...@thykier.net