Bug 236177 - (perl-NetAddr-IP) Review Request: perl-NetAddr-IP - Manages IPv4 and IPv6 addresses and subnets
Review Request: perl-NetAddr-IP - Manages IPv4 and IPv6 addresses and subnets
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Chris Weyl
Fedora Package Reviews List
:
: 247625 (view as bug list)
Depends On:
Blocks: 676263
  Show dependency treegraph
 
Reported: 2007-04-12 07:34 EDT by Andreas Thienemann
Modified: 2011-02-09 04:23 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-07-10 11:43:49 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
cweyl: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Andreas Thienemann 2007-04-12 07:34:21 EDT
Spec URL: http://helena.bawue.de/~ixs/perl-NetAddr-IP/perl-NetAddr-IP.spec
SRPM URL: http://helena.bawue.de/~ixs/perl-NetAddr-IP/perl-NetAddr-IP-4.004-1.src.rpm
Description:
This module provides an object-oriented abstraction on top of IP addresses
or IP subnets, that allows for easy manipulations. Version 4.xx of
NetAdder::IP will will work older versions of Perl and does not use
Math::BigInt as in previous versions.
Comment 1 Chris Weyl 2007-04-12 15:39:41 EDT
Missing br's on:

Test::More
Test::Pod
Pod::Coverage

There's an rfc included under docs/...  Why not add it to %doc?
Comment 2 Andreas Thienemann 2007-04-26 04:12:29 EDT
Kay, new package at http://helena.bawue.de/~ixs/perl-NetAddr-IP/
Comment 3 Chris Weyl 2007-04-26 11:41:41 EDT
Oddly enough, the module itself is (as near as I can determine) definitely
under the Artistic license, though it includes a copy of the GPL at
Lite/Util/GPL.  Weird. --along the same lines, Makefile.PL complains if
Test::Pod isn't installed, but no test actually uses it.

The non-versioned requires on perl(NetAddr::IP::Util) would normally need to
be filtered, but as this package also provides perl(NetAddr::IP::Util), it's
really not worth the effort :)

Missing BR on Test::More -- approval is contingent on this being added (which
can be done post-import).

+ source files match upstream:
4f0ef938abf0b1e43ca96fb0068381df  NetAddr-IP-4.004.tar.gz
4f0ef938abf0b1e43ca96fb0068381df  ../NetAddr-IP-4.004.tar.gz
+ package meets naming and versioning guidelines.
+ specfile is properly named, is cleanly written and uses macros consistently.
+ dist tag is present.
+ build root is correct.
+ license field matches the actual license.
+ license is open source-compatible.  (Artistic) License text not included upstream.
+ latest version is being packaged.
X BuildRequires are proper.
+ compiler flags are appropriate.
+ %clean is present.
+ package installs properly
+ debuginfo package looks complete.
+ rpmlint is silent.
+ final provides and requires are sane:
 ** perl-NetAddr-IP-4.004-0.2.fc6.x86_64.rpm
 == rpmlint
 == provides
 Util.so()(64bit)  
 perl(NetAddr::IP) = 4.4
 perl(NetAddr::IP::Lite) = 1.5
 perl(NetAddr::IP::Util) = 1.3
 perl(NetAddr::IP::UtilPP) = 1.2
 perl(NetAddr::IP::Util_IS) = 1.00
 perl-NetAddr-IP = 4.004-0.2.fc6
 == requires
 libc.so.6()(64bit)  
 libc.so.6(GLIBC_2.2.5)(64bit)  
 libc.so.6(GLIBC_2.3.4)(64bit)  
 libnsl.so.1()(64bit)  
 perl(:MODULE_COMPAT_5.8.8)  
 perl(AutoLoader)  
 perl(Carp)  
 perl(DynaLoader)  
 perl(Exporter)  
 perl(NetAddr::IP::Lite) >= 1.02
O perl(NetAddr::IP::Util)  
O perl(NetAddr::IP::Util) >= 0.17
 perl(NetAddr::IP::Util_IS)  
 perl(strict)  
 perl(vars)  
 ** perl-NetAddr-IP-debuginfo-4.004-0.2.fc6.x86_64.rpm
 == rpmlint
 == provides
 Util.so.debug()(64bit)  
 perl-NetAddr-IP-debuginfo = 4.004-0.2.fc6
 == requires
+ %check is present and all tests pass
+ no shared libraries are added to the regular linker search paths.
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets present.
+ code, not content.
+ documentation is small, so no -docs subpackage is necessary.
+ %docs are not necessary for the proper functioning of the package.
+ no headers.
+ no pkgconfig files.
+ no libtool .la droppings.
+ not a GUI app.

APPROVED
Comment 4 Chris Weyl 2007-05-11 11:34:02 EDT
ping?
Comment 5 Andreas Thienemann 2007-05-16 05:25:13 EDT
Sorry, must have missed this being approved.

New Package CVS Request
=======================
Package Name: perl-NetAddr-IP
Short Description: Manages IPv4 and IPv6 addresses and subnets
Owners: andreas@bawue.net
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

Comment 6 Steven Pritchard 2007-07-10 11:08:54 EDT
*** Bug 247625 has been marked as a duplicate of this bug. ***
Comment 7 Jason Tibbitts 2007-07-10 11:29:31 EDT
Is there some reason why this hasn't been built yet?
Comment 8 Andreas Thienemann 2007-07-10 11:43:49 EDT
fixed
Comment 9 Andreas Thienemann 2007-07-10 12:10:42 EDT
Package Change Request
======================
Package Name: perl-NetAddr-IP
New Branches: F-7
Comment 10 Kevin Fenzi 2007-07-10 19:05:51 EDT
cvs done.

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