Bug 894100

Summary: Review Request: php-pear-File-Fstab - Read and write fstab files
Product: [Fedora] Fedora Reporter: Remi Collet <fedora>
Component: Package ReviewAssignee: Johan Cwiklinski <fedora>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora, notting, package-review
Target Milestone: ---Flags: fedora: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-23 01:28:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
fedora-review results none

Description Remi Collet 2013-01-10 17:32:44 UTC
Spec URL: https://raw.github.com/remicollet/remirepo/1eff2458ecb9eb1b73218a4888dcba0a6766e802/php/horde/php-horde-horde/php-horde-horde.spec
SRPM URL: http://rpms.famillecollet.com/SRPMS/php-pear-File-Fstab-2.0.3-1.remi.src.rpm
Description: 
File_Fstab is an easy-to-use package which can read & write UNIX fstab
files. It presents a pleasant object-oriented interface to the fstab.
Features:
* Supports blockdev, label, and UUID specification of mount device.
* Extendable to parse non-standard fstab formats by defining a new Entry
class for that format.
* Easily examine and set mount options for an entry.
* Stable, functional interface.
* Fully documented with PHPDoc.

Fedora Account System Username: remi

Comment 1 Remi Collet 2013-01-10 17:36:19 UTC
Sorry,
Spec URL: https://raw.github.com/remicollet/remirepo/1eff2458ecb9eb1b73218a4888dcba0a6766e802/php/pear/php-pear-File-Fstab/php-pear-File-Fstab.spec


Notes : 
- target Fedora, EL-5, EL-6
- new dependency of horde 5.0.3

Comment 2 Johan Cwiklinski 2013-01-11 07:40:09 UTC
Created attachment 676697 [details]
fedora-review results

Comment 3 Johan Cwiklinski 2013-01-11 07:43:11 UTC
Here is the phpci extensions report output:
BASE: /home/builder/894100-php-pear-File-Fstab/srpm-unpacked/File_Fstab-2.0.3
-------------------------------------------------------------------------------
PHP COMPAT INFO EXTENSION SUMMARY
-------------------------------------------------------------------------------
  EXTENSION                             EXT min/Max       PHP min/Max     COUNT
-------------------------------------------------------------------------------
  Core                                  4.0.0             4.0.0              31
  pcre                                  0.0               4.0.0               2
  standard                              4.0.7             4.0.0              18
-------------------------------------------------------------------------------
A TOTAL OF 3 EXTENSIONS WERE FOUND
REQUIRED PHP 4.0.0 (MIN)

Comment 4 Johan Cwiklinski 2013-01-11 07:47:56 UTC
php-common requirement is a false positive here. Requires are correct.

Package traball is missing LICENSE file, but that is already reported upstream, and LICENSE file is provided.

Issues that has been fixed in the specfile has been reported upstream.

Well, guidelines are followed, no blocker :)

***** APPROVED *****

Comment 5 Remi Collet 2013-01-11 07:51:17 UTC
Thanks for the fast review.


New Package SCM Request
=======================
Package Name: php-pear-File-Fstab
Short Description: Read and write fstab files
Owners: remi
Branches: f17 f18 el5 el6
InitialCC:

Comment 6 Gwyn Ciesla 2013-01-11 14:40:21 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2013-01-11 15:14:56 UTC
php-pear-File-Fstab-2.0.3-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/php-pear-File-Fstab-2.0.3-1.fc18

Comment 8 Fedora Update System 2013-01-11 15:16:08 UTC
php-pear-File-Fstab-2.0.3-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/php-pear-File-Fstab-2.0.3-1.fc17

Comment 9 Fedora Update System 2013-01-11 15:20:49 UTC
php-pear-File-Fstab-2.0.3-1.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/php-pear-File-Fstab-2.0.3-1.el5

Comment 10 Fedora Update System 2013-01-11 15:21:38 UTC
php-pear-File-Fstab-2.0.3-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-pear-File-Fstab-2.0.3-1.el6

Comment 11 Fedora Update System 2013-01-12 15:02:53 UTC
php-pear-File-Fstab-2.0.3-1.fc17 has been pushed to the Fedora 17 testing repository.

Comment 12 Fedora Update System 2013-01-23 01:28:17 UTC
php-pear-File-Fstab-2.0.3-1.fc17 has been pushed to the Fedora 17 stable repository.

Comment 13 Fedora Update System 2013-01-23 01:42:56 UTC
php-pear-File-Fstab-2.0.3-1.fc18 has been pushed to the Fedora 18 stable repository.

Comment 14 Fedora Update System 2013-01-29 19:59:03 UTC
php-pear-File-Fstab-2.0.3-1.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 15 Fedora Update System 2013-01-29 19:59:23 UTC
php-pear-File-Fstab-2.0.3-1.el5 has been pushed to the Fedora EPEL 5 stable repository.