Bug 1102941 (php-pclzip)

Summary: Review Request: php-pclzip - Compression and extraction functions for Zip formatted archives
Product: [Fedora] Fedora Reporter: Shawn Iwinski <shawn>
Component: Package ReviewAssignee: Remi Collet <fedora>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora, package-review, Tr
Target Milestone: ---Flags: fedora: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: php-pclzip-2.8.2-1.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-06-18 22:25:20 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:
Bug Depends On:    
Bug Blocks: 1102942    
Attachments:
Description Flags
phpci.log
none
review.txt none

Description Shawn Iwinski 2014-05-29 21:20:55 UTC 
Spec URL: https://raw.githubusercontent.com/siwinski/rpms/f2002c6480c0a6a5ab718720fbdfbe17a3c06142/php-pclzip.spec

SRPM URL: http://siwinski.fedorapeople.org/SRPMS/php-pclzip-2.8.2-1.fc20.src.rpm

Description: 
PclZip library offers compression and extraction functions for Zip formatted
archives (WinZip, PKZIP).

PclZip gives you the ability to manipulate zip formatted archives. You can
create an archive, list the content and extract all its content in the file
system.

PclZip defines an object class which represent a Zip Archive. This class
manages the archive properties and offers access method and actions on
the archive.


Fedora Account System Username: siwinski
Comment 1 Remi Collet 2014-06-02 13:02:06 UTC 
Created attachment 901455 [details]
phpci.log
Comment 2 Remi Collet 2014-06-02 13:03:02 UTC 
Created attachment 901456 [details]
review.txt
Comment 3 Remi Collet 2014-06-02 13:03:52 UTC 
[!]: Dist tag is present (not strictly required in GL).
  Use %{?dist} instead of %{dist}
[!] rpmlint
  php-pclzip.noarch: E: incorrect-fsf-address /usr/share/doc/php-pclzip/gnu-lgpl.txt
  Should be reported to upstream
[x]: Latest version is packaged.
  Ok, but this version was released 5 years ago
  This project seems dead.
  You should understand you will probably have to maintain it in case of bug



Please fix %{?dist} on import

No blocker

== APPROVED ==
Comment 4 Shawn Iwinski 2014-06-05 20:05:10 UTC 
THANKS for the review!  I'll fix %{?dist} on import.


New Package SCM Request
=======================
Package Name: php-pclzip
Short Description: Compression and extraction functions for Zip formatted archives
Upstream URL: http://www.phpconcept.net/pclzip
Owners: siwinski
Branches: f20 el6 epel7
InitialCC:
Comment 5 Gwyn Ciesla 2014-06-06 12:15:34 UTC 
Git done (by process-git-requests).
Comment 6 Fedora Update System 2014-06-06 23:16:28 UTC 
php-pclzip-2.8.2-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/php-pclzip-2.8.2-1.fc20
Comment 7 Fedora Update System 2014-06-06 23:16:37 UTC 
php-pclzip-2.8.2-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-pclzip-2.8.2-1.el6
Comment 8 Fedora Update System 2014-06-09 15:36:52 UTC 
php-pclzip-2.8.2-1.el6 has been pushed to the Fedora EPEL 6 testing repository.
Comment 9 Fedora Update System 2014-06-18 22:25:20 UTC 
php-pclzip-2.8.2-1.fc20 has been pushed to the Fedora 20 stable repository.
Comment 10 Fedora Update System 2014-07-07 19:02:58 UTC 
php-pclzip-2.8.2-1.el6 has been pushed to the Fedora EPEL 6 stable repository.