On Thu, Mar 29, 2012 at 01:31:09PM +0200, Frans Klaver wrote: > On Wed, Mar 28, 2012 at 11:57 PM, Jeff King <peff@xxxxxxxx> wrote: > > > +static int exists_in_PATH(const char *file) > [...] > > I expect that if more post-mortem checking is done, this function is > going to need a sibling that provides you with the first found entry > in PATH, so you can do more checks on it. It should be easy to write it that way. I'm not personally planning on adding more checks, but I think it's worth considering future additions. > One of the things I ran into while working on [1] is that quite some > errors that are produced can also be caused by the interpreter. Yeah, they can be confusing and hard to track down. I'll leave that topic out of this round, and you can build on it if you like. -Peff -- 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