Junio C Hamano <gitster@xxxxxxxxx> writes: > Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > >> This v3 is all Johannes's patches. The outstanding review on v2 could >> be clarified with a commit message change, which I've addressed, and >> v2 conflicted with a cache.h change that's since landed in "master", >> which I've rebased this on. >> >> Junio: We're getting closer to the release so it would be great to >> have this. It's been broken for a long time, but having this finaly >> fixed in v2.22 would be great. The functional code changes are also >> isolated to the fsmonitor code path, which reduces the risk and makes >> this easier to review/reason about. > > Thanks. With this many topic backlog in flight (and not yet in > 'pu'), a resend like this, which is much easier to handle than mere > reference to the previous discussion thread, is really appreciated. Hmph, surprisingly, 1/2 alone did not reproduce any breakage. ... goes and looks ... Ah, that's false alarm. As I use prove, "make test" treats known breakage as a non-event and does not show, which made me lose a few minutes scratching my head wondering what I did wrong. It would have been nicer if it were easy to use 1/2 alone with the new test marked to expect success, but with a patch split into two like this, it requires going in to the test script and changing s/_failure/_success/, which is a bit suboptimial. Anyway, will queue this to 'pu' and hopefully we can merge it down soonish. Thanks.