t/lib-httpd: use write_script to copy CGI scripts
authorJeff King <peff@peff.net>
Thu, 22 May 2014 09:28:56 +0000 (05:28 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 23 May 2014 19:41:50 +0000 (12:41 -0700)
Using write_script will set our shebang line appropriately
with $SHELL_PATH. The script that is there now is quite
simple and likely to succeed even with a non-POSIX /bin/sh,
but it does not hurt to be defensive.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/lib-httpd.sh
t/lib-httpd/broken-smart-http.sh [changed mode: 0755->0644]
index 252cbf163bcff90aabe39a48219e00565f577ffe..8e680efbf04dace3a503a2e788732d840dcf73ef 100644 (file)
@@ -105,10 +105,14 @@ else
                "Could not identify web server at '$LIB_HTTPD_PATH'"
 fi
 
+install_script () {
+       write_script "$HTTPD_ROOT_PATH/$1" <"$TEST_PATH/$1"
+}
+
 prepare_httpd() {
        mkdir -p "$HTTPD_DOCUMENT_ROOT_PATH"
        cp "$TEST_PATH"/passwd "$HTTPD_ROOT_PATH"
-       cp "$TEST_PATH"/broken-smart-http.sh "$HTTPD_ROOT_PATH"
+       install_script broken-smart-http.sh
 
        ln -s "$LIB_HTTPD_MODULE_PATH" "$HTTPD_ROOT_PATH/modules"
 
old mode 100755 (executable)
new mode 100644 (file)
index f7ebfff..82cc610
@@ -1,4 +1,3 @@
-#!/bin/sh
 printf "Content-Type: text/%s\n" "html"
 echo
 printf "%s\n" "001e# service=git-upload-pack"