Waiting for Bug #190252

New spec and SRPM using %{_datadir}/pear/.pkgxml

Spec URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.5-1.fc5.src.rpm

I'd like to see some movement on these php-pear package reviews, but we have a
distinct lack of packaging guidelines for PHP modules.

Here are a few issues I noticed:
"PEAR" in the summary isn't really descriptive; maybe something like "Abstracted
logging facility for PHP" would make more sense.

I guess RPM won't extract the php-pear(*) provides as it will for Perl, which is
too bad.  I wonder if it could be taught.

Use "BuildArch:" instead of "BuildArchitectures:"; it lines up better.

rpmlint disagrees with the overlong line in the description.

Could you explain the comment in %prep?

Perhaps rpmlint could be taught to do the sanity check so it doesn't have to
live in the spec.

Could you explain the comment in %postun?

I think you might need Requires(post): php-pear and Requires(postun): php-pear
(or is that php-pear(PEAR)?).

Unfortunately I can't do a proper test because the updated php-pear isn't in FC5
yet and rawhide is broken at the moment.

There are movement, see Bug #190066.
The new php-pear-1.4.9 will allow us to progress.

Description used, is the one provide upstream on the package.xml. I must agree
it isn't very descriptive, but i don't know if it's a good idea to chance it.

For %prep. This comment is from pear template.spec
# XXX Source files location is missing here in pear cmd
It mean pear is only use to build "pearrc" (source will be provide in %install)

The Sanity check is use to check the job done by pear, because version 1.4.6
sometime left %[buildroot} relative path in .php file.
Using pear-1.4.9 and --packagingroot (insteaed of -R) solve this problem.

Of course i can remove it, but it could be useful for people who want to rebuild
the RPM for another distro.

%postun workaround ( ... || true) is for pear-1.4.6 which is unable to uninstall
somme package. No problem with pear-1.4.9.
This workaround is to avoid scriptlet (and uninstall) failure.
In this case the package is uninstalled, but not unregistred in pear extension
list. 
A solution could be to (Build)Requires pear > 1.4.7 ???

Yes php-pear(PEAR) is provide by php-pear.

