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-1.fc5.src.rpm Description: The Log framework provides an abstracted logging system. It supports logging to console, file, syslog, SQL, Sqlite, mail, and mcal targets. It also provides a subject - observer mechanism.
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