Bug 879397 - Review Request: php-Slim - PHP micro framework
Summary: Review Request: php-Slim - PHP micro framework
Keywords:
Status: CLOSED CURRENTRELEASE
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: 2012-11-22 21:13 UTC by Johan Cwiklinski
Modified: 2012-12-12 22:53 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-05 06:54:07 UTC
Type: ---
Embargoed:
fedora: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
php-Slim-review.txt (9.06 KB, text/plain)
2012-11-25 07:57 UTC, Remi Collet
fedora: review?
Details

Description Johan Cwiklinski 2012-11-22 21:13:23 UTC
Spec URL: http://odysseus.x-tnd.be/fedora/php-Slim/php-Slim.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/php-Slim/php-Slim-2.1.0-2.fc17.src.rpm
Description: PHP micro framework

Fedora Account System Username: trasher

Comment 1 Remi Collet 2012-11-25 07:57:00 UTC
Created attachment 651409 [details]
php-Slim-review.txt

Generated by fedora-review 0.3.1 (b71abc1) last change: 2012-10-16
Buildroot used: fedora-17-x86_64
Command line :/usr/bin/fedora-review -b 879397


Please fixed the listed issues.

Comment 2 Johan Cwiklinski 2012-11-25 10:58:17 UTC
Thank you for taking this one :)

I'll fix the issues very soon. You're right about the paths, I'll change them, no need for an additional sub directory :)

Comment 4 Johan Cwiklinski 2012-11-25 13:53:55 UTC
Hum... I forget to remove phpunit.xml.dist from %doc... The new version:
Spec URL: http://odysseus.x-tnd.be/fedora/php-Slim/php-Slim.spec
RPMS URL: http://odysseus.x-tnd.be/fedora/php-Slim/php-Slim-2.1.0-4.fc17.src.rpm

And the specfile's diff:
http://odysseus.x-tnd.be/fedora/php-Slim/2.1.0-3_2.1.0-4.diff

Comment 5 Johan Cwiklinski 2012-11-25 14:49:24 UTC
I did not pay attention to a rpmlint issue on phpunit.xml.dist permissions. I've also removed the %if in %check that is not needeed.

Here the new version:
Spec URL: http://odysseus.x-tnd.be/fedora/php-Slim/php-Slim.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/php-Slim/php-Slim-2.1.0-5.fc17.src.rpm

And the specfile's diff:
http://odysseus.x-tnd.be/fedora/php-Slim/2.1.0-4_2.1.0-5.diff

Comment 6 Remi Collet 2012-11-25 14:53:27 UTC
All blockers taken off, looks good.


==== APPROVED ====

Comment 7 Johan Cwiklinski 2012-11-25 14:56:00 UTC
New Package SCM Request
=======================
Package Name: php-Slim
Short Description: PHP micro framework
Owners: trasher
Branches: f17 f18 el6
InitialCC:

Comment 8 Gwyn Ciesla 2012-11-25 16:03:29 UTC
Git done (by process-git-requests).

Comment 9 Fedora Update System 2012-11-25 17:38:20 UTC
php-Slim-2.1.0-5.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/php-Slim-2.1.0-5.fc17

Comment 10 Fedora Update System 2012-11-25 17:48:18 UTC
php-Slim-2.1.0-5.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-Slim-2.1.0-5.el6

Comment 11 Fedora Update System 2012-11-25 19:27:14 UTC
php-Slim-2.1.0-5.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/php-Slim-2.1.0-5.fc18

Comment 12 Fedora Update System 2012-11-26 01:50:09 UTC
php-Slim-2.1.0-5.fc17 has been pushed to the Fedora 17 testing repository.

Comment 13 Fedora Update System 2012-12-05 06:54:08 UTC
php-Slim-2.1.0-5.fc17 has been pushed to the Fedora 17 stable repository.

Comment 14 Fedora Update System 2012-12-05 07:15:53 UTC
php-Slim-2.1.0-5.fc18 has been pushed to the Fedora 18 stable repository.

Comment 15 Fedora Update System 2012-12-12 22:53:40 UTC
php-Slim-2.1.0-5.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.