Bug 518017 - Review Request: php-pecl-sphinx - PECL extension for Sphinx SQL full-text search engine
Summary: Review Request: php-pecl-sphinx - PECL extension for Sphinx SQL full-text sea...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Remi Collet
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-08-18 13:51 UTC by Andrew Colin Kissa
Modified: 2014-08-29 13:00 UTC (History)
3 users (show)

Fixed In Version: 1.0.0-2.fc11
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-24 05:13:41 UTC
Type: ---
Embargoed:
fedora: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Andrew Colin Kissa 2009-08-18 13:51:18 UTC
Spec URL: http://topdog-software.com/oss/SRPMS/fedora/php-pecl-sphinx/php-pecl-sphinx.spec
SRPM URL: http://topdog-software.com/oss/SRPMS/fedora/php-pecl-sphinx/php-pecl-sphinx-1.0.0-1.fc11.src.rpm
Description:
This extension provides PHP bindings for libsphinxclient, 
client library for Sphinx the SQL full-text search engine.


rpmlint is clean, and scratch build successful - http://koji.fedoraproject.org/koji/taskinfo?taskID=1612311

Comment 1 Remi Collet 2009-09-06 05:47:46 UTC
All seems ok in the spec.

Just a quick note

Froms upstream, it requires PHP 5.1.3 (which is still not available in EPEL4), so, for documentation:
BuildRequires: php-devel >= 5.1.3

A minimal %check seems usefull (even if not mandatory), for example :

%check
# simple module load test
cd %{pecl_name}-%{version}
php --no-php-ini \
    --define extension_dir=modules \
    --define extension=%{pecl_name}.so \
    --modules | grep %{pecl_name}

I will do a full review asap.

Comment 3 Remi Collet 2009-09-11 14:15:26 UTC
REVIEW:

:) rpmlint is silent
php-pecl-sphinx.src: I: checking
php-pecl-sphinx.x86_64: I: checking
php-pecl-sphinx-debuginfo.x86_64: I: checking
3 packages and 1 specfiles checked; 0 errors, 0 warnings.
:) package name
:) spec file name
:) package must meet the Packaging Guidelines (PHP, ...)
:) license approved (PHP)
:) license match upstream
:) no license bundled by upstream
:) specfile legible, english, UTF8
:) sources math upstream
d5bd7c00a8e036d7f82e98ebb0d9b210  sphinx-1.0.0.tgz
:) build on F-11 x86_64 against PHP 5.3.1RC1
:) build on mock (F-11 i386)
:) BR ok
:) no locale
:) no shared lib (php extension are not "real" shared lib)
:) own all directories that it creates
:) perms ok
:) %clean section
:) consistently use macros
:) contain code
:) no large doc
:) run without %doc
:) no -devel, .pc needed
:) no GUI
:) not own files or directories already owned by other packages
:) %install, with rm -rf
:) extension (un)registration ok (pecl list)
:) final Requires
/usr/bin/pecl 
libsphinxclient-0.0.1.so()(64bit)
php(api) = 20090626
php(zend-abi) = 20090626
:) final provides
php-pecl(sphinx) = 1.0.0
sphinx.so()(64bit)  
php-pecl-sphinx = 1.0.0-2.fc8
php-pecl-sphinx(x86-64) = 1.0.0-2.fc8


So : APPROVED

Comment 4 Andrew Colin Kissa 2009-09-11 14:28:21 UTC
Thanks Remi,

New Package CVS Request
=======================
Package Name: php-pecl-sphinx
Short Description: PECL extension for Sphinx SQL full-text search engine
Owners: topdog
Branches: F-10 F-11 F-12 EL-5

Comment 5 Jason Tibbitts 2009-09-11 20:19:03 UTC
CVS done.

Comment 6 Fedora Update System 2009-09-14 16:24:41 UTC
php-pecl-sphinx-1.0.0-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/php-pecl-sphinx-1.0.0-2.fc10

Comment 7 Fedora Update System 2009-09-14 16:24:46 UTC
php-pecl-sphinx-1.0.0-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/php-pecl-sphinx-1.0.0-2.fc11

Comment 8 Fedora Update System 2009-09-15 07:46:25 UTC
php-pecl-sphinx-1.0.0-2.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update php-pecl-sphinx'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-9587

Comment 9 Fedora Update System 2009-09-15 07:54:25 UTC
php-pecl-sphinx-1.0.0-2.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update php-pecl-sphinx'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-9619

Comment 10 Fedora Update System 2009-09-24 05:13:36 UTC
php-pecl-sphinx-1.0.0-2.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2009-09-24 05:18:41 UTC
php-pecl-sphinx-1.0.0-2.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Remi Collet 2014-08-29 12:08:22 UTC
New Package SCM Request
=======================
Package Name: php-pecl-sphinx
Short Description: PECL extension for Sphinx SQL full-text search engine
Upstream URL: http://pecl.php.net/package/sphinx
Owners: topdog remi
Branches: epel7
InitialCC: remi

Comment 13 Gwyn Ciesla 2014-08-29 13:00:02 UTC
Git done (by process-git-requests).


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