Opened 9 years ago
Closed 9 years ago
#1080 closed task (wontfix)
Use GLIBC xattr for attr tools and acl tools and library
Reported by: | William Harrington | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | CLFS Standard 3.1.0 |
Component: | BOOK | Version: | CLFS Standard GIT |
Keywords: | Cc: | berzerkula@…, jonathan@…, chris@…, cross-lfs@… |
Description
Chris mentioned to use xattr.h from Glibc to build ACL. We don't need the ATTR package in the book.
Reference: http://lists.gnu.org/archive/html/acl-devel/2014-05/msg00001.html
I've edited some files and have attached a patch to apply to ACL.
Attachments (2)
Change History (11)
by , 9 years ago
Attachment: | acl-2.2.52-sys_xattr-1.patch added |
---|
comment:1 by , 9 years ago
Actually, we probably can't remove the ATTR package as we need the attr, getfattr, and setfattr programs and man-pages.
comment:2 by , 9 years ago
For ATTR, we can build the tools attr, getfattr, setfattr and install the man pages for them and skip installing the headers and libraries.
The tests will work fine. All 114 passed for me.
I've attached a patch for ATTR. This will make intallation of ATTR in all books much shorter and simpler. No need to install libraries. Just have one install of ATTR in each book.
comment:3 by , 9 years ago
Summary: | Remove ATTR package and use GLIBC sys/xattr.h → Use GLIBC xattr for attr tools and acl tools and library |
---|
by , 9 years ago
Attachment: | attr-2.4.47-sys_xattr-1.patch added |
---|
comment:4 by , 9 years ago
With my server build the following tools link to libattr and means that they may need edits to use <sys/xattr.h> and not attempt to link to libattr with -lattr.
/bin/install /bin/cp /bin/mv /lib/libacl.so.1.1.0 /lib/libcap.so.2.22 /usr/bin/msgfilter /usr/bin/getfattr /usr/bin/patch /usr/bin/msginit /usr/bin/bsdtar /usr/bin/chacl /usr/bin/xgettext /usr/bin/msgcomm /usr/bin/msgcmp /usr/bin/msgfmt /usr/bin/bsdcpio /usr/bin/msgcat /usr/bin/setfacl /usr/bin/getfacl /usr/bin/msgmerge /usr/bin/recode-sr-latin /usr/bin/msgunfmt /usr/bin/setfattr /usr/bin/attr /usr/bin/xorriso /usr/bin/msgen /usr/bin/msgexec /usr/bin/msggrep /usr/bin/msgconv /usr/bin/msgattrib /usr/bin/msguniq /usr/lib/python2.7/site-packages/samba/xattr_native.so /usr/lib/libgettextsrc-0.18.3.so /usr/lib/libarchive.so.13.1.2 /usr/lib/samba/libntvfs.so /usr/lib/samba/libsmbd_base.so /usr/lib/gettext/hostname /usr/lib/gettext/urlget /usr/lib/libgettextlib-0.18.3.so /usr/sbin/proftpd /usr/sbin/usermod /usr/sbin/useradd
comment:6 by , 9 years ago
Priority: | major → trivial |
---|
comment:7 by , 9 years ago
Shadow needs to be hacked for this. I don't think it is worth all of the effort to edit a bunch of source to use <sys/xattr.h> and change all the configure scripts which are looking for libattr.
comment:8 by , 9 years ago
Yes have to agree with your last comment, checked gentoo, arch, fedora, all use stand alone attr lib
comment:9 by , 9 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Allows acl to use xattr from libc