Bug 489106 – app-arch/tar-1.27-r1[acl]: Disabled support for ACL

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Arfrever Frehtes Taifersar Arahesis 2013-10-23 08:02:47 UTC]

configure.ac of app-arch/tar-1.27-r1 has a bug, which causes that when --with-posix-acls or --without-posix-acls is passed, then support for ACL is disabled.

$ cd /tmp
$ touch test_file
$ tar -cf archive.tar --acls test_file
tar: POSIX ACL support is not available
$

configure.ac contains:

AC_ARG_WITH([posix-acls],
    AS_HELP_STRING([--without-posix-acls],
                   [do not use POSIX.1e access control lists]),
    [with_posix_acls=no])
if test "x$with_posix_acls" != "xno"; then
...

www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/External-Software.html
"Macro: AC_ARG_WITH (package, help-string, [action-if-given], [action-if-not-given])"

configure.ac sets with_posix_acls=no as action-if-given instead of action-if-not-given.

[Comment 1 Lars Wendler (Polynomial-C) (RETIRED) 2013-10-23 10:38:57 UTC]

Created attachment 361708 [details, diff]
tar-1.27-acl_configure_fix.patch

Possible fix...

[Comment 2 Lars Wendler (Polynomial-C) (RETIRED) 2013-10-23 10:45:22 UTC]

Created attachment 361710 [details, diff]
tar-1.27-r2.ebuild.diff

ebuild patch. The reason I am posting this is because of the 

  EPATCH_OPTS="-Z"

variable I am using to keep timestamps of the modified files the same like the unmodified files. The rason I did this was because otherwise tar build system would switch to maintainer mode and re-run autotool by itself which we cannot do in an @system package.

I don't know if this is a feasible fix so comments are more than welcome.

[Comment 3 Lars Wendler (Polynomial-C) (RETIRED) 2013-10-23 13:42:48 UTC]

Sent patch to upstream...

[Comment 4 Lars Wendler (Polynomial-C) (RETIRED) 2013-10-24 06:45:27 UTC]

+*tar-1.27-r2 (24 Oct 2013)
+
+  24 Oct 2013; Lars Wendler <[e-mail]> -tar-1.27-r1.ebuild,
+  +tar-1.27-r2.ebuild, +files/tar-1.27-acl_configure_fix.patch:
+  Fixed "acl" USE flag reported by Arfrever Frehtes Taifersar Arahesis in bug
+  #489106. Moved dependency on sys-apps/attr from RDEPEND to DEPEND (bug
+  #489170).
+

Upstream accepted the fix and tar-1.27-r2 contains the patch I attached to this bug.