Bug 874688 (Horde_Ldap) - Review Request: php-horde-Horde-Ldap - Horde LDAP libraries
Summary: Review Request: php-horde-Horde-Ldap - Horde LDAP libraries
Alias: Horde_Ldap
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Shawn Iwinski
QA Contact: Fedora Extras Quality Assurance
Depends On: Horde_Exception Horde_Util Horde_Test
Blocks: Horde_Core
TreeView+ depends on / blocked
Reported: 2012-11-08 16:31 UTC by Remi Collet
Modified: 2013-03-21 15:43 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2013-02-17 03:19:16 UTC
Type: ---
shawn: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)
php-horde-Horde-Ldap-review.txt (6.57 KB, text/plain)
2013-02-04 18:52 UTC, Shawn Iwinski
no flags Details
phpci.log (18.63 KB, text/x-log)
2013-02-04 18:52 UTC, Shawn Iwinski
no flags Details

Description Remi Collet 2012-11-08 16:31:20 UTC
Spec URL: https://raw.github.com/remicollet/remirepo/master/php/horde/php-horde-Horde-Ldap/php-horde-Horde-Ldap.spec
SRPM URL: http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Ldap-2.0.0-1.remi.src.rpm
A set of classes for connecting to LDAP servers and working with directory

Fedora Account System Username: remi

Comment 2 Shawn Iwinski 2013-02-04 18:52:22 UTC
Created attachment 692952 [details]

Generated by fedora-review 0.3.1 (b71abc1) last change: 2012-10-16
Buildroot used: fedora-rawhide-x86_64
Command line :/usr/bin/fedora-review -b 874688 --mock-config fedora-rawhide-x86_64

Comment 3 Shawn Iwinski 2013-02-04 18:52:48 UTC
Created attachment 692953 [details]

Comment 4 Shawn Iwinski 2013-02-04 19:00:14 UTC
A couple questions:

1) php(language): I am assuming you are going to wait until the newest version of PHP is pushed to RHEL 5 and 6 before pushing these packages out?  As of today (as far as I know) RHEL's PHP does not provide "php(language)" but I see (in the build system) that you added this virtual provide to new releases.

2) Since "Requires: php(language) >= 5.3.0" shouldn't there also be "BuildRequires: php(language) >= 5.3.0"?

No blockers.

===== APPROVED =====

Comment 5 Shawn Iwinski 2013-02-04 19:20:11 UTC
Also, for EPEL 5, the following are missing, but I see a bunch of your back-ports are missing these as well (just pointing these out):

1) "rm -rf %{buildroot}" in %install

2) "%clean
    rm -rf %{buildroot}"

Comment 6 Remi Collet 2013-02-05 07:33:06 UTC
For php(language), you're right, I will switch back to php-common until RHEL-6.4 is released. and add the missing backport stuff.

Thanks for the review.

New Package SCM Request
Package Name: php-horde-Horde-Ldap
Short Description: Horde LDAP libraries
Owners: remi nb
Branches: f18 el6

Comment 7 Gwyn Ciesla 2013-02-05 13:47:24 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2013-02-05 16:44:33 UTC
php-horde-Horde-Ldap-2.0.2-1.fc18 has been submitted as an update for Fedora 18.

Comment 9 Fedora Update System 2013-02-05 16:45:04 UTC
php-horde-Horde-Ldap-2.0.2-1.el6 has been submitted as an update for Fedora EPEL 6.

Comment 10 Fedora Update System 2013-02-05 21:22:55 UTC
php-horde-Horde-Ldap-2.0.2-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 11 Fedora Update System 2013-02-17 03:19:21 UTC
php-horde-Horde-Ldap-2.0.2-1.fc18 has been pushed to the Fedora 18 stable repository.

Comment 12 Fedora Update System 2013-02-22 19:05:21 UTC
php-horde-Horde-Ldap-2.0.2-1.el6 has been pushed to the Fedora EPEL 6 stable repository.

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