Bug 715286
| Summary: | Fix inconsistent permissions fixing on /var/cache/mock in SPEC template | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Yury V. Zaytsev <yury> | ||||||||
| Component: | mock | Assignee: | Clark Williams <williams> | ||||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||||
| Severity: | low | Docs Contact: | |||||||||
| Priority: | unspecified | ||||||||||
| Version: | rawhide | CC: | mebrown, williams | ||||||||
| Target Milestone: | --- | ||||||||||
| Target Release: | --- | ||||||||||
| Hardware: | All | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | mock-1.1.15-1.el6 | Doc Type: | Bug Fix | ||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | Environment: | ||||||||||
| Last Closed: | 2011-10-02 23:01:15 UTC | Type: | --- | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | --- | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Yury V. Zaytsev
2011-06-22 12:41:10 UTC
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. |