From: "J. Bruce Fields" <bfields@fieldses.org>
To: Kenneth Dsouza <kdsouza@redhat.com>
Cc: linux-nfs@vger.kernel.org
Subject: Re: [PATCH] Add support for -R option in nfs4_getfacl so we can list the ACLs of all files and directories recursively.
Date: Wed, 21 Feb 2018 19:34:34 -0500 [thread overview]
Message-ID: <20180222003434.GD6535@fieldses.org> (raw)
In-Reply-To: <20180221225548.GC6535@fieldses.org>
On Wed, Feb 21, 2018 at 05:55:48PM -0500, J. Bruce Fields wrote:
> People have been request recursive operation and I keep not getting
> around to it, thanks for doing this.
>
> How are you testing this?
Also, on top of this patch (which I've committed, thanks) it might also
be nice to:
- add -L and -P options to choose whether you want to follow
symlinks, as "getfacl" does. I think this is just a matter of
passing the right flags to nftw.
- switch to using getopt instead of parsing the flags by hand
(looks like nfs4_setfacl already does this).
--b.
next prev parent reply other threads:[~2018-02-22 0:34 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-07 21:30 [PATCH] Add support for -R option in nfs4_getfacl so we can list the ACLs of all files and directories recursively Kenneth Dsouza
2018-02-21 22:55 ` J. Bruce Fields
2018-02-22 0:34 ` J. Bruce Fields [this message]
2018-02-22 17:42 ` Kenneth Dsouza
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180222003434.GD6535@fieldses.org \
--to=bfields@fieldses.org \
--cc=kdsouza@redhat.com \
--cc=linux-nfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.