Due to project name conflict with phpci (PHP Continuous Integration) the command "phpci" have been renamed to "phpcompatinfo" The update (version 2.17.0) in on the road to F >= 17 and EPEL-6. Can you please update the /usr/share/fedora-review/scripts/php-phpci.sh. (probably you can raise the dependency on 2.17.0) Sorry for that.
Notice : RPM of version 2.17.0 will provides both "phpcompatinfo" and "phpci" (a symlink). So, as this requests still valid, it is not urgent ;)
This message is a reminder that Fedora 17 is nearing its end of life. Approximately 4 (four) weeks from now Fedora will stop maintaining and issuing updates for Fedora 17. It is Fedora's policy to close all bug reports from releases that are no longer maintained. At that time this bug will be closed as WONTFIX if it remains open with a Fedora 'version' of '17'. Package Maintainer: If you wish for this bug to remain open because you plan to fix it in a currently maintained version, simply change the 'version' to a later Fedora version prior to Fedora 17's end of life. Bug Reporter: Thank you for reporting this issue and we are sorry that we may not be able to fix it before Fedora 17 is end of life. If you would still like to see this bug fixed and are able to reproduce it against a later version of Fedora, you are encouraged change the 'version' to a later Fedora version prior to Fedora 17's end of life. Although we aim to fix as many bugs as possible during every release's lifetime, sometimes those efforts are overtaken by events. Often a more recent Fedora release includes newer upstream software that fixes bugs or makes them obsolete.
We should take the opportunity to rename the plugin the the much better fedora-review-plugin-phpcompatinfo while fixing this.
Perhaps it will be even simpler to drop the subpackage and provides the /usr/share/fedora-review/scripts/php-phpci.sh in the main package. If phpcompatinfo command is missing, the reviewer will have a message about the need to install this command for an "exhaustive" review. While he currently need to know that the sub-package exists If you really want to keep the sub-package, I will propose to simply use fedora-review-plugin-php even if it currently only provides the phpcompatinfo stuff, it could be improved later, with more php stuff.
(In reply to Remi Collet from comment #4) > Perhaps it will be even simpler to drop the subpackage and provides the > /usr/share/fedora-review/scripts/php-phpci.sh in the main package. > > If phpcompatinfo command is missing, the reviewer will have a message about > the need to install this command for an "exhaustive" review. > While I actually have considered this problem, this simple solution didn't pop up in my head. Thanks! This is definitely the way to go.
Fixed in devel, https://fedorahosted.org/FedoraReview/changeset/0543b88e49a5f5382c8b651166de9a43cfdf6bce. Of course any feedback, especially no early in the 0.5.1 cycle is more than welcome!
fedora-review-0.5.0-2.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/fedora-review-0.5.0-2.fc18
fedora-review-0.5.0-2.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/fedora-review-0.5.0-2.el6
fedora-review-0.5.0-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/fedora-review-0.5.0-2.fc19
Package fedora-review-0.5.0-2.el6: * should fix your issue, * was pushed to the Fedora EPEL 6 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=epel-testing fedora-review-0.5.0-2.el6' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-EPEL-2013-11828/fedora-review-0.5.0-2.el6 then log in and leave karma (feedback).
Remi wrt your comment on the update: I know the package uses "phpci" command, however since the symlink exists this is OK. Next release of f-r will use phpcompatinfo due to upstream commit. I was just a bit lazy (I'd have to generate/apply the patch :-D). Is there a plan to remove phpci symlink during lifetime of F18? I guess this turned into "remove phpci subpackage" update.
> Is there a plan to remove phpci ... Yes. I plan to remove it, but only in f >= 20, so I'm waiting for this fix to be able to do it
fedora-review-0.5.0-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/fedora-review-0.5.0-3.el6
fedora-review-0.5.0-3.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/fedora-review-0.5.0-3.fc19
fedora-review-0.5.0-3.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/fedora-review-0.5.0-3.fc18
fedora-review-0.5.0-3.fc18 has been pushed to the Fedora 18 stable repository. If problems still persist, please make note of it in this bug report.
fedora-review-0.5.0-3.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report.
fedora-review-0.5.0-3.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.
fedora-review-0.5.1-1.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/fedora-review-0.5.1-1.fc18
fedora-review-0.5.1-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/fedora-review-0.5.1-1.el6
fedora-review-0.5.1-1.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/fedora-review-0.5.1-1.fc19
fedora-review-0.5.1-1.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report.
fedora-review-0.5.1-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.
fedora-review-0.5.1-1.fc18 has been pushed to the Fedora 18 stable repository. If problems still persist, please make note of it in this bug report.
fedora-review-0.5.1-1.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report.