From: Jeff Hostetler Date: Fri, 11 Mar 2022 21:01:56 GMT Subject: Re: [PATCH v6 16/30] compat/fsmonitor/fsm-listen-darwin: add MacOS header files for FSEvent Message-ID: <0e28579e-4ba8-eb7e-b979-f9e93b7c1e51@jeffhostetler.com> In-Reply-To: On 3/9/22 1:57 PM, Junio C Hamano wrote: > Jeff Hostetler writes: > >> On 3/7/22 5:37 AM, Ævar Arnfjörð Bjarmason wrote: >>> On Tue, Mar 01 2022, Jeff Hostetler via GitGitGadget wrote: >>> >>>> From: Jeff Hostetler >>>> [...] >> [...] >> >> Ævar sent feedback (thanks!) on 8 commits in the V6 version >> on March 7. I started responding to each as I got to them >> in my inbox yesterday, but I'd like to take a break from >> responding individually to each of them inside of Part 2. >> >> Since most of the feedback is for "general cleanup" and since >> Part 2 V6 is already in "next", I'd like to revisit these >> issues with a few "cleanup" commits on top of Part 3 (which >> is still in active review), rather than re-rolling or >> appending "fixup" commits onto Part 2. > > Sounds good. Prepending "preliminary clean-up" before part 3 would > be even cleaner, I would suspect. > > In any case, let's consider part 2 "more or less done" unless we see > a glaring mistake that requires us to revert and redo it from > scratch. > > Thanks. > The cleanup here turned into 16 small commits. I'll send them as a "Part 2.5" so that they stand alone and can either be appended to part 2 or treated as a new branch. GGG wouldn't let me send fixup! commits, so inside of each commit message is a "fixup! ..." line which you can use if you want to squash them into part 2. But otherwise they have a normal (non-fixup) commit summary. After I send that I'll send a new version of part 3 that builds upon them. Thanks Jeff