Bug 607618
| Summary: | Get rid of the annoying gefacl messages | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Petr Šplíchal <psplicha> | ||||
| Component: | beakerlib | Assignee: | Petr Šplíchal <psplicha> | ||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | low | ||||||
| Version: | 14 | CC: | azelinka, jhutar, mcermak, ohudlick, pmuller | ||||
| Target Milestone: | --- | Keywords: | Patch | ||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2012-07-25 13:53:32 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: |
|
||||||
Anybody can review? Patch seems good and might be applied. This bug appears to have been reported against 'rawhide' during the Fedora 14 development cycle. Changing version to '14'. More information and reason for this action is here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping Thanks for the review, pushed to git: http://git.fedorahosted.org/git/?p=beakerlib.git;a=commitdiff;h=f6522d6 What about applying this patch to production beakerlib? Should be present in latest beakerlib, I just did not updated the bugzilla. Closing, this should be in for a long time. |
Created attachment 426576 [details] trivial patch fixing getfacl & backup test bug Description of problem: While running rlFileBackup() I see the following getfacl messages repeated in the output: getfacl: Removing leading '/' from absolute path names getfacl: Removing leading '/' from absolute path names getfacl: Removing leading '/' from absolute path names ... The list can be quite long if backing up deep paths. Solution for this is to use the -p option. A simple patch attached. Contains also a trivial fix for an old backup test bug: Usage: setfacl [-bkndRLP] { -m|-M|-x|-X ... } file ... Try `setfacl --help' for more information. Unit tests run, passed. OK to push?