From: Derrick Stolee Date: Wed, 17 Aug 2022 14:51:09 GMT Subject: Re: [PATCH v2 0/5] scalar: enable built-in FSMonitor Message-ID: <69be513b-b6c3-3a92-6152-fddc835a6723@github.com> In-Reply-To: On 8/16/2022 7:58 PM, Victoria Dye via GitGitGadget wrote: > This series enables the built-in FSMonitor [1] on 'scalar'-registered > repository enlistments. To avoid errors when unregistering an enlistment, > the FSMonitor daemon is explicitly stopped during 'scalar unregister'. I hadn't looked at this code in a while, so I poked around and asked some questions that might not even need answering. Outside of a nit involving a test prereq, this version looks fine to me. Thanks, -Stolee