On Mon, Jun 02, 2014 at 12:05:29PM -0700, Junio C Hamano wrote: > Johannes Sixt <j.sixt@xxxxxxxxxxxxx> writes: > > >> Jeremiah Mahler (9): > >> compat/mingw.c: expand MinGW support for sigaction > >> connect.c: replace signal() with sigaction() > >> progress.c: replace signal() with sigaction() > >> write_or_die.c: replace signal() with sigaction() > >> daemon.c: replace signal() with sigaction() > >> builtin/log.c: replace signal() with sigaction() > >> builtin/merge-index.c: replace signal() with sigaction() > >> builtin/verify-tag.c: replace signal() with sigaction() > >> sigchain.c: replace signal() with sigaction() > > > > The series without patch 9/9 works on Windows so far. > > > > Without patch patch 9/9 and a more complete implementation of sigaction in > > compat/mingw.c the series misses its goal. But even if you complete it, it > > is IMHO only code churn without practical merits. > > Hmm, you sound a bit harsher than you usually do---although I > sort of share with you the doubt on the practical merits. > Alright, I'm dropping it. Too much work for no real gain other than some piece of mind. Thanks Johannes and Junio for your feedback. -- Jeremiah Mahler jmmahler@xxxxxxxxx http://github.com/jmahler -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html