Red Hat Bugzilla – Bug 242545
Review Request: ftplib - Library of FTP routines
Last modified: 2007-11-30 17:12:06 EST
Spec URL: http://www.auroralinux.org/people/spot/review/ftplib.spec
SRPM URL: http://www.auroralinux.org/people/spot/review/ftplib-3.1-1.fc7.src.rpm
ftplib is a set of routines that implement the FTP protocol. They allow
applications to create and access remote files through function calls
instead of needing to fork and exec an interactive ftp client program.
It also has a subpackage, qftp, which is a simple CLI application that uses ftplib.
Note: This package is a BuildRequires for xbiso (242544)
Created attachment 156114 [details]
spec file patch
OK source files match upstream:
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
BAD license field doesn't matches the actual license.
the library is LGPL, the client is GPL
OK license is open source-compatible. License text not included upstream.
OK latest version is being packaged.
OK BuildRequires are proper.
OK compiler flags are appropriate.
OK %clean is present.
OK debuginfo package looks complete.
OK rpmlint is silent.
OK final provides and requires look sane
OK shared libraries are added to the regular linker search paths, ldconfig is run
OK doesn't create new directories
OK doesn't own any directories it shouldn't.
OK no duplicates in %files.
OK file permissions are appropriate.
OK correct scriptlets are present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK headers and devel libs are in devel subpackage
OK no pkgconfig files.
OK no libtool .la
OK not a GUI app.
?? package builds in mock (x86_64) - still downloading packages ;-)
Everything looks good, only the licences should be corrected - see the attached
New SPEC: http://www.auroralinux.org/people/spot/review/ftplib.spec
New SRPM: http://www.auroralinux.org/people/spot/review/ftplib-3.1-2.fc7.src.rpm
OK package builds in mock (i386) and FC-6 (x86_64)
So the package is APPROVED
Did my own CVS. For the record:
New Package CVS Request
Package Name: ftplib
Short Description: Library of FTP routines
Branches: FC-5 FC-6 F-7
Built, pushing forward, thanks for the review!