Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 190156 - Review Request: php-pear-HTTP
Summary: Review Request: php-pear-HTTP
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: Fedora Package Reviews List
: 197561 (view as bug list)
Depends On: 190252
TreeView+ depends on / blocked
Reported: 2006-04-28 08:40 UTC by Remi Collet
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2006-09-02 20:45:39 UTC
Type: ---
petersen: fedora-cvs+

Attachments (Terms of Use)

Description Remi Collet 2006-04-28 08:40:13 UTC
Spec URL:
The HTTP class is a class with static methods for doing
miscellaneous HTTP related stuff like date formatting,
language negotiation or HTTP redirection.

Also see Bug #190066

Comment 1 Remi Collet 2006-04-30 07:21:12 UTC
Waiting for Bug #190252

Comment 2 Remi Collet 2006-05-06 18:02:19 UTC
New spec and SRPM using %{_datadir}/pear/.pkgxml

Spec URL:

Build succeed with mock.

Comment 3 Aurelien Bompard 2006-05-20 09:06:53 UTC
Needs work:
 * don't remove the buildroot in %prep (it breaks rpmbuild -bi --short-circuit)
 * do the sanity check in %check
 * it would be nice to add "|| :" at the end of the pear lines in the scriptlets,
   to make sure they won't return a failure and block the whole rpm transaction

 * Use "BuildArch:" instead of the full BuildArchitectures
 * Include the LICENCE as you did with php-pear-Mail. This is not a requirement 
   though (it would be if the tarball shipped the license, which is not the case

Comment 4 Remi Collet 2006-05-20 15:27:29 UTC
Spec URL:
Mock Build.log :

- Require pear >= 1.4.9
- bundle the v3.01 PHP LICENSE file
- use --packagingroot (instead of -R)
- move check from install to check

Comment 5 Aurelien Bompard 2006-05-20 15:53:25 UTC
Review for release 3:
* RPM name is OK
* Source HTTP-1.4.0.tgz is the same as upstream
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* Works fine


Note : The line "%{__install} -m 644 -c %{SOURCE1} LICENSE" is more usually
found in %prep, but that's not a blocker.

Comment 6 Christopher Stone 2006-06-27 23:47:11 UTC
This package should have:
Requires(post):   php-pear >= 1.4.9
Requires(postun): php-pear >= 1.4.9

The dist tags should be removed from the %changelog
Use of ||: must be explained in spec file or removed

Comment 7 Paul Howarth 2006-06-28 07:52:06 UTC
(In reply to comment #6)
> Use of ||: must be explained in spec file or removed

The "||:" in the scriptlets should stay. In an rpm transaction involving
multiple packages (for example, an update of multiple php-pear-* packages), if
one of the scriptlets "fails", the rpm transaction stops at that point. This can
leave the system and its rpm database in a bad state. Supposing five packages
were updated and the %postun of the first one failed. The transaction would stop
at that point, leaving two copies of the other four packages installed.

See also where you'll find
many more examples of the use of this idiom.

Comment 8 Christopher Stone 2006-06-28 08:02:25 UTC
Okay, my bad then.  Remi: you can ignore my comments about ||: on all my reviews
of your packages.

Comment 9 Remi Collet 2006-07-03 20:45:57 UTC
Approved version 1.4.0-3 push to CVS.
Waiting for PHP guidelines before running the build and adding the last
suggestion (Comment #5 and #6).

Comment 10 Remi Collet 2006-07-15 08:56:28 UTC
*** Bug 197561 has been marked as a duplicate of this bug. ***

Comment 11 Remi Collet 2006-07-16 14:27:15 UTC
Spec URL:

Release 1.4.0-4 pushed to CVS (use the new /etc/rpm/macros.pear)
Waiting for PHP guidelines before running the build.

Comment 12 Kevin Fenzi 2006-09-02 20:45:39 UTC
Looks like the guidelines got finalized and this package was built and pushed 
out. I am going to close this NEXTRELEASE. Feel free to re-open if I am doing 
so in error. 

Comment 13 Remi Collet 2007-03-31 08:24:12 UTC
Package Change Request
Package Name: php-pear-HTTP
New Branches: EL-5

Comment 14 Jens Petersen 2007-04-02 12:48:27 UTC

Note You need to log in before you can comment on or make changes to this bug.