Show 43 quoted lines
> -----Original Message-----
> From: Eric Sunshine <sunshine@sunshineco.com>
> Sent: Monday, September 26, 2022 10:16 PM
> To: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> Cc: Eric DeCosta via GitGitGadget <gitgitgadget@gmail.com>; Git List
> <git@vger.kernel.org>; Jeff Hostetler <git@jeffhostetler.com>; Torsten
> Bögershausen <tboegi@web.de>; Ramsay Jones
> <ramsay@ramsayjones.plus.com>; Johannes Schindelin
> <Johannes.Schindelin@gmx.de>; Eric DeCosta <edecosta@mathworks.com>
> Subject: Re: [PATCH v12 6/6] fsmonitor: add documentation for allowRemote
> and socketDir options
>
> On Mon, Sep 26, 2022 at 11:15 AM Ævar Arnfjörð Bjarmason
> <avarab@gmail.com> wrote:
> > On Sat, Sep 24 2022, Eric DeCosta via GitGitGadget wrote:
> > > Add documentation for 'fsmonitor.allowRemote' and
> 'fsmonitor.socketDir'.
> > > Call-out experimental nature of 'fsmonitor.allowRemote' and limited
> > > file system support for 'fsmonitor.socketDir'.
> > >
> > > Signed-off-by: Eric DeCosta <edecosta@mathworks.com>
> > > ---
> > > -git-fsmonitor--daemon - A Built-in File System Monitor
> > > +git-fsmonitor--daemon - A Built-in Filesystem Monitor
> >
> > We have ~400 uses of "filesystem" in-tree, but ~100 for "file system".
> > I don't mind the change per-se, but this looks like an odd "while at it"
> > change.
> >
> > > A daemon to watch the working directory for file and directory
> > > -changes using platform-specific file system notification facilities.
> > > +changes using platform-specific filesystem notification facilities.
> >
> > More while-at-it...
>
> These changes may have been in response to my review[1], though I did say
> that such a change was outside the scope of this series. Perhaps I need to
> choose my wording more carefully?
>
> [1]: https://lore.kernel.org/git/CAPig+cQ5SMw+0Cwtw47LQM59-
> mQjJaOPe_LTybAC2j=3F9OywA@mail.gmail.com/ <https://protect-
> us.mimecast.com/s/STwOCG6A7DIYgAqYc7Wdth?domain=lore.kernel.org>
>
Eric DeCosta <edecosta@mathworks.com>