Bug 573848 (perl-NetPacket)

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.