Bug 987132 (php-ZendFramework2) - Review Request: php-ZendFramework2 - Zend Framework 2
Summary: Review Request: php-ZendFramework2 - Zend Framework 2
Keywords:
Status: CLOSED ERRATA
Alias: php-ZendFramework2
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Gwyn Ciesla
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-22 18:59 UTC by Shawn Iwinski
Modified: 2013-12-01 02:49 UTC (History)
4 users (show)

Fixed In Version: php-ZendFramework2-2.2.5-2.el6
Clone Of:
Environment:
Last Closed: 2013-11-24 03:44:20 UTC
Type: ---
Embargoed:
gwync: fedora-review+


Attachments (Terms of Use)

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.


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