php-pear-1.4.9 is in rawhide and in FC5-testing (see Bug #190252)

> Description used, is the one provide upstream on the package.xml. I must agree
> it isn't very descriptive, but i don't know if it's a good idea to chance it.

Upstream can be broken in many ways.  We have to change the descriptions for
Perl modules as well.  Since the summary is the first thing the users will see,
it must be as descriptive as possible in the 60 or so characters available.

> For %prep. This comment is from pear template.spec

Do we have a pear template in fedora-rpmdevtools?  I don't see one.

[sanity check]
> Of course i can remove it, but it could be useful for people who want to rebuild
> the RPM for another distro.

We don't usually worry about that, but my point is that rpmlint is our sanity
checker and it's worth discussing whether it should be taught to check for
things like that.

Are you targeting FC4 with these packages?  If not, we should just require the
unbuggy php-pear version once it has been released.

I'm still waiting for either a buildable rawhide or the updated php-pear package
in FC5 to do a full review.  If anyone can answer the question of whether the
scriptlets need Requires(post) and Requires(postun) dependencies, please chip in.

> Do we have a pear template in fedora-rpmdevtools?  I don't see one.

/usr/share/pear/data/PEAR/template.spec provides by php-pear (old command : pear
makerpm).
or
/usr/share/pear/data/PEAR_Command_Packaging/template.spec provides by 
php-pear-PEAR-Command-Packaging (Bug #185423, new command pear make-rpm-spec,
soon in Extras).

I think this template is not really suitable for Extras and could really by
improve, but i'm not the packager for this.

Spec URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.4-3.fc5.src.rpm
Mock build log : http://remi.collet.free.fr/rpms/extras/php-pear-Log-build.log

- Require pear >= 1.4.9
- Requires(hint): (only comment actually) + description
- bundle the v3.01 PHP LICENSE file (as in php-pear)
- use --packagingroot (instead of -R)
- check from install to check (as in php-pear)

ooops.
SRPM URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.5-3.fc5.src.rpm

Spec URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM URL: http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.6-1.fc5.src.rpm
Mock build log : http://remi.collet.free.fr/rpms/extras/php-pear-Log-build.log

- update to 1.9.6
- install Licence in prep

Hey guys, I did not know this review request existed.  I created a php-pear-MDB2
review request and php-pear-Log review request.

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=196823

php-pecl-xdebug request:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=196749

php-pear-MDB2 reqeust:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=196793

NOTE that my php-pear-Log spec file uses these two packages.

*** Bug 196823 has been marked as a duplicate of this bug. ***

This package should require php-pear-MDB2 since I already went through the
trouble of making this package.  Please review this package so that it may be
closed.

SPEC : http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM : http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.7-1.fc5.src.rpm
Mock : http://remi.collet.free.fr/rpms/extras/php-pear-Log-build.log

Changes : 
- update to 1.9.7
- use new macros from /etc/rpm/macros.pear
- add Requires php-pear(DB) and php-pear(MDB2)

MDB2/DB are defined as optional upstream and i'm using Log extension without
them for a while. But you're probably right that we must include "optional"
requires as mandatory while they are not handled by rpm/yum.

SPEC : http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM : http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.8-2.fc5.src.rpm
Mock : http://remi.collet.free.fr/rpms/extras/php-pear-Log-build.log

Changes : 
- update to 1.9.8
- failsafe scriplet

Hi Remi, can you update the %setup and %install for this guy too?  Thanks.

SPEC : http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM : http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.8-3.fc5.src.rpm
Mock : http://remi.collet.free.fr/rpms/extras/php-pear-Log-build.log

%changelog
* Mon Sep 04 2006 Remi Collet <Fedora> 1.9.8-3
- new and simpler %%prep and %%install

Note : i've also update php-pear-Mail and php-pear-HTTP (already approved) on
the CVS.

Hi Remi, since I need this built for FC-5, can you update the spec file to match
as closely as possible the latest template:

http://cvs.fedora.redhat.com/viewcvs/*checkout*/fedora-rpmdevtools/spectemplate-php-pear.spec?root=fedora&rev=1.4&sortby=date

You may also want to include changes that hanvn't been officially committed to
cvs yet:

https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=135697

SPEC : http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM : http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.8-4.fc5.src.rpm
Mock : http://remi.collet.free.fr/rpms/extras/php-pear-Log-build.log

Note :
- i didn't use %{ClassName} as it's very useful on a template but not on a
"generated" specfile.
- i didn't check for package.xml/package2.xml as auto-generation knows which one use
- i create docdir in the main build directory (cleaner, i think)

Looking good, please change:

- Summary should not have "PEAR", a summary should be as short as possible while
still being descriptive enough to convey what it is.  Extra stuff like "A" or
"The" or "PEAR" should be removed from summaries.

- Set BuildRequires:  php-pear >= 1:1.4.9-1.2

Normally I would approve it now and request you make the changes in CVS, but the
BuildRequires is a blocker and must be fixed before I can approve it.

SPEC : http://remi.collet.free.fr/rpms/extras/php-pear-Log.spec
SRPM : http://remi.collet.free.fr/rpms/extras/php-pear-Log-1.9.8-5.fc5.src.rpm

I've change then BR, but i really think it's not a BR
- 1.4.9 is required to have --packagingroot working
- 1.4.9-1.1 is required for memory limit, but 8M enough for this extension
- 1.4.9-1.2 is required for macros, which are embeded in the spec.
so...

Yeah, I just logged in to approve this anyway because you had already defined
the macros.  Looks good now anyway, approved.

BTW, could you be so kind and sync and build this package for FC5?  I'm going to
need this for some of my packages which I want to build on FC5.  Thanks for all
your work on this and the other php packages.  :D

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