[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-07-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

Christof Damian chris...@damian.net changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #8 from Christof Damian chris...@damian.net 2010-07-04 07:24:42 
EDT ---
Package Change Request
==
Package Name: php-phpunit-PHP-TokenStream
New Branches: EL-6
Owners: cdamian

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-07-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

--- Comment #9 from Kevin Fenzi ke...@tummy.com 2010-07-04 21:43:16 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2010-02-26 
22:36:16 EST ---
php-phpunit-PHP-TokenStream-0.9.1-2.fc12 has been pushed to the Fedora 12
stable repository.  If problems still persist, please make note of it in this
bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|php-phpunit-PHP-TokenStream |php-phpunit-PHP-TokenStream
   |-0.9.1-2.fc13   |-0.9.1-2.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||php-phpunit-PHP-TokenStream
   ||-0.9.1-2.fc13
 Resolution||ERRATA

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2010-02-23 
17:01:58 EST ---
php-phpunit-PHP-TokenStream-0.9.1-2.fc13 has been submitted as an update for
Fedora 13.
http://admin.fedoraproject.org/updates/php-phpunit-PHP-TokenStream-0.9.1-2.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

--- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2010-02-22 15:11:28 EST 
---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

Christof Damian chris...@damian.net changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Christof Damian chris...@damian.net 2010-02-21 05:12:49 
EST ---
Thanks for the Review. I will change the summary on import.

New Package CVS Request
===
Package Name: php-phpunit-PHP-TokenStream
Short Description: Wrapper around PHP's tokenizer extension
Owners: cdamian
Branches: F-12 F-13
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension

2010-02-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=561981

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com
 AssignedTo|nob...@fedoraproject.org|fed...@famillecollet.com
   Flag||fedora-review+

--- Comment #1 from Remi Collet fed...@famillecollet.com 2010-02-21 02:16:39 
EST ---
REVIEW

+ rpmlint
php-phpunit-PHP-TokenStream.spec: I: checking-url
http://pear.phpunit.de/get/PHP_TokenStream-0.9.1.tgz (timeout 10 seconds)
php-phpunit-PHP-TokenStream.src: I: checking
php-phpunit-PHP-TokenStream.src: W: spelling-error Summary(en_US) PHP's -
HP's, PCP's, P HP's
php-phpunit-PHP-TokenStream.src: W: spelling-error Summary(en_US) tokenizer -
tokenism, token, patronizer
php-phpunit-PHP-TokenStream.src: W: spelling-error %description -l en_US PHP's
- HP's, PCP's, P HP's
php-phpunit-PHP-TokenStream.src: W: spelling-error %description -l en_US
tokenizer - tokenism, token, patronizer
php-phpunit-PHP-TokenStream.src: I: checking-url
http://github.com/sebastianbergmann/php-token-stream (timeout 10 seconds)
php-phpunit-PHP-TokenStream.src: I: checking-url
http://pear.phpunit.de/get/PHP_TokenStream-0.9.1.tgz (timeout 10 seconds)
php-phpunit-PHP-TokenStream.noarch: I: checking
php-phpunit-PHP-TokenStream.noarch: W: spelling-error Summary(en_US) PHP's -
HP's, PCP's, P HP's
php-phpunit-PHP-TokenStream.noarch: W: spelling-error Summary(en_US) tokenizer
- tokenism, token, patronizer
php-phpunit-PHP-TokenStream.noarch: W: spelling-error %description -l en_US
PHP's - HP's, PCP's, P HP's
php-phpunit-PHP-TokenStream.noarch: W: spelling-error %description -l en_US
tokenizer - tokenism, token, patronizer
php-phpunit-PHP-TokenStream.noarch: I: checking-url
http://github.com/sebastianbergmann/php-token-stream (timeout 10 seconds)
php-phpunit-PHP-TokenStream.noarch: W: no-documentation
2 packages and 1 specfiles checked; 0 errors, 9 warnings.
+ package name ok
+ spec file name ok
+ package meet the PHP Guidelines
+ License ok : BSD
+ License is upstream
+ spec in english and legible
+ no license file in sources is provided
+ sources match the upstream sources
328c7d9e95fc9b0078b180bbb8c532d1  PHP_TokenStream-0.9.1.tgz
+ Source URL ok
+ build  on F12.x86_64
+ BuildRequires ok
php-pear = 1:1.8.1
php-channel(pear.phpunit.de)
+ no locale
+ no .so
+ own all directories that it creates
+ no duplicate file
+ %defattr ok
+ %clean section
+ use macros consistently
+ no %define
+ contain code
+ no documentation
+ no devel
+ no pkgconfig
+ no sub-package
+ no GUI
+ don't own files or directories already owned by other packages
+ %install start with rm -rf 
+ valid UTF-8
+ build in koji
+ no test suite (Tests dir in git not provided,)
+ scriptlets ok
- Final Requires 
/usr/bin/pear  
php-channel(pear.phpunit.de)  
php-common = 5.2.0
php-tokenizer  
+ Final Provides ok
php-pear(pear.phpunit.de/PHP_TokenStream) = 0.9.1
php-phpunit-PHP-TokenStream = 0.9.1-1.fc8
+ Koji scratch build
http://koji.fedoraproject.org/koji/taskinfo?taskID=2002482


removing the PHP's in sumnary should be fine.



* APPROVED *

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review