Spec URL: http://mcpierce.fedorapeople.org/rubygem-cobbler.spec SRPM URL: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.1-1.fc9.src.rpm Description: Provides Ruby bindings to interact with a Cobbler server.
* General OK - Package name But why a gem? XX - License info is accurate Specify LGPL based on COPYING File OK - License tag is correct and licenses are approved OK - License files are installed as %doc OK - Specfile name XX - Specfile is legible Explicitly list the test files, and their permissions. No Loops. OK - No prebuilt binaries included OK - BuildRoot value (one of the recommended values) OK - PreReq not used OK - Source md5sum matches upstream OK - No hardcoded pathnames OK - Package owns all the files it installs OK - 'Requires' create needed unowned directories XX - Package builds successfully on i386 and x86_64 (mock) OUTPUT of "rpmbuild --rebuild rubygem-cobbler-0.0.1-1.fc9.src.rpm " Installing rubygem-cobbler-0.0.1-1.fc9.src.rpm warning: user mcpierce does not exist - using root warning: group mcpierce does not exist - using root warning: user mcpierce does not exist - using root warning: group mcpierce does not exist - using root Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.30412 + umask 022 + cd /home/bkearney/rpmbuild/BUILD /var/tmp/rpm-tmp.30412: line 23: cd: /home/bkearney/rpmbuild/BUILD: No such file or directory error: Bad exit status from /var/tmp/rpm-tmp.30412 (%prep) OUTPUT of "rpmbuild -bb rubygem-cobbler.spec" Processing files: rubygem-cobbler-0.0.1-1.fc9 warning: File listed twice: /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/COPYING warning: File listed twice: /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/NEWS warning: File listed twice: /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/README warning: File listed twice: /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/config/cobbler.yml Provides: config(rubygem-cobbler) = 0.0.1-1.fc9 rubygem(cobbler) = 0.0.1 Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(VersionedDependencies) <= 3.0.3-1 Requires: /usr/bin/ruby config(rubygem-cobbler) = 0.0.1-1.fc9 rubygems Checking for unpackaged file(s): /usr/lib/rpm/check-files /var/tmp/rubygem-cobbler-0.0.1-1.fc9-root-bkearney Wrote: /home/bkearney/rpmbuild/RPMS/noarch/rubygem-cobbler-0.0.1-1.fc9.noarch.rpm Executing(%clean): /bin/sh -e /var/tmp/rpm-tmp.53996 OK - BuildRequires sufficient OK - File permissions set properly OK - Macro usage is consistent XX - rpmlint is silent rpmlint rubygem-cobbler-0.0.1-1.fc9.src.rpm rubygem-cobbler.src: W: mixed-use-of-spaces-and-tabs (spaces: line 34, tab: line 8) 1 packages and 0 specfiles checked; 0 errors, 1 warnings. rpmlint rubygem-cobbler-0.0.1-1.fc9.noarch.rpm rubygem-cobbler.noarch: E: file-in-usr-marked-as-conffile /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/config/cobbler.yml rubygem-cobbler.noarch: W: conffile-without-noreplace-flag /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/config/cobbler.yml 1 packages and 0 specfiles checked; 1 errors, 1 warnings. * Package a rubygem OK - Package is named rubygem-%{gemname} OK - Source points to full URL of gem OK - Package version identical with gem version OK - Package Requires and BuildRequires rubygems OK - Package provides rubygem(%{gemname}) = %version OK - Package requires gem dependencies correctly OK - %prep and %build are empty XX - %gemdir defined properly, and gem installed into it [bkearney@localhost noarch]$ sudo rpm --install rubygem-cobbler-0.0.1-1.fc9.noarch.rpm package rubygem-cobbler-0.0.1-1.fc9.noarch is already installed [bkearney@localhost noarch]$ irb irb(main):001:0> require "cobbler" LoadError: no such file to load -- cobbler from (irb):1:in `require' from (irb):1 OK - Package owns its directories under %gemdir ** noarch rubygem OK - No arch-specific content in %{gemdir} OK - Package is noarch
To clarify, anything with an XX should be resolved. Also, the output of OUTPUT of "rpmbuild --rebuild rubygem-cobbler-0.0.1-1.fc9.src.rpm " Does build, but gives me the following warnings: [bkearney@localhost pierce]$ rpmbuild --rebuild rubygem-cobbler-0.0.1-1.fc9.src.rpm Installing rubygem-cobbler-0.0.1-1.fc9.src.rpm warning: user mcpierce does not exist - using root warning: group mcpierce does not exist - using root warning: user mcpierce does not exist - using root warning: group mcpierce does not exist - using root Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.20548 + umask 022 If you resolve and reload the rpms I will re-verify.
UPDATED: Specfile: http://mcpierce.fedorapeople.org/rubygem-cobbler.spec SRPM: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.1-3.fc9.src.rpm RPM: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.1-3.fc9.noarch.rpm (In reply to comment #1) > * General > > OK - Package name > But why a gem? It makes it easier for developers to consume it as a dependency, along the lines of Maven. > XX - License info is accurate > > Specify LGPL based on COPYING File Fixed. It is now LGPLv2+, same as Cobbler. > XX - Specfile is legible > > Explicitly list the test files, and their permissions. No Loops. Done. > XX - Package builds successfully on i386 and x86_64 (mock) > > OUTPUT of "rpmbuild --rebuild rubygem-cobbler-0.0.1-1.fc9.src.rpm " > Installing rubygem-cobbler-0.0.1-1.fc9.src.rpm > warning: user mcpierce does not exist - using root > warning: group mcpierce does not exist - using root > warning: user mcpierce does not exist - using root > warning: group mcpierce does not exist - using root > Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.30412 > + umask 022 > + cd /home/bkearney/rpmbuild/BUILD > /var/tmp/rpm-tmp.30412: line 23: cd: /home/bkearney/rpmbuild/BUILD: No such > file or directory > error: Bad exit status from /var/tmp/rpm-tmp.30412 (%prep) This error's not due to the RPM. > XX - rpmlint is silent > > rpmlint rubygem-cobbler-0.0.1-1.fc9.src.rpm > rubygem-cobbler.src: W: mixed-use-of-spaces-and-tabs (spaces: line 34, tab: > line 8) > 1 packages and 0 specfiles checked; 0 errors, 1 warnings. > > rpmlint rubygem-cobbler-0.0.1-1.fc9.noarch.rpm > rubygem-cobbler.noarch: E: file-in-usr-marked-as-conffile > /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/config/cobbler.yml > rubygem-cobbler.noarch: W: conffile-without-noreplace-flag > /usr/lib/ruby/gems/1.8/gems/cobbler-0.0.1/config/cobbler.yml > 1 packages and 0 specfiles checked; 1 errors, 1 warnings. Fixed. > XX - %gemdir defined properly, and gem installed into it > > [bkearney@localhost noarch]$ sudo rpm --install > rubygem-cobbler-0.0.1-1.fc9.noarch.rpm > package rubygem-cobbler-0.0.1-1.fc9.noarch is already installed > [bkearney@localhost noarch]$ irb > irb(main):001:0> require "cobbler" > LoadError: no such file to load -- cobbler > from (irb):1:in `require' > from (irb):1 Fixed. In the GemSpec, require_path was misconfigured as "." rather than "lib".
I verified. all changes look good. APPROVED Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import the package. Close this bug as RAWHIDE once it's been successfully imported and built.
New Package CVS Request ======================= Package Name: rubygem-cobbler Short Description: Provides Ruby bindings to interact with a Cobbler server. Owners: mcpierce Branches: F-9 InitialCC: mcpierce
Hey Bryan. I don't seem to see you in the packager group... have you been sponsored yet there? You do need to be in that group to approve packages...
(In reply to comment #6) > Hey Bryan. I don't seem to see you in the packager group... > have you been sponsored yet there? You do need to be in that group to approve > packages... Ok.. misread the guidelines. Since I did an initial review, but not in the packager group what does darryl need to do next?
I was hoping to knock out some of the other ruby package reviews today; one more shouldn't take long, so I'll try to look at this in a bit.
I'm going to assign this to myself; I don't mean to steal the ticket, but I fear losing track of it since I see some things that need fixing. OK. First problem: The upstream source does not match what is in this package. I downloaded the src.rpm from comment #3, unpacked it, and used spectool -g to download the Source URLs and compared them. This either points to a naughty upstream which modified the tarball without changing the version (which should never be done) or the tarball was modified before the src.rpm was built (which should only be done in rare circumstances where legalities require it). Wow, unpacking the data.tar.gz file inside the gem gives lots of complaints from tar: tar: examples/has_distro.rb: implausibly old time stamp 1969-12-31 18:00:00 What on earth is going on there? The license of cobbler.rb and all files in lib/cobbler is GPLv2 only. The specfile says LGPLv2+. I see this was discussed earlier but I don't understand why the sources in the tarball contradict the COPYING file in the tarball. In any case, the COPYING file takes second seat here, but I still wouldn't approve this package until the upstream licensing issue is clarified. There seems to be a test suite included; is there any reason for not running it? X source files do not 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. X license field matches the actual license. ? latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly. * rpmlint is silent. * final provides and requires are sane: rubygem(cobbler) = 0.0.1 rubygem-cobbler = 0.0.1-3.fc10 = /usr/bin/ruby rubygems ? %check is not present, but a test suite seems to exist. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * no scriptlets present. * code, not content. * documentation is small, so no -doc subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * rubygem(%{gemname}) provided. * gems installed properly. * gems installed to proper directory. * gems, cache, and specifications subdirectories owned properly.
I've fixed the licensing to state that it is released under LGPLv2.1, and added a call to the tests in the %check section. Specfile: http://mcpierce.fedorapeople.org/rubygem-cobbler.spec SRPM: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.2-2.fc9.src.rpm Upstream: http://mcpierce.fedorapeople.org/cobbler-0.0.2.gem
Sorry for taking so long; I was confused by the CVS flag being set and thought the review was done. The package in comment 10 doesn't build for me: + rake test /var/tmp/rpm-tmp.1JWf6s: line 29: rake: command not found I urge doing a quick mock or koji scratch build when updating packages so that issues like this become obvious. On the upside, the licensing issue is OK now, and the included gem file now matches upstream. Unpacking it still gives "implausibly old time stamp 1969-12-31" which you didn't seem to address in your comments at all. I find it odd that the "source" directory in the unpacked gem contains the gem again. Is that somehow normal for gems?
I've added a build requirement for rubygem-rake which will provide the rake executable needed. As for the dates, not sure what was going on there. I took the gem that was up on fedorapeople.org and unpacked it and saw the same thing you did. I created a whole new gem and unpacked that and things look fine. So I've cleaned out my fedorapeople.org directory and put up only the latest upstream, spec and SRPM. Please download: SPEC: http://mcpierce.fedorapeople.org/rubygem-cobbler.spec SRPM: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.2-3.fc9.src.rpm SRC: http://fedorapeople.org/~mcpierce/cobbler-0.0.2.gem
Something has gone odd with this package; it seems to require itself in order to build, and since its not in the distro currently I can't build it at all. I'm guessing you meant to add a build dependency on rubygem-rake but instead typoed rubygem-cobbler. Unfortunately, the tests then fail with no such file to load -- flexmock/test_unit so, knowing very little about ruby, I did a yum search and guessed that adding a ruby-flexmock build dependency would work, and amazingly it did. Please, I beg of you: koji build --scratch rubygem-cobbler-0.0.2-3.fc9.src.rpm Takes only a couple of minutes, saves me way more than a couple of minutes. I'm still seeing the weird timestamp issue unpacking the gem but honestly its more of a confusing annoyance than a blocker. At this point I would approve a package fixed as detailed above to build properly.
Not sure what's up with the weird timestamp. I built the gem locally and then unpacked it and am getting none of what you're seeing. Is it possible the RPM process is repackaging the gem, like it does with JARs? I took your advice, ran koji local and all built properly. Spec: http://mcpierce.fedorapeople.org/rubygem-cobbler.spec SRPM: http://mcpierce.fedorapeople.org/rubygem-cobbler-0.0.2-4.fc9.src.rpm GEM: http://mcpierce.fedorapeople.org/cobbler-0.0.2.gem
rpm doesn't do any kind of repacking of game. I asked folks on IRC to download the GEM URL above and they also saw the bizarre timestamping issue. The package, however, is OK now. APPROVED
New Package CVS Request ======================= Package Name: rubygem-cobbler Short Description: Provides Ruby bindings to interact with a Cobbler server. Owners: mcpierce Branches: F-9 F-10 InitialCC: mcpierce
cvs done.
Package Change Request ====================== Package Name: rubygem-cobbler New Branches: F-8 EL-5
Any reason this ticket is stil open, given that the package is already in the relevant stable Fedora branches?
None that I'm aware. It can be closed AFAIAC.
I don't understand why you didn't just close it, but OK, I'll do it.