[PATCH] CTDB NFS Ganesha example call-out improvements
Martin Schwenke
martin at meltin.net
Fri Feb 24 00:56:42 UTC 2017
On Fri, 24 Feb 2017 11:48:29 +1100, Amitay Isaacs <amitay at gmail.com>
wrote:
> On Thu, Feb 23, 2017 at 4:37 PM, Martin Schwenke <martin at meltin.net> wrote:
>
> > I was adding some wiki documentation about how CTDB's NFS call-outs
> > work. I referenced nfs-ganesha-callout, so I decided to fix it up a
> > little, even though it is still just an example in the documentation
> > directory.
> >
> > Please review and maybe push...
> >
> >
> In the patch "ctdb-doc: Fix shellcheck warning in example NFS ganesha
> call-out"
>
> [ -d "$_dir" ] || continue # Handle no directories case
>
> Wouldn't the following make it clearer?
>
> [ ! -d "$_dir" ] && break # Handle no directories case
Yeah, let's go with that.
I was actually thinking of using an if-statement for clarity but decided
to keep it brief... either way... :-)
peace & happiness,
martin
More information about the samba-technical
mailing list