Bug 607618 - Get rid of the annoying gefacl messages
Get rid of the annoying gefacl messages
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: beakerlib (Show other bugs)
14
All Linux
low Severity medium
: ---
: ---
Assigned To: Petr Šplíchal
Fedora Extras Quality Assurance
: Patch
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-24 09:43 EDT by Petr Šplíchal
Modified: 2016-05-31 21:39 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2012-07-25 09:53:32 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
trivial patch fixing getfacl & backup test bug (1.48 KB, patch)
2010-06-24 09:43 EDT, Petr Šplíchal
no flags Details | Diff

  None (edit)
Description Petr Šplíchal 2010-06-24 09:43:33 EDT
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?
Comment 1 Petr Šplíchal 2010-07-27 05:52:02 EDT
Anybody can review?
Comment 2 Jan Hutař 2010-07-27 06:09:59 EDT
Patch seems good and might be applied.
Comment 3 Bug Zapper 2010-07-30 08:13:10 EDT
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
Comment 4 Petr Šplíchal 2010-11-04 05:18:03 EDT
Thanks for the review, pushed to git:

http://git.fedorahosted.org/git/?p=beakerlib.git;a=commitdiff;h=f6522d6
Comment 5 Martin Cermak 2011-03-21 08:46:58 EDT
What about applying this patch to production beakerlib?
Comment 6 Petr Muller 2011-03-21 08:57:41 EDT
Should be present in latest beakerlib, I just did not updated the bugzilla.
Comment 8 Petr Muller 2012-07-25 09:53:32 EDT
Closing, this should be in for a long time.

Note You need to log in before you can comment on or make changes to this bug.