Re: Latest builtin-commit series

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Sep 18, 2007, 22:24 UTC
Message-ID: <Pine.LNX.4.64.0709182324230.28395@racer.site>
In-Reply-To: <20070918213903.GA14488@steel.home>

Hi,

On Tue, 18 Sep 2007, Alex Riesen wrote:

Show 8 quoted lines

> Kristian H?gsberg, Tue, Sep 18, 2007 17:23:29 +0200:
> >       * Set the test suite default editor to '/bin/true' instead of ':'.
> >         Since we're not exec'ing the editor from shell anymore, ':'
> >         won't work.  Maybe we should special case ':' in launch_editor
> >         or perhaps make launch_editor use system(3).  Not sure.
> 
> Special case "" (empty string)? MinGW may have problems with
> /bin/true, any future exotic ports notwithstanding (OS/2, anyone?).

No problem.  At least in msysGit.

Ciao, Dscho