diff options
| author | Markus Heidelberg <markus.heidelberg@web.de> | 2009-01-20 00:43:26 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2009-01-19 22:39:31 -0800 |
| commit | 35641310afa04aaa0ba5f31be6bdb2cf23ec100f (patch) | |
| tree | b7c91041d8a6c9ba0bd625b25dbbb3aecd5fb193 /t/lib-httpd.sh | |
| parent | bc08fc4e850794c76da367bb628e508e6d41b1eb (diff) | |
| download | git-35641310afa04aaa0ba5f31be6bdb2cf23ec100f.tar.gz | |
use uppercase POSIX compliant signals for the 'trap' command
In 'man 1p trap' there is written:
"Implementations may permit names with the SIG prefix or ignore case
in signal names as an extension."
So change the lowercase signals to uppercase, which is POSIX compliant
instead of being an extension.
There wasn't anybody claiming that it doesn't work, but there was a bug
with using a signal with the SIG prefix, which is an extension as well.
So let's play it safe and change it, since it doesn't hurt anyone.
While at it, also convert 8 indentation spaces to 1 tab character.
Signed-off-by: Markus Heidelberg <markus.heidelberg@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/lib-httpd.sh')
| -rw-r--r-- | t/lib-httpd.sh | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/t/lib-httpd.sh b/t/lib-httpd.sh index 6ac312b905..3824020ca1 100644 --- a/t/lib-httpd.sh +++ b/t/lib-httpd.sh @@ -20,9 +20,9 @@ HTTPD_DOCUMENT_ROOT_PATH=$HTTPD_ROOT_PATH/www if ! test -x "$LIB_HTTPD_PATH" then - say "skipping test, no web server found at '$LIB_HTTPD_PATH'" - test_done - exit + say "skipping test, no web server found at '$LIB_HTTPD_PATH'" + test_done + exit fi HTTPD_VERSION=`$LIB_HTTPD_PATH -v | \ @@ -84,7 +84,7 @@ prepare_httpd() { start_httpd() { prepare_httpd - trap 'stop_httpd; die' exit + trap 'stop_httpd; die' EXIT "$LIB_HTTPD_PATH" -d "$HTTPD_ROOT_PATH" \ -f "$TEST_PATH/apache.conf" $HTTPD_PARA \ @@ -92,7 +92,7 @@ start_httpd() { } stop_httpd() { - trap 'die' exit + trap 'die' EXIT "$LIB_HTTPD_PATH" -d "$HTTPD_ROOT_PATH" \ -f "$TEST_PATH/apache.conf" -k stop |