Do not use 'mock --shell' for non-interactive scripts as it will stop working in feature.
Additonally passing mock --shell foo --someopt will evaluate --someopt as argument to mock. And unless you want shell expansion it is better to use --chroot.
So you should either
mock --chroot 'foo --someopt'
mock --chroot -- foo --someopt
And btw current code does not work with mock, so I'm raising Priority
Created attachment 980402 [details]
Proposed patch to resolve this issue
I have applied the patch to upstream (thanks!). Had to fix a few other test cases failing but that's all set so I'll have a look at updated packages.
fedora-review-0.5.2-2.fc20 has been submitted as an update for Fedora 20.
fedora-review-0.5.2-2.fc21 has been submitted as an update for Fedora 21.
fedora-review-0.5.2-2.el6 has been submitted as an update for Fedora EPEL 6.
* should fix your issue,
* was pushed to the Fedora 21 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing fedora-review-0.5.2-2.fc21'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).
fedora-review-0.5.2-2.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report.
*** Bug 1185672 has been marked as a duplicate of this bug. ***
fedora-review-0.5.2-2.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report.