Re: [PATCH 3/3 FIXED] help (Windows): Display HTML in default browser using Windows' shell API
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jul 13, 2008, 08:58 UTC
- Message-ID
- <7v1w1yupn4.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <1215895277.487916ed43464@webmail.nextra.at>
Johannes Sixt <johannes.sixt@telecom.at> writes:
Show 16 quoted lines
> Zitat von Junio C Hamano <gitster@pobox.com>: > >> Steffen Prohaska <prohaska@zib.de> writes: >> >> >> Do you mean to have that printf() or is it a leftover debugging >> >> statement? >> > >> > I mean to have it. >> >> Ok, I was just checking. Unless other Windows users complain, will apply >> as-is. As you might guess, I am completely neutral on this one. > > I'm working on followups to this series, and it turns out to be more > convenient to have system_path() in exec_cmd.c instead of path.c. > It'll make sense if I resend the series with an updated version of 1/3 > (instead of a patch that merely moves the function around).
Ok, will drop these three patches and wait for replacement from yours to appear, and then we will see which ones to apply.