[Bug 478927] Review Request: globus-rsl -Globus Toolkit - Resource Specification Language Library

bugzilla at redhat.com bugzilla at redhat.com
Sat Apr 18 20:28:02 UTC 2009


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=478927





--- Comment #5 from Mattias Ellert <mattias.ellert at fysast.uu.se>  2009-04-18 16:28:01 EDT ---
(In reply to comment #4)
> Fedora review globus-rsl-5.0-1.fc12.i586.rpm 2008-04-18
> 
> (btw: the posted spec file and the spec file contained in the srpm are not
> consistent)

Sorry about that. It seems I copied the last version of the SRPM to the server
with the wrong name after the last update - so the previous one was not
overwritten as I intended. The difference between the versions was that I added
globus-common as a Requires so that the package Requires a package that owns
the direcories where it puts its files. I have updated the SRPM with the
intended one. The specfile was the intended one.

> + OK
> ! needs attention
> 
> ! rpmlint output (including the license file as doc gets rid of the no docs
> warning)

I just removed the license file as doc in this version since a reviewer of a
different package strongly argued that it shouldn't be there since it is not
part of the source tree for this package (see bug 467235 comment 3). I really
need all the packages to be the same, otherwise it is not maintainable. If you
strongly disagree with the previous reviewer, I will try to do some
arbitration.

> ! The package contains the license file

See above.

> ! Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'

globus-xio-devel Requires globus-core, which Requires pkgconfig.

-- 
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.




More information about the Fedora-package-review mailing list