Spec URL: http://katzj.fedorapeople.org/review/grubby/grubby.spec SRPM URL: http://katzj.fedorapeople.org/review/grubby/grubby-6.0.86-1.fc11.src.rpm Description: grubby is the command-line tool that manipulates bootloader config files Note: this is split out from the old mkinitrd package and thus has matching versioning. This might change. F11 has a separate binary package; with the push for dracut in F12 we will want a completely separate source package. This version also loses the nash dependence (yay!)
Oh yeah, when we get this in and built, then we'll do the surgery both on the mkinitrd repo and package to remove the (then obvious) conflict
Builds fine and rpmlint is clean. I thought I'd be able to fall back on the mkinitrd review, but it hasn't been completed yet and it looks like it has pretty much the same issues that this one does. So: Where does the tarball come from? I guess it's generated out of git somehow; if it's really not directly downloadable anywhere, some instructions on generating it would be good. See https://fedoraproject.org/wiki/Packaging:SourceURL There's a bit of weird formatting in the %description. I guess someone tried to fill it with a fixed right margin, so the word spacing is uneven. Not a blocker, but I thought I'd mention it. Really, the source thing is the only issue I see here. X can't compare source files match upstream. * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * summary is OK. * description is OK. * dist tag is present. * build root is OK (hopefully soon to go away anyway). * license field matches the actual license. * license is open source-compatible. * license text included in package. ? latest version is being packaged (can't t * BuildRequires are proper (shouldn't need pkgconfig, but it doesn't hurt). * compiler flags are appropriate. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly. * debuginfo package looks complete. * rpmlint is silent. * final provides and requires are sane: grubby = 6.0.86-1.fc11 grubby(x86-64) = 6.0.86-1.fc11 = /bin/bash /bin/sh libblkid.so.1()(64bit) libglib-2.0.so.0()(64bit) libpopt.so.0()(64bit) * %check is present and all tests pass: 120 (100%) tests passed, 0 (0%) tests failed * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * no generically named files * code, not content. * documentation is small, so no -doc subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * no headers. * no pkgconfig files. * no static libraries. * no libtool .la files.
(In reply to comment #2) > Where does the tarball come from? I guess it's generated out of git somehow; > if it's really not directly downloadable anywhere, some instructions on > generating it would be good. See > https://fedoraproject.org/wiki/Packaging:SourceURL Yeah, it's just a git snap. I'll add instructions > There's a bit of weird formatting in the %description. I guess someone tried > to fill it with a fixed right margin, so the word spacing is uneven. Not a > blocker, but I thought I'd mention it. Also fixed up New spec file at http://katzj.fedorapeople.org/review/grubby/grubby.spec
Thanks, looks good. The only thing I can say is that your checkout instructions are missing a "cd grubby" but it's pretty obvious. APPROVED
Gracias New Package CVS Request ======================= Package Name: grubby Short Description: Command line tool for updating boot loader configuration file Owners: katzj,pjones Branches: devel InitialCC:
CVS done.
This is in rawhide now and wasn't pushed to any release branches, so I see no reason for this ticket to still be open.