Hi Peff, On Wed, 25 Jan 2017, Jeff King wrote: > On Wed, Jan 25, 2017 at 05:58:42PM +0100, Johannes Schindelin wrote: > > > - if (access(path.buf, X_OK) < 0) > > + if (access(path.buf, X_OK) < 0) { > > +#ifdef STRIP_EXTENSION > > + strbuf_addstr(&path, ".exe"); > > I think STRIP_EXTENSION is a string. Should this line be: > > strbuf_addstr(&path, STRIP_EXTENSION); Yep. v2 coming, Johannes