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?
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.