Created attachment 670366 [details]
phpci.log

Created attachment 670367 [details]
php-Analog-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 890417

[!]: Requires correct, justified where necessary.
[!]: Package installs properly.
     Note: Installation errors (see attachment)
	php-mongo => php-pecl(mongo)

I will prefer to not require this optional dependency (pull a lot of stuff)
Proposal:
   remove the requires, and add a comment in package description
or create a php-Analog-mongo sub-package (seems a bit ugly for a single file)


[!]: Package complies to the Packaging Guidelines
	I don't understand version, why 1.0.0.1 ?
	If post release, should be 1.0.0-1.gitxxx
	If pre release , should be 1.0.0.1-0.1.gitxxx
	or justify/explain

[!]: If the source package does not include license text(s) as a separate file
     from upstream, the packager SHOULD query upstream to include it.

From https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#License_Text
"Common licenses that require including their texts with all derivative works include ASL 2.0, EPL, BSD and MIT."

So open a upstream request and:
   wait for upstream new release
or add the LICENSE to the package sources

[!]: %check is present and all tests pass.

%check
phpunit tests

[!]: Rpmlint is run on all installed packages.
php-Analog.src: E: specfile-error error: Macro % has illegal name (%define)

	-%global %reldate 20121224
	+%global reldate 20121224

or drop this unused macro.

I agree with you for the pecl-mongo requirement, I'll drop it and write a line about it. I'll also drop unused reldate macro.

The version... Well, I guess I should not package anything after the few days following Xmas day :D

I've reported license issue upstream:
https://github.com/jbroadway/analog/issues/2

Thank you Remi to take that review :)

New version:
Spec URL: http://odysseus.x-tnd.be/fedora/php-Analog/php-Analog.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/php-Analog/php-Analog-1.0.0-1.git876d8a3bb.fc18.src.rpm

* version (1.0.0) => fixed
* requires (mongo) => fixed
* License (upstream request + file) => fixed

- Mongo - Save to MongoDB collection, requires php-pecl(mongo)
  package to be installed)

=> typo, remove trailing )

[!]: %check is present and all tests pass.

%check
phpunit tests

Ooops, typo fixed and %check added :)

I had an issue running tests, fix that has been included in specfile has been submitted upstream:
https://github.com/jbroadway/analog/issues/3

Here the new version:
Spec URL: http://odysseus.x-tnd.be/fedora/php-Analog/php-Analog.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/php-Analog/php-Analog-1.0.0-2.git876d8a3bb.fc18.src.rpm

* typo => fixed
* %check => fixed

All seems fine, no blocker.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4827045


=== APPROVED ===

Thank you for the review :)

New Package SCM Request
=======================
Package Name: php-Analog
Short Description: PHP micro logging package
Owners: trasher
Branches: f17 f18 el6
InitialCC:

Git done (by process-git-requests).

php-Analog-1.0.0-2.git876d8a3bb.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/php-Analog-1.0.0-2.git876d8a3bb.fc17

php-Analog-1.0.0-2.git876d8a3bb.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/php-Analog-1.0.0-2.git876d8a3bb.fc18

php-Analog-1.0.0-2.git876d8a3bb.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-Analog-1.0.0-2.git876d8a3bb.el6

php-Analog-1.0.0-2.git876d8a3bb.el6 has been pushed to the Fedora EPEL 6 testing repository.

php-Analog-1.0.0-2.git876d8a3bb.fc18 has been pushed to the Fedora 18 stable repository.

php-Analog-1.0.0-2.git876d8a3bb.fc17 has been pushed to the Fedora 17 stable repository.

php-Analog-1.0.0-2.git876d8a3bb.el6 has been pushed to the Fedora EPEL 6 stable repository.