Re: [PATCH 3/3] git-instaweb: Don't assume Apache executable is named apache2

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 30, 2010, 16:03 UTC
Message-ID: <7v39w48q4a.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1277865900-25044-3-git-send-email-dpmcgee@gmail.com>

Dan McGee <dpmcgee@gmail.com> writes:

> On Arch Linux, we keep the original 'httpd' name for the exectuable and
> don't rename it or the path to the modules.

Sorry, but I cannot parse the last 6 words here.

Show 8 quoted lines

> Remove some of these assumptions
> and add the httpd name in where it may be required.
> ...
>  resolve_full_httpd () {
>  	case "$httpd" in
> -	*apache2*|*lighttpd*)
> +	*apache2*|*lighttpd*|*httpd*)
> +		# yes, *httpd* covers *lighttpd* above, but it is there for clarity

That may make things work better on Arch but I wonder if there are platforms that allow users to point at their favorite, non-Apache, http server with a canonical name "httpd" that gets broken by this change. Am I being too paranoid?

Other two patches looked very sane.  Thanks.