Bug 1828338

Summary: man: make attr.1 man page generic
Product: Red Hat Enterprise Linux 8 Reporter: Achilles Gaikwad <agaikwad>
Component: attrAssignee: Kamil Dudka <kdudka>
Status: CLOSED DUPLICATE QA Contact: qe-baseos-daemons
Severity: low Docs Contact:
Priority: unspecified    
Version: 8.1CC: kdudka
Target Milestone: rc   
Target Release: 8.0   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-05-04 16:00:16 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Achilles Gaikwad 2020-04-27 14:52:06 UTC
Description of problem:

The attr(1) man page states it is specifically for XFS filesystem,
however the command works for other filesystems as well. Therefore, we
remove the XFS specific bits and pieces from the man page and make the
man page more generic.

Version-Release number of selected component (if applicable):
NA

How reproducible:
NA

Steps to Reproduce:
NA

Actual results:
Man page pretends that it is XFS specific.

Expected results:

Consider the following patch that removes all the XFS specific bits and makes the man page generic. Something similar should be used for fixing the man page of attr(1).

https://lists.nongnu.org/archive/html/acl-devel/2020-02/msg00002.html


Additional info:

I sent the patch to upstream but did not get any feedback. This is a request for a customer case, therefore raising a Bugzilla for the issue.

Comment 1 Kamil Dudka 2020-05-04 14:19:15 UTC
Could I merge this bug with bug #1828330 ?  I do not think we need a separate bug for each man page of a single component.

Would it be fine to use the Debian patch instead?

    https://git.hadrons.org/cgit/debian/pkgs/attr.git/tree/debian/patches/man-attr-no-xfs-specific.patch

I would prefer not to diverge with them if not necessary.  On the other hand, we do not have to wait for upstream to merge it first.

Comment 2 Achilles Gaikwad 2020-05-04 15:14:24 UTC
Hello Kamil,

Thanks for looking into this. I appreciate your help here. :)

Yes, you can merge this request with bug 1828330.

>Would it be fine to use the Debian patch instead?

And the Debian patch is better! Please use the Debian patch instead.

Please let me know if I can contribute to this in any way further. Clearing the needinfo for now.

-Achilles

Comment 3 Kamil Dudka 2020-05-04 16:00:16 UTC
Achilles, thank you for the quick reply on this!  I am merging the bugs...

*** This bug has been marked as a duplicate of bug 1828330 ***