Re: svn commit: r22476 - in branches/1.4.x: . subversion/tests/cmdline

[dlr@tigris.org]
> --- branches/1.4.x/subversion/tests/cmdline/davautocheck.sh (original)
> +++ branches/1.4.x/subversion/tests/cmdline/davautocheck.sh Tue Nov 28 08:59:29 2006
> @@ -99,6 +99,15 @@
> # dont assume sbin is in the PATH
> PATH="/usr/sbin:/usr/local/sbin:$PATH"
>
> +# Remove any proxy environmental variables that effect wget or curl.
> +# We don't need a proxy to connect to localhost and having the proxy
> +# environmental variables set breaks the Apache configuration file
> +# test below, since wget or curl will ask the proxy to connect to
> +# localhost.
> +export -n PROXY
> +export -n http_proxy
> +export -n HTTPS_PROXY

Hmmm. I didn't notice this when it hit trunk.

I know you don't care all that much whether "#!/bin/sh" actually means
"#!/bin/sh", but this introduces a bashism that could easily be
avoided:

  unset PROXY
  unset http_proxy
  unset HTTPS_PROXY

This was tested on five non-bash shells (ksh88, ksh93, zsh, dash,
posh), none of which recognize "export -n".