login
Header Space

 
 

[PATCH 2/3] t3404: use configured shell instead of /bin/sh

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Junio C Hamano <gitster@...>
Cc: Whit Armstrong <armstrong.whit@...>, <git@...>
Date: Wednesday, February 20, 2008 - 8:00 pm

The fake-editor shell script invoked /bin/sh; normally this
is fine, unless the /bin/sh doesn't meet our compatibility
requirements, as is the case with Solaris. Specifically, the
$() syntax used by fake-editor is not understood.

Signed-off-by: Jeff King <peff@peff.net>
---
 t/t3404-rebase-interactive.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh
index e5ed745..62e65d7 100755
--- a/t/t3404-rebase-interactive.sh
+++ b/t/t3404-rebase-interactive.sh
@@ -61,8 +61,8 @@ test_expect_success 'setup' '
 	git tag I
 '
 
-cat > fake-editor.sh <<\EOF
-#!/bin/sh
+echo "#!$SHELL" >fake-editor
+cat >> fake-editor.sh <<\EOF
 case "$1" in
 */COMMIT_EDITMSG)
 	test -z "$FAKE_COMMIT_MESSAGE" || echo "$FAKE_COMMIT_MESSAGE" > "$1"
-- 
1.5.4.2.247.g107bd

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/3] solaris test results, Jeff King, (Wed Feb 20, 7:59 pm)
Re: [PATCH 0/3] solaris test results, Junio C Hamano, (Wed Feb 20, 8:34 pm)
Re: [PATCH 0/3] solaris test results, Jeff King, (Wed Feb 20, 8:41 pm)
[PATCH 2/3] t3404: use configured shell instead of /bin/sh, Jeff King, (Wed Feb 20, 8:00 pm)
speck-geostationary