Bug 448205 (pear-Net-IPv4)

I can't sponsor you, I'm currently in the same boat as you currently but I did
review the package.  rpmlint showed no issues except for the lack of
documentation.  I don't see any issues with the packaging.

There is no documentation in the upstream code, so this should be normal.

New files:

Spec URL: http://www.alphatek.info/divers/php-pear-Net-IPv4.spec
SRPM URL: http://www.alphatek.info/divers/php-pear-Net-IPv4-1.3.0-2.fc9.src.rpm

As agreed with Chris, i take this review and will do steven spondoring

REVIEW :
* rpmlint :
W: no-documentation
* The package is named according to the Package Naming Guidelines.
* source files match upstream:
35ee77683b4fd34d7a9aa77c199b2eda  Net_IPv4-1.3.0.tgz
* The spec file name match the base package
* The package must the PHP Packaging Guidelines
* License field in the package spec file match the actual license 
* license is open source-compatible (PHP)
* latest version is being packaged.
* The spec file is written in American English
* The spec file is legible
* build in mock (F-9, noarch)
* All build dependencies are listed in BuildRequires
* no locales
* no shared library files
* package own all directories that it creates
* Package don't own files or directories already owned by other packages
* no duplicate files
* Permissions on files are set properly
* %clean section
* package consistently use macros
* The package contain code
* no documentation
* no library
* no desktop file
* %install, start with rm -rf %{buildroot}
* All filenames are valid UTF-8
* scriptlets are sane
* Final Provides ok
php-pear(Net_IPv4) = 1.3.0
php-pear-Net-IPv4 = 1.3.0-2.fc9
* Final Requires ok
/usr/bin/pear  
php-pear(PEAR)  
* install ok
* work ok, with litle test case

<?php
require ("Net/IPv4.php");
$ip = Net_IPv4::parseAddress("192.168.0.4/24");
if ($ip) print_r($ip);
else echo "failed\n";
?>


Should just change Remy to Remi in the changelog ;)

APPROVED

Please wait for you sponsoring before doing the CVS request.

New Package CVS Request
=======================
Package Name: pear-Net-IPv4
Short Description: Class used for calculating IPv4 address information
Owners: moixs
Branches: F-9
InitialCC: moixs, remi
Cvsextras Commits: yes

cvs done.

Sorry but I messed up, I used the alias instead of the real package name to
create the CVS access. Could you destroy pear-Net-IPv4 CVS and create the
following instead:

New Package CVS Request
=======================
Package Name: php-pear-Net-IPv4
Short Description: Class used for calculating IPv4 address information
Owners: moixs
Branches: F-9
InitialCC: moixs, remi
Cvsextras Commits: yes

cvs done

php-pear-Net-IPv4-1.3.0-2.fc9 has been submitted as an update for Fedora 9

php-pear-Net-IPv4-1.3.0-2.fc9 has been pushed to the Fedora 9 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 php-pear-Net-IPv4'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2008-6636

php-pear-Net-IPv4-1.3.0-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Taking ownership according to 
http://fedoraproject.org/wiki/Orphaned_package_that_need_new_maintainers#Claiming_Ownership_of_a_Deprecated_Package

Package Change Request
======================
Package Name: php-pear-Net-IPv4
Owners: remi

Unretired, take ownership.

Package Change Request
======================
Package Name: php-pear-Net-IPv4
New Branches: el6 epel7
Owners: remi

Git done (by process-git-requests).