Bug 436737

I suppose to conform to Fedora standards I should have named it MimeDecode
instead of mimeDecode... I'll do that next release.

First comments :

Package name if ok : php-pear-Mail-mimeDecode

You should use
  %setup -q -c
  cd Mail_mimeDecode-%{version}

Because package.xml is outside any directory in the tarball

As you Require pear > 1.6.0, you can probably BuildRequires same version (not
usefull to build if you can't install it).

From rpmlint :
 W: invalid-license PHP License

You must fix version => PHP

You must not own /usr/share/pear/Mail which is already owned by
php-pear-Mail-Mime (which is required).

It could be valuable to add a comment (for packager) on how to run test after
installation :  pear run-tests -p Mail_mimeDecode

Spec URL: http://theholbrooks.org/RPMS/php-pear-Mail-mimeDecode.spec
SRPM URL: http://theholbrooks.org/RPMS/php-pear-Mail-mimeDecode-1.5.0-2.fc8.src.rpm

Applied all your suggested changes.  Thanks Remi!

REVIEW :
* rpmlint :
W: no-documentation
* The package is named according to the Package Naming Guidelines.
* The spec file name match the base package
* The package must the PHP Packaging Guidelines
* License field in the package spec file DOES NOT match the actual license
* The spec file is written in American English
* The spec file is legible
* build in mock (F-8, 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(Mail_mimeDecode) = 1.5.0
php-pear-Mail-mimeDecode = 1.5.0-2.fc8

* Final Requires ok
/usr/bin/pear  
php-pear(Mail_Mime) > 1.4.0
php-pear(PEAR) >= 1.6.0


MUST use epoch for pear BR (to avoid use of php-pear-4.3.9, p.e.) :
BuildRequires:	php-pear >= 1:1.6.0

(note that php-pear(PEAR) >= 1.6.0 is ok, doesn't need epoch)

MUST fix license => BSD (i haven't notice this in Comment #2, sorry)

 

Spec URL: http://theholbrooks.org/RPMS/php-pear-Mail-mimeDecode.spec
SRPM URL: http://theholbrooks.org/RPMS/php-pear-Mail-mimeDecode-1.5.0-3.fc8.src.rpm

- Changed license to BSD and added epoch to BR

APPROVED

New Package CVS Request
=======================
Package Name: php-pear-Mail-mimeDecode
Short Description: transcodes mime messages
Owners: static,limb,remi
Branches: F8, F9
InitialCC: 
Cvsextras Commits: yes

cvs done.

php-pear-Mail-mimeDecode-1.5.0-3.fc8 has been submitted as an update for Fedora 8

Committed and built, thanks again Remi!

php-pear-Mail-mimeDecode-1.5.0-3.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Package Change Request
======================
Package Name: php-pear-Mail-mimeDecode
New Branches: EL-6
Owners: remi

CVS Done