On 06/18/2012 12:23 AM, Junio C Hamano wrote:
Torsten Bögershausen<tboegi@xxxxxx> writes:
PERL_PATH should be exported from test-lib.sh; we dot-source
GIT-BUILD-OPTIONS to get its value, but it is not propagated to the
Thanks, that works for me.
Do you squeeze some like that onto pu?
@@ -493,6 +493,7 @@ GIT_ATTR_NOSYSTEM=1
export PATH GIT_EXEC_PATH GIT_TEMPLATE_DIR GIT_CONFIG_NOSYSTEM GIT_ATTR_NOSYSTEM
. "$GIT_BUILD_DIR"/GIT-BUILD-OPTIONS
+export PERL_PATH
> I see other uses of bare "perl" that the patch forgot to convert,
> and the worst part is that one of these uses of bare "perl" appears
> in test-lib.sh _before_ it dot-sources GIT-BUILD-OPTIONS to pick up
> the correct version of Perl, so the above single liner cannot be the
> whole fix.
How do you like the following idea:
- rename t/test-lib.sh into t/test-lib.sh.sh
- Change perl into @@PERL_PATH@@ in this file
- Add the missing EXPORT PERL_PATH in test-lib.sh.sh
- Preprocess t/test-lib.sh from t/test-lib.sh.sh, as seen below
diff --git a/Makefile b/Makefile
index 0914133..765107a 100644
--- a/Makefile
+++ b/Makefile
@@ -442,6 +442,7 @@ SCRIPT_LIB += git-rebase--interactive
SCRIPT_LIB += git-rebase--merge
SCRIPT_LIB += git-sh-setup
SCRIPT_LIB += git-sh-i18n
+SCRIPT_LIB += t/test-lib.sh
SCRIPT_PERL += git-add--interactive.perl
SCRIPT_PERL += git-difftool.perl
@@ -2018,6 +2019,7 @@ sed -e '1s|#!.*/sh|#!$(SHELL_PATH_SQ)|' \
-e 's|@@LOCALEDIR@@|$(localedir_SQ)|g' \
-e 's/@@NO_CURL@@/$(NO_CURL)/g' \
-e 's/@@USE_GETTEXT_SCHEME@@/$(USE_GETTEXT_SCHEME)/g' \
+ -e 's|@@PERL_PATH@@|$(PERL_PATH)|g' \
-e $(BROKEN_PATH_FIX) \
$@.sh >$@+
endef
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html