[Bug 457839] Review Request: rubygem-cobbler - Provides Ruby bindings to interact with a Cobbler server.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457839


Jason Tibbitts <tibbs@xxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|bkearney@xxxxxxxxxx         |tibbs@xxxxxxxxxxx
               Flag|fedora-review+              |fedora-review?




--- Comment #9 from Jason Tibbitts <tibbs@xxxxxxxxxxx>  2008-08-26 12:16:56 EDT ---
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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]