Actual results: /var/cache/mock is declared 2775 in setup, then overridden in postinstall Expected results: /var/cache/mock should be 2775 everywhere Additional info: Please see patch attached.
Created attachment 505994 [details] Trivial fix
Created attachment 505999 [details] Additional fix Additionally, it does not make sense to set /var/cache/mock to 2775 upon %install because it's done in %files anyway. Needless duplication clutters the SPEC and makes it harder to maintain.
Thanks, I've added your patches (I combined the two into one git commit) for the next release.
Created attachment 506226 [details] Yet another bug in the SPEC, this time /var/{lib,cache}/permissions were not set correctly in the files section Hi! You are quite welcome; thanks for the prompt reaction. There seems to be yet another bug, however. The %attr macro as used in the SPEC only applies to *files*, not *directories*, thus the permissions were not being set correctly on the lib folder (not sure how important is that). This bug would surface with the cache directory if one day the fix from %post will be removed and it hit me already on a clean install. Thanks!
Additionally, as I noted in an e-mail, it would be nice to have the tarballs updated on the FedoraHosted page. I can do this for you, but I need an account :-/
The last patch (attachment 506226 [details]) is still relevant.
Ok, queued for the 1.1.15 release
mock-1.1.15-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/mock-1.1.15-1.fc15
mock-1.1.15-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/mock-1.1.15-1.el6
mock-1.0.22-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/mock-1.0.22-1.el5
mock-1.1.15-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/mock-1.1.15-1.fc14
Package mock-1.1.15-1.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 mock-1.1.15-1.el6' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/mock-1.1.15-1.el6 then log in and leave karma (feedback).
mock-1.1.15-1.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.1.15-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.0.22-1.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report.
mock-1.1.15-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.