OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
OK - License (BSD)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
dad6200744d2c8f326fbec1f5e0c8249  Net_POP3-1.3.6.tgz
dad6200744d2c8f326fbec1f5e0c8249  Net_POP3-1.3.6.tgz.1
f2c485f9e2b306f5f75c7c71d373a26cad09667b  Net_POP3-1.3.6.tgz
f2c485f9e2b306f5f75c7c71d373a26cad09667b  Net_POP3-1.3.6.tgz.1
OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.

OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
See below - Package owns all the directories it creates.
See below - No rpmlint output.
OK - final provides and requires are sane:

SHOULD Items:

OK - Should build in mock.  
OK - Should build on all supported archs
OK - Should have dist tag
OK - Should package latest version

Issues:

1. I don't think that:
Requires(hint): php-pear(Net_Auth_SASL) >= 1.0
is usable with any rpm shipped by Fedora. I suppose it just makes it a hard 
Requires. Perhaps just change that to a Requires? Also, I can't seem to find 
that requirement in yet, should that package be added?

2. rpmlint says:
W: php-pear-Net-POP3 no-documentation
Which I guess can be ignored, since upstream has had no docs.

3. The
/usr/share/pear/Net
directory doesn't seem to be owned by this package,
it seems owned by php-pear-Net-FTP.
Should that package be required? or should this package also
own that directory?

(In reply to comment #1)
> Issues:
> 
> 1. I don't think that:
> Requires(hint): php-pear(Net_Auth_SASL) >= 1.0
> is usable with any rpm shipped by Fedora. I suppose it just makes it a hard 
> Requires. Perhaps just change that to a Requires? Also, I can't seem to find 
> that requirement in yet, should that package be added?

Even though (hint) does not do anything right now, I use this for optional
packages.  I added it because the Summary on the web page:
"...using optional Auth_SASL package"


> 3. The
> /usr/share/pear/Net
> directory doesn't seem to be owned by this package,
> it seems owned by php-pear-Net-FTP.
> Should that package be required? or should this package also
> own that directory?

This package Requires php-pear(Net_Socket) >= 1.0 which should own the Net
directory.

> 

(In reply to comment #2)
> (In reply to comment #1)
> > Issues:
> > 
> > 1. I don't think that:
> > Requires(hint): php-pear(Net_Auth_SASL) >= 1.0
> > is usable with any rpm shipped by Fedora. I suppose it just makes it a hard 
> > Requires. Perhaps just change that to a Requires? Also, I can't seem to find 
> > that requirement in yet, should that package be added?
> 
> Even though (hint) does not do anything right now, I use this for optional
> packages.  I added it because the Summary on the web page:
> "...using optional Auth_SASL package"

Although this should read "Auth_SASL" not "Net_Auth_SASL".  I will fix this. 
Nice catch.

Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-POP3.spec
SRPM URL:
http://tkmame.retrogames.com/fedora-extras/php-pear-Net-POP3-1.3.6-2.src.rpm

%changelog
* Sat Jan 13 2007 Christopher Stone <chris.stone> 1.3.6-2
- Fix Requires for Auth_SASL

ok, that appears to fix up all the blockers I see.

This package is APPROVED. 

Don't forget to close this review request NEXTRELEASE once it's been imported 
and built. 

- Imported into CVS
- Entry added to owners.list
- built successfully for devel
- cvs sync request added for FC5/6

THANKS FOR THE REVIEW!

Package Change Request
======================
Package Name: php-pear-Net-POP3
New Branches: EL-5