On Tue, Jul 22, 2008 at 03:14:12AM -0400, Jeff King wrote: > static struct child_process pager_process = { > .argv = pager_argv, > - .in = -1 > + .in = -1, > +#ifndef __MINGW32__ > + .preexec_cb = pager_preexec, > +#endif I couldn't recall if this initializer style is portable enough for us. It was already there wrapped in ifdefs, but perhaps it was only ok because the mingw version always uses the same compiler? -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