Re: [PATCH] Makefile: add clean-obsolete-scripts target

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jan 10, 2007, 13:42 UTC
Message-ID: <Pine.LNX.4.63.0701101441360.22628@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To: <81b0412b0701100453t63b62867ne67819e91f1319b9@mail.gmail.com>

Hi,

On Wed, 10 Jan 2007, Alex Riesen wrote:

Show 11 quoted lines

> On 1/10/07, Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
> > 
> > On Cygwin, newly builtins are not recognized, because there exist both
> > the executable binaries (with .exe extension) _and_ the now-obsolete
> > scripts (without extension), but the script is executed.
> > 
> > "make clean-obsolete-scripts" removes these ambiguities by removing the
> > older of these file pairs.
> 
> Not enough. You have to cleanup the build directory too,
> otherwise tests fail, and git-hacking on git itself is ... complicated.

Doesn't it handle that?

	ls *$X '$(DESTDIR_SQ)$(gitexecdir_SQ)'/*$X

Note the first "*$X". It really should pick up on all executables in the build directory. Or do I miss something?

Ciao, Dscho