On Sun, Sep 16 2018, Duy Nguyen wrote:
> On Fri, Sep 14, 2018 at 11:56 PM Junio C Hamano <[email protected]> wrote: >> * jn/gc-auto (2018-07-17) 3 commits >> - gc: do not return error for prior errors in daemonized mode >> - gc: exit with status 128 on failure >> - gc: improve handling of errors reading gc.log >> >> "gc --auto" ended up calling exit(-1) upon error, which has been >> corrected to use exit(1). Also the error reporting behaviour when >> daemonized has been updated to exit with zero status when stopping >> due to a previously discovered error (which implies there is no >> point running gc to improve the situation); we used to exit with >> failure in such a case. >> >> What's the donness of this one? >> cf. <[email protected]> > > This topic has stayed in 'pu' for a long time. I thought it was > concluded that this was a good change? Jeff, Jonathan? There's still outstanding feedback on it, including my: https://public-inbox.org/git/[email protected]/ I think it would be great to merge [12]/3 down, and have a re-submission of 3/3 stand-alone, once that's merged down, so we can untangle obviously correct bugfixes ([12]/3) with meaningful changes in exit code behavior (3/3).
