Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 516516 - Review Request: globus-gridftp-server-control - Globus Toolkit - Globus GridFTP Server Library
Summary: Review Request: globus-gridftp-server-control - Globus Toolkit - Globus GridF...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Steve Traylen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 516517
TreeView+ depends on / blocked
 
Reported: 2009-08-10 11:01 UTC by Mattias Ellert
Modified: 2009-11-03 02:00 UTC (History)
3 users (show)

Fixed In Version: 0.36-1.el5
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-10-21 00:38:24 UTC
Type: ---
Embargoed:
steve.traylen: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Mattias Ellert 2009-08-10 11:01:32 UTC
Spec URL: http://www.grid.tsl.uu.se/review/globus-gridftp-server-control.spec
SRPM URL: http://www.grid.tsl.uu.se/review/globus-gridftp-server-control-0.36-1.fc11.src.rpm

Description:
The Globus Toolkit is an open source software toolkit used for building Grid
systems and applications. It is being developed by the Globus Alliance and
many others all over the world. A growing number of projects and companies are
using the Globus Toolkit to unlock the potential of grids for their cause.

The globus-gridftp-server-control package contains:
Globus GridFTP Server Library

Globus Toolkit Packaging Guidelines:
https://fedoraproject.org/wiki/Packaging:Globus

Comment 1 Steve Traylen 2009-10-14 21:06:07 UTC
Review of globus-gridftp-server-control:

koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1746671


MUST:

no: 
$ rpmlint globus-gridftp-server-control.spec ../SRPMS/globus-gridftp-server-control-0.36-1.fc11.src.rpm ../RPMS/x86_64/globus-gridftp-server-control-*
globus-gridftp-server-control-debuginfo.x86_64: E: description-line-too-long This package provides debug information for package globus-gridftp-server-control.
globus-gridftp-server-control-devel.x86_64: W: no-documentation
4 packages and 1 specfiles checked; 1 errors, 1 warnings.

which is fine 
yes: Package name - globus guidelines
yes: Spec File name - globus guidelines
yes: Packaging guidelines
yes: Apache 2.0
yes: .spec ASL 2.0
yes: GLOBUS_LICENSE included.
yes: american english.
yes: Upstream source matches.
$ diff -r --brief gt4.2.1-all-source-installer/source-trees/gridftp/server-lib/src ~/rpmbuild/SOURCES/globus_gridftp_server_control-0.36
Only in /home/steve/rpmbuild/SOURCES/globus_gridftp_server_control-0.36: GLOBUS_LICENSE


which is expected.

yes: builds as per koji build above.
yes: locales not present.
yes: Calls ldconfig appropriate
yes: No system libs included.
yes: not relocatable.
yes: All files and directories created in the package are owned.
yes: files not listed twice.
yes: defattr used everywhere.
yes: Buildroot is removed in %clean.
yes: consistant macros
yes: no large docs.
yes: docs not needed runtime.
yes: headers in -devel file.
yes: libs correctly in -devel package.
yes: main package is needed by devel package.
yes: no .la files present.
yes: no gui.
yes: does not own files belonging to other packages.
yes: buildroot removed in %install.
yes: utf8

APPROVED

Comment 2 Steve Traylen 2009-10-14 21:17:26 UTC
Sorry not approved, auto pilot.

The summary line is a little long which I realise is the fact that
the package is name is so long.

Steve

Comment 3 Mattias Ellert 2009-10-15 17:55:32 UTC
(In reply to comment #2)
> Sorry not approved, auto pilot.
> 
> The summary line is a little long which I realise is the fact that
> the package is name is so long.
> 
> Steve  

The only package for which rpmlint complains about the length of the description line is the debuginfo package. The definition of the description of the debuginfo package is not part if the specfile, but is defined in the %debug_info macro. Any package that has a name longer that 27 characters will trip on this.

[ellert@ellert ~]$ rpm -E %debug_package

%ifnarch noarch
%package debuginfo 
Summary: Debug information for package %{name}
Group: Development/Debug
AutoReqProv: 0
%description debuginfo
This package provides debug information for package %{name}.
Debug information is useful when developing applications that use this
package or when debugging this package.
%files debuginfo -f debugfiles.list
%defattr(-,root,root)
%endif

I can't do much about this except redefining the macro, which I find to be an overkill.

When I first saw this rpmlint error I checked what other packages had done. There are many packages in Fedora with names longer than 27 characters. The current practice seems to be to ignore this error for debuginfo packages. (See bug 525304, bug 523326, bug 516516, bug 498189, bug 496429 and others.)

Comment 4 Mattias Ellert 2009-10-15 18:08:40 UTC
Sorry, I did not intend to reference back to this bug (516516) - that was a C&P error.

Comment 5 Steve Traylen 2009-10-19 11:44:50 UTC
Could we at least submit a bug on the macro. It could be changed more
generally I say to perhaps always word wrap at the desired length.

Steve

APPROVED

Comment 6 Mattias Ellert 2009-10-19 13:14:04 UTC
(In reply to comment #5)
> Could we at least submit a bug on the macro. It could be changed more
> generally I say to perhaps always word wrap at the desired length.

Such a bug report already exists: see bug 484616.
(It was already submitted by someone else.)(In reply to comment #5)

Comment 7 Mattias Ellert 2009-10-19 13:15:25 UTC
New Package CVS Request
=======================
Package Name: globus-gridftp-server-control
Short Description: Globus Toolkit - Globus GridFTP Server Library
Owners: ellert
Branches: F-11 F-12 EL-4 EL-5
InitialCC:

Comment 8 Kevin Fenzi 2009-10-19 16:24:52 UTC
cvs done.

Comment 9 Fedora Update System 2009-10-19 19:43:39 UTC
globus-gridftp-server-control-0.36-1.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/globus-gridftp-server-control-0.36-1.fc12

Comment 10 Fedora Update System 2009-10-19 19:43:44 UTC
globus-gridftp-server-control-0.36-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/globus-gridftp-server-control-0.36-1.fc11

Comment 11 Fedora Update System 2009-10-19 19:43:45 UTC
globus-gridftp-server-control-0.36-1.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/globus-gridftp-server-control-0.36-1.el5

Comment 12 Fedora Update System 2009-10-19 19:43:50 UTC
globus-gridftp-server-control-0.36-1.el4 has been submitted as an update for Fedora EPEL 4.
http://admin.fedoraproject.org/updates/globus-gridftp-server-control-0.36-1.el4

Comment 13 Fedora Update System 2009-10-21 00:38:19 UTC
globus-gridftp-server-control-0.36-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2009-11-03 01:58:42 UTC
globus-gridftp-server-control-0.36-1.el4 has been pushed to the Fedora EPEL 4 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2009-11-03 01:59:59 UTC
globus-gridftp-server-control-0.36-1.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.