On Sun, May 15, 2022 at 9:54 AM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Carlo Arenas <carenas@xxxxxxxxx> writes: > > > On Thu, May 12, 2022 at 6:20 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > >> > >> Heh. I am a bit surprised that double sudo would become a separate > >> prerequisite, > > > > It is because it goes away in the optional patch 4, since it won't be > > needed anymore after that. > > Hmph, it may not be needed, but it should still work, in which case > it probably is still worth testing, even with the optional patch #4. Just because it works, it doesn't mean we have to test it. IMHO tests are better reserved for things that might not work or that should work, so this gets in the same bucket than "triple sudo" would go, and interestingly enough we could add that test without having to add a new prerequisite too! > No? Your call, my assumption was (since this patch is part of this series, albeit optional), that the "double sudo" need will be short lived and therefore better not to have this test to begin with, or remove it as soon as the need is gone, which in practice would be the time between when this series (without the optional patch) is released, and the time that optional path gets released (maybe as part of another series, since it might be dropped from this one). alternatively we can make it not optional, and then the test will NEVER be needed. Carlo
