Bug 256521 - Review Request: php-pear-Creole - A database abstraction layer for PHP5
Summary: Review Request: php-pear-Creole - A database abstraction layer for PHP5
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Remi Collet
QA Contact: Fedora Extras Quality Assurance
Depends On: 275211 283401
TreeView+ depends on / blocked
Reported: 2007-08-27 14:22 UTC by Alexander Kahl
Modified: 2007-11-30 22:12 UTC (History)
3 users (show)

Fixed In Version: 1.1.0-5.fc7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-10-11 01:47:26 UTC
Type: ---
fedora: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Alexander Kahl 2007-08-27 14:22:57 UTC
Spec URL: http://prelive.iconmobile.com/dev31/fedora-icm-repo/Fedora/7/SPECS/php-pear-Creole.spec
SRPM URL: http://prelive.iconmobile.com/dev31/fedora-icm-repo/Fedora/7/SRPMS/php-pear-Creole-1.1.0-3.fc7.src.rpm
Creole is a database abstraction layer for PHP5. It abstracts PHP's
native db-specific API to create more portable code while also
providing developers with a clean fully object-oriented interface
based loosely on the API for Java's JDBC.


This is my first package and I'm searching for a sponsor.
php-pear-Creole relies on php-channel-phpdb that is also available in the same repository. I wasn't sure about the channel's license because there are no clear indicators if there is any, so I've left the license field 'N/A' for now.
My employer explicitly allowed me to release packages I've made at work and there are several other PHP5-related ones available in my repository that I'd like to commit as soon as I've got approval.

Comment 1 Remi Collet 2007-09-03 14:17:33 UTC
Please, submit php-channel-phpdb to review and add it as a blocker for this review.

There is no Guidelines yet for "channel", we have to work on it.

- Please, don't use %{pear_phpdir}/* in %file
- keep version in php-pear BR (mainly to avoid EL-4 build)

I don't know about License for channel, but you could probably use the same
(LGPLv2) than the package provided by this channel or simply "Public domain"
(N/A is not a acceptable)

I could do a review, but i couldn't sponsor you.

Comment 2 Remi Collet 2007-09-03 14:23:36 UTC
- also change name to php-pear-creole (no uppercase in upstream package name)

Comment 3 Alexander Kahl 2007-09-03 15:27:33 UTC
Thanks Remi for helping me!

- add php-channel-phpdb as a blocker: fixed
- change license field in the channel: fixed
- replace ${pear_phpdir} with ${_datadir}/pear: fixed
- keep version in php-pear BR: sorry, what does that mean?
- name change to php-pear-creole: fixed

New Spec URL:

What can I do to improve my chances finding a sponsor?

Comment 4 Remi Collet 2007-09-08 05:47:07 UTC
You must use 
BuildRequires:  php-pear >= 1:1.4.9-1.2
Because macro used for this spec are not defined in previous pear release.

I will prefer R and BR on php-channel(pear.phpdb.org) rather then php-channel-phpdb.
Ok, there is no guidelines yet, but i will propose this

When i wrote don't use %{pear_phpdir}/*, i mean use %{pear_phpdir} but not *
Probably ${pear_phpdir}/creole

I can't build this RPM, don't understand why.
"pear install" hangs.

I will try later.

Comment 5 Remi Collet 2007-09-08 06:24:10 UTC
It seems there is a bug in PEAR.

Build ok with F7 pear 1.5.0

Build hangs with newer version (1.5.4, 1.6.1).
This issue also affects php-pear-PHPUnit.

See Bug #283401

Comment 6 Alexander Kahl 2007-09-11 15:29:15 UTC
Updated Spec URL:
Updated SRPM URL:

- proper php-pear requirement
- changed php-channel requirement syntax
- rectified file ownership

Shall I nail the pear version requirement to < 1.5.4?

Comment 7 Remi Collet 2007-09-11 18:15:28 UTC
> Shall I nail the pear version requirement to < 1.5.4?

No. Bug #283401 is closed (pear 1.6.2 pushed in rawhide)

Comment 8 Alexander Kahl 2007-09-18 12:46:27 UTC
Any progress here? Is -5 OK?

Comment 9 Remi Collet 2007-09-22 14:59:33 UTC
Have you been sponsored ?

Comment 10 Mamoru TASAKA 2007-09-22 15:03:09 UTC
(In reply to comment #9)
> Have you been sponsored ?

Now I am sponsoring Alexander (bug 275831)

Comment 11 Alexander Kahl 2007-10-01 20:17:48 UTC
Remi, will you continue the review now? 

Comment 12 Remi Collet 2007-10-02 06:27:07 UTC

I will work on it this week-end.

Comment 13 Remi Collet 2007-10-08 05:24:41 UTC
* rpmlint :
W: no-documentation
* The package is named according to the Package Naming Guidelines.
* The spec file name match the base package
* The package meet the PHP Packaging Guidelines (see below)
* Fedora approved license (LGPLv2)
* License field in the package spec file match the actual license
* The spec file is written in American English
* The spec file is legible
* The sources used to build the package must match the upstream source
87480d20b83be9ae859153539ec57c70  creole-1.1.0.tgz
* build in mock (F-7, 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(pear.phpdb.org/creole) = 1.1.0
php-pear-creole = 1.1.0-5.fc8
* Final Requires
php-cli >= 5.0.0

** No test suite or example to run.

There is actually no PHP Guidelines for non standard channel, but this spec
conforms with other RPM.


Comment 14 Alexander Kahl 2007-10-09 16:15:39 UTC
New Package CVS Request
Package Name: php-pear-creole
Short Description: A database abstraction layer for PHP5
Owners: akahl
Branches: F-7
InitialCC: remi
Cvsextras Commits: yes

Comment 15 Kevin Fenzi 2007-10-10 00:18:10 UTC
cvs done.

Comment 17 Fedora Update System 2007-10-11 01:47:24 UTC
php-pear-creole-1.1.0-5.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.

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