Re: [PATCH v2] rollback lock files on more signals than just SIGINT
- From
Mike Ralphson <mike.ralphson@gmail.com>
- Date
- Jun 5, 2008, 08:02 UTC
- Message-ID
- <e2b179460806050102m1ca15b2fj61d32ca45e4770d4@mail.gmail.com>
- In-Reply-To
- <7vhcc9xg5j.fsf@gitster.siamese.dyndns.org>
2008/6/4 Junio C Hamano <gitster@pobox.com>:
Show 10 quoted lines
> "Mike Ralphson" <mike.ralphson@gmail.com> writes: > >> 2008/5/29 Paolo Bonzini <bonzini@gnu.org>: >> ... >> This addition to the testsuite breaks it on AIX with the default sh >> (ksh). Replacing the explicit sh -c with $SHELL_PATH -c fixes it for >> me (as I have SHELL_PATH pointing to bash). If that's acceptable I can >> post a patch if necessary. > > Like the attached one?
Yes, something like that 8-) Thanks for applying. Actually getting patches out from my work domain without the egregious disclaimer, and to get them to actually show up in the list is still a bit of a faff here.
> I noticed quite a many "sh" dependencies in other test scripts:
I'm happy to pick up a bit of janitor work on the tests after this release. I notice we have a few non-executable test scripts, and several 'duplicate' test numbers which might also want to be looked at?
Cheers, Mike