Bug 1157700

Summary: Review Request: php-ocramius-code-generator-utils - A set of code generator utilities built on top of PHP-Parsers
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
Target Milestone: ---Flags: fedora: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: php-ocramius-code-generator-utils-0.3.0-2.el7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-11-16 14:47:23 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: 1157708    
Attachments:
Description Flags
phpci.log
none
review.txt none

Description Shawn Iwinski 2014-10-27 14:16:28 UTC
Spec URL: https://raw.githubusercontent.com/siwinski/rpms/375ca5028ea7c8c7df11df954265a85a417ffee3/php-ocramius-code-generator-utils.spec

SRPM URL: https://siwinski.fedorapeople.org/copr/php-experimental/php-ocramius-code-generator-utils-0.3.0-1.fc20.src.rpm

Description:
A set of code generator utilities built on top of PHP-Parsers that ease its use
when combined with Reflection.


Fedora Account System Username: siwinski


COPR build: http://copr.fedoraproject.org/coprs/siwinski/php-experimental/build/55152/

Comment 1 Remi Collet 2014-11-04 08:52:26 UTC
Created attachment 953491 [details]
phpci.log

 phpCompatInfo version 3.5.0 static analyze run in src directory

Comment 2 Remi Collet 2014-11-04 08:53:06 UTC
Created attachment 953492 [details]
review.txt

Generated by fedora-review 0.5.2 (63c24cb) last change: 2014-07-14

Comment 3 Remi Collet 2014-11-04 08:55:01 UTC
Minor cleanup (no blocker)

- ensure you use the correct spec (see review.txt) without the debug "find"
- consider to silent the include (in test autoloader) to avoid some warning at build time


Everything seems OK.

=== APPROVED ===

Comment 4 Shawn Iwinski 2014-11-04 17:45:09 UTC
(In reply to Remi Collet from comment #3)
> Minor cleanup (no blocker)
> 
> - ensure you use the correct spec (see review.txt) without the debug "find"

Oooops, my bad.  That was a debug b/c of an issue w/ the previous PHPParser release.

> - consider to silent the include (in test autoloader) to avoid some warning
> at build time

I'm not sure how that un-silenced.  Perhaps during my debug.

> Everything seems OK.
> 
> === APPROVED ===

I'll make sure everything is cleaned up after initial import.

Comment 5 Shawn Iwinski 2014-11-04 17:49:31 UTC
THANKS for the review!


New Package SCM Request
=======================
Package Name: php-ocramius-code-generator-utils
Short Description: A set of code generator utilities built on top of PHP-Parsers
Upstream URL: https://github.com/Ocramius/CodeGenerationUtils
Owners: siwinski
Branches: f21 epel7
InitialCC:

Comment 6 Gwyn Ciesla 2014-11-05 12:09:24 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2014-11-06 16:38:28 UTC
php-ocramius-code-generator-utils-0.3.0-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/php-ocramius-code-generator-utils-0.3.0-2.fc21

Comment 8 Fedora Update System 2014-11-06 16:38:34 UTC
php-ocramius-code-generator-utils-0.3.0-2.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/php-ocramius-code-generator-utils-0.3.0-2.el7

Comment 9 Fedora Update System 2014-11-07 05:32:53 UTC
php-ocramius-code-generator-utils-0.3.0-2.fc21 has been pushed to the Fedora 21 testing repository.

Comment 10 Fedora Update System 2014-11-16 14:47:23 UTC
php-ocramius-code-generator-utils-0.3.0-2.fc21 has been pushed to the Fedora 21 stable repository.

Comment 11 Fedora Update System 2014-11-24 21:22:58 UTC
php-ocramius-code-generator-utils-0.3.0-2.el7 has been pushed to the Fedora EPEL 7 stable repository.