Bug 239906

I suggest changing %description to:
  OS independent wrapper class for executing ping calls.
to correct a misspelling and add a period.

I think this package should have a depdendency on iputils (for /bin/ping). 
Otherwise I don't see how it could perform its function.

rpmlint complains:
  W: php-pear-Net-Ping incoherent-version-in-changelog 1.4.1-1 2.4.1-1.fc7
The package version is 2.4.1 but the (only) changelog entry refers to 1.4.1. 
Probably just a typo.

There is a basic test included which can easily be run at build time with:
   %check
   cd tests
   php test_result_parsers.php

* source files match upstream:
   17a169c66b5a9f3cd71126d16efcd92c75f9116243a6fcf9732d6f2d6ca0811d  
   Net_Ping-2.4.1.tgz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
X rpmlint has a valid complaint.
? final provides and requires:
   php-pear(Net_Ping) = 2.4.1
   php-pear-Net-Ping = 2.4.1-1.fc7
  =
   /bin/sh
   /usr/bin/pear

X %check is not present, but there's a runnable test suite.
* 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.

Thanks for the review.

Spec URL: http://remi.collet.free.fr/rpms/extras/php-pear-Net-Ping.spec
SRPM URL:
http://remi.collet.free.fr/rpms/extras/php-pear-Net-Ping-2.4.1-2.fc8.src.rpm

%changelog
* Wed May 16 2007 Remi Collet <Fedora> 2.4.1-2
- From review, correct typo.
- add Requires: /bin/ping
- add simple test in %%check 

I requires /bin/ping rather than iputils after asking #fedora-devel. I don't
really prefer one solution to another. It will work even if ping gets moved to
another package.

When packaging pear stuff, most often, tests must be run after installing the
package (doesn't run in Buildroot, even with R changed to BR).

Running tests after installation allow to be more complete (need configuration
change, databse...), ex running
/usr/share/doc/php-pear-Net-Ping-2.4.1/examples/example.php.

But, of course, i agree to include test when exists and works.

Well, file dependencies in /bin are fine according to the guidelines.  (There
are several directories for which they are OK, but outside of those yum has to
fetch additional dependency information which makes things slow.)

I know that most PEAR modules don't have tests which are runnable at build time,
but I happened to notice that this one could be run which is why I pointed it out.

Everything looks good to me now.

APPROVED

New Package CVS Request
=======================
Package Name: php-pear-Net-Ping
Short Description: Execute ping
Owners: Fedora
Branches: devel FC-5 FC-6 EL-5 (FC-7 ???)
InitialCC: 

Package Change Request
======================
Package Name: php-pear-Net-Ping
New Branches: F-7

CVS done.