Spec URL: http://hpejakle.fedorapeople.org/packages/perl-NetPacket.spec SRPM URL: http://hpejakle.fedorapeople.org/packages/perl-NetPacket-0.41.1-1.fc11.src.rpm Description: NetPacket provides a base class for a cluster of modules related to decoding and encoding of network protocols. Each NetPacket descendent module knows how to encode and decode packets for the network protocol it implements. Consult the documentation for the module in question for protocol- specific implementation. Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2054702 Rpmlint output: 2 packages and 1 specfiles checked; 0 errors, 0 warnings.
So, aside from the spelling error in %description, everything looks good. ....except for the license. I see references to the "Perl Artistic License", and a (defunct) url is given to document it. Given the age of the software, I have to wonder if it's the Artistic v1, rather than v2. I do see a github repo listed in META.yml; is there any chance we can query upstream for clarification? koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2054702 Spec looks sane, clean and consistent; license is correct (Artistic 2.0); make test passes cleanly. Source tarballs match upstream (sha1sum): ac2ebb4084beab7f73eeb4e1a994d24636de2575 NetPacket-0.41.1.tar.gz ac2ebb4084beab7f73eeb4e1a994d24636de2575 NetPacket-0.41.1.tar.gz.srpm Final provides / requires are sane: 0 packages and 1 specfiles checked; 0 errors, 0 warnings. =====> perl-NetPacket-0.41.1-1.fc14.noarch.rpm <===== ====> rpmlint perl-NetPacket.noarch: W: spelling-error %description -l en_US descendent -> descendant, descender, descend 1 packages and 0 specfiles checked; 0 errors, 1 warnings. ====> provides for perl-NetPacket-0.41.1-1.fc14.noarch.rpm perl(NetPacket) = 0.41.1 perl-NetPacket = 0.41.1-1.fc14 perl(NetPacket::ARP) = 0.41.1 perl(NetPacket::Ethernet) = 0.41.1 perl(NetPacket::ICMP) = 0.41.1 perl(NetPacket::IGMP) = 0.41.1 perl(NetPacket::IP) = 0.41.1 perl(NetPacket::TCP) = 0.41.1 perl(NetPacket::UDP) = 0.41.1 ====> requires for perl-NetPacket-0.41.1-1.fc14.noarch.rpm perl(constant) perl(:MODULE_COMPAT_5.10.1) perl(NetPacket) perl(NetPacket::IP) perl(strict) perl(vars) =====> perl-NetPacket-0.41.1-1.fc14.src.rpm <===== ====> rpmlint perl-NetPacket.src: W: spelling-error %description -l en_US descendent -> descendant, descender, descend 1 packages and 0 specfiles checked; 0 errors, 1 warnings. ====> provides for perl-NetPacket-0.41.1-1.fc14.src.rpm ====> requires for perl-NetPacket-0.41.1-1.fc14.src.rpm perl(Module::Build) perl(Test::More)
I have asked upstream for clarification, hope they will respond soon. hmm, my rpmlint is quiet. How did you use it that it checks for spelling errors?
upstream released new version with confirmation that License is GPL+ or Artistic 2.0 (same as perl) spelling error fixed. Spec URL: http://hpejakle.fedorapeople.org/packages/perl-NetPacket.spec SRPM URL: http://hpejakle.fedorapeople.org/packages/perl-NetPacket-0.42.0-1.fc11.src.rpm
Hi Chris, ping?
Jan, Could you update NetPacket to the latest upstream version? Regards, jpo
Created attachment 446770 [details] Patch for EPEL5 Downgrades the Module::Build version requirement so that it the module can be packaged for EPEL5. Error messages without the patch: ---------- ... + /usr/bin/perl Build.PL installdirs=vendor Module::Build version 0.360100 required--this is only version 0.280700 at Build.PL line 5 BEGIN failed--compilation aborted at Build.PL line 5, <DATA> line 537. error: Bad exit status from /var/tmp/rpm-tmp.54415 (%build) ... ----------
The latest upstream version is 1.0.1.
Hello Jan, could you provide current version 1.0.1? I'll retake this review.
Marcela, I don't mind submitting perl-NetPacket 1.0.1 for review if Jan doesn't give any news in the next few days. jpo
Ok, I'll be waiting for ping from one of you.
Marcela, Here are the SRPM and specfile of NetPacket 1.0.1: SRPM : http://um-pe09-2.di.uminho.pt./fedora/perl-NetPacket-1.0.1-1.el5.src.rpm Specfile: http://um-pe09-2.di.uminho.pt./fedora/perl-NetPacket.spec Major changes: * updated to version 1.0.1 * disables one of the tests and applies a patch to downgrade the Module::Build version requirement so that it can be successfully built in EPEL >= 5 and Fedora >= 12. /jpo
- rpmlint OK - package must be named according to Guidelines OK - spec file name must match the base package %{name} OK - package must meet the Packaging Guidelines OK - package must be licensed with Fedora approved license OK - license field must match actual license OK - text of the license in its own file must be included in %doc OK - sources must match the upstream source OK - package MUST successfully compile and build OK http://koji.fedoraproject.org/koji/taskinfo?taskID=2711790 - architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK - build dependencies must be listed in BuildRequires OK - handle locales properly with %find_lang macro OK - shared library files must call ldconfig in %post(un) OK - packages must NOT bundle system libraries OK - package must own all directories that it creates OK - permissions on files must be set properly OK - package must consistently use macros OK - package must contain code, or permissable content OK - large documentation must go in a -doc OK - %doc must not affect the runtime of the application OK - header files must be in a -devel package OK - static libraries must be in a -static package OK - library files that end in .so (without suffix) must go in a -devel OK - devel package usually require base package OK - packages must NOT contain any .la libtool archives OK - GUI applications must include a %{name}.desktop file OK - packages must not own files or directories already owned by other packages OK rpm -q --provides perl-NetPacket perl(NetPacket) = 1.0.1 perl(NetPacket::ARP) = 1.0.1 perl(NetPacket::Ethernet) = 1.0.1 perl(NetPacket::ICMP) = 1.0.1 perl(NetPacket::IGMP) = 1.0.1 perl(NetPacket::IP) = 1.0.1 perl(NetPacket::TCP) = 1.0.1 perl(NetPacket::UDP) = 1.0.1 perl-NetPacket = 1.0.1-1.fc15 rpm -q --requires perl-NetPacket perl(:MODULE_COMPAT_5.12.2) perl(NetPacket) perl(NetPacket::IP) perl(constant) perl(strict) perl(vars) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(VersionedDependencies) <= 3.0.3-1 rpmlib(PayloadIsXz) <= 5.2-1 Patch for EL-5 build looks fine. ACCEPT
New Package SCM Request ======================= Package Name: perl-NetPacket Short Description: Assemble/disassemble network packets at the protocol level Owners: jpo Branches: f13 f14 el5 el6 InitialCC: perl-sig
Git done (by process-git-requests).
perl-NetPacket-1.0.1-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/perl-NetPacket-1.0.1-1.el5
perl-NetPacket-1.0.1-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/perl-NetPacket-1.0.1-1.fc13
perl-NetPacket-1.0.1-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/perl-NetPacket-1.0.1-1.fc14
perl-NetPacket-1.0.1-1.el5 has been pushed to the Fedora EPEL 5 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-NetPacket'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/perl-NetPacket-1.0.1-1.el5
perl-NetPacket-1.0.1-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
perl-NetPacket-1.0.1-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
perl-NetPacket-1.0.1-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.