Re: [PATCH] file-has-acl: revert unintended change in behavior of ls -L

[Jim Meyering]

Kamil Dudka wrote:
> The commit 95f7c57 introduced an unintended change in behavior of ls -L.
> I am attaching a patch that restores the old behavior.  Thanks in advance
> for considering the patch!
>
> Kamil
>
> From 75836c03cb21d616591b11164b626556d9f26152 Mon Sep 17 00:00:00 2001
> From: Kamil Dudka <address@hidden>
> Date: Mon, 3 Oct 2011 12:17:22 +0200
> Subject: [PATCH] file-has-acl: revert unintended change in behavior of ls -L
>
> * lib/file-has-acl.c (acl_extended_file_wrap): A wrapper around
> acl_extended_file () that allows to call acl_extended_file_nofollow ()
> only if the function is available and the file is not a symbolic link.
> (file_has_acl): Remove code that caused problems.  Call
> acl_extended_file_wrap ().

Hi Kamil,

Thank you for the patch.
Can you describe how to make "ls -L" misbehave without this patch?
I.e., if there isn't already a test in coreutils to exercise this,
I'd like to add one.

Also, if you can reference a bugzilla number, that would be nice.
Looks like it's this one:

  https://bugzilla.redhat.com/720325