Bug 987132 (php-ZendFramework2)

Summary: Review Request: php-ZendFramework2 - Zend Framework 2
Product: [Fedora] Fedora Reporter: Shawn Iwinski <shawn>
Component: Package ReviewAssignee: Gwyn Ciesla <gwync>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora, felix, gwync, notting
Target Milestone: ---Keywords: Reopened
Target Release: ---Flags: gwync: fedora-review+
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: php-ZendFramework2-2.2.5-2.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-11-24 03:44:20 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Shawn Iwinski 2013-07-22 18:59:18 UTC
Spec URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-ZendFramework2.spec

SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-ZendFramework2-2.2.1-1.fc19.src.rpm

Description:
Zend Framework 2 is an open source framework for developing web applications
and services using PHP 5.3+. Zend Framework 2 uses 100% object-oriented code
and utilizes most of the new features of PHP 5.3, namely namespaces, late
static binding, lambda functions and closures.

Zend Framework 2 evolved from Zend Framework 1, a successful PHP framework
with over 15 million downloads.


Fedora Account System Username: siwinski

Comment 1 Gwyn Ciesla 2013-09-12 13:21:45 UTC
Firt, this is 2.2.1, 2.2.4 is out.

- rpmlint checks return: 

php-ZendFramework2.spec:10: W: macro-in-comment %{version}
There is a unescaped macro after a shell style comment in the specfile. Macros
are expanded everywhere, so check if it can cause a problem in this case and
escape the macro with another leading % if appropriate.

php-ZendFramework2.spec:10: W: macro-in-comment %{version}
There is a unescaped macro after a shell style comment in the specfile. Macros
are expanded everywhere, so check if it can cause a problem in this case and
escape the macro with another leading % if appropriate.

php-ZendFramework2.spec:11: W: macro-in-comment %{version}
There is a unescaped macro after a shell style comment in the specfile. Macros
are expanded everywhere, so check if it can cause a problem in this case and
escape the macro with another leading % if appropriate.

php-ZendFramework2.spec:11: W: macro-in-comment %{version}
There is a unescaped macro after a shell style comment in the specfile. Macros
are expanded everywhere, so check if it can cause a problem in this case and
escape the macro with another leading % if appropriate.

Fix.

Lots of spelling and nodocs errors, ignorable.

- package meets naming guidelines
- package meets packaging guidelines
- license ( BSD ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 

So macros, and most current release, and that's it.  Pretty good otherwise.

Comment 2 Shawn Iwinski 2013-09-14 13:58:11 UTC
(In reply to Jon Ciesla from comment #1)
> So macros, and most current release, and that's it.  Pretty good otherwise.

Updated per review:

Spec URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-ZendFramework2.spec

SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-ZendFramework2-2.2.4-1.fc19.src.rpm

Comment 3 Gwyn Ciesla 2013-09-16 13:52:55 UTC
Awesome.  APPROVED.

Comment 4 Shawn Iwinski 2013-09-25 13:40:00 UTC
THANKS for the review!


New Package SCM Request
=======================
Package Name: php-ZendFramework2
Short Description: Zend Framework 2
Owners: siwinski
Branches: f19 f20 el6
InitialCC:

Comment 5 Gwyn Ciesla 2013-09-25 13:43:21 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2013-09-28 16:08:16 UTC
php-ZendFramework2-2.2.4-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/php-ZendFramework2-2.2.4-1.fc20

Comment 7 Fedora Update System 2013-09-28 16:08:27 UTC
php-ZendFramework2-2.2.4-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/php-ZendFramework2-2.2.4-1.fc19

Comment 8 Fedora Update System 2013-09-28 16:08:37 UTC
php-ZendFramework2-2.2.4-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-ZendFramework2-2.2.4-1.el6

Comment 9 Fedora Update System 2013-09-28 18:41:06 UTC
php-ZendFramework2-2.2.4-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 10 Fedora Update System 2013-10-22 05:08:05 UTC
php-ZendFramework2-2.2.4-1.fc19 has been pushed to the Fedora 19 stable repository.

Comment 11 Shawn Iwinski 2013-11-10 20:36:42 UTC
Package Change Request
======================
Package Name: php-ZendFramework2
New Branches: 
Owners: siwinski remi heffer
InitialCC:

Comment 12 Fedora Update System 2013-11-10 20:46:18 UTC
php-ZendFramework2-2.2.5-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/php-ZendFramework2-2.2.5-1.fc20

Comment 13 Fedora Update System 2013-11-10 20:46:35 UTC
php-ZendFramework2-2.2.5-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-ZendFramework2-2.2.5-1.el6

Comment 14 Fedora Update System 2013-11-10 20:46:49 UTC
php-ZendFramework2-2.2.5-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/php-ZendFramework2-2.2.5-1.fc19

Comment 15 Gwyn Ciesla 2013-11-11 12:58:02 UTC
Misformatted request, clearing flag.

Comment 16 Fedora Update System 2013-11-11 18:33:08 UTC
php-ZendFramework2-2.2.5-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 17 Fedora Update System 2013-11-24 03:44:20 UTC
php-ZendFramework2-2.2.5-2.fc20 has been pushed to the Fedora 20 stable repository.

Comment 18 Fedora Update System 2013-12-01 02:49:39 UTC
php-ZendFramework2-2.2.5-2.el6 has been pushed to the Fedora EPEL 6 stable repository.