[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Fix cross compilation with wine for the case SHELL != bash.
From: |
Ralf Wildenhues |
Subject: |
Re: [PATCH] Fix cross compilation with wine for the case SHELL != bash. |
Date: |
Sun, 9 Nov 2008 20:51:19 +0100 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
Hello Alexei,
* Alexei Sheplyakov wrote on Sat, Nov 08, 2008 at 07:51:20PM CET:
> I've tried to cross compile the GMP (http://gmplib.org) library using libtool
> version 2.2.7a (from current git). The compilation went fine, however, the
> test
> suite failed due to GMP dll being not found:
And you are using a shell with an echo that interprets backslashes by
default. Don't do that; just use bash. You will find more issues in
Libtool. We sometimes (by accident I think) assume echo on w32 system
is well-behaved: all w32 shells are fixed this way now.
Nevertheless, I've applied your patch as below, and put you in THANKS.
Thanks!
Ralf
2008-11-09 Alexei Sheplyakov <address@hidden> (tiny change)
Fix cross compilation with wine for the case SHELL != bash.
* libltdl/config/ltmain.m4sh (func_to_host_path): Avoid escape
sequence interpretation by 'echo', for w32-style paths.
* THANKS: Update.
diff --git a/libltdl/config/ltmain.m4sh b/libltdl/config/ltmain.m4sh
index 696b6ab..005c6a8 100644
--- a/libltdl/config/ltmain.m4sh
+++ b/libltdl/config/ltmain.m4sh
@@ -2600,7 +2600,7 @@ func_to_host_path ()
;;
*cygwin* )
func_to_host_path_tmp1=`cygpath -w "$1"`
- func_to_host_path_result=`echo "$func_to_host_path_tmp1" |\
+ func_to_host_path_result=`$ECHO "$func_to_host_path_tmp1" |\
$SED -e "$lt_sed_naive_backslashify"`
;;
* )
@@ -2613,7 +2613,7 @@ func_to_host_path ()
# the odd construction:
func_to_host_path_tmp1=`winepath -w "$1" 2>/dev/null`
if test "$?" -eq 0 && test -n "${func_to_host_path_tmp1}"; then
- func_to_host_path_result=`echo "$func_to_host_path_tmp1" |\
+ func_to_host_path_result=`$ECHO "$func_to_host_path_tmp1" |\
$SED -e "$lt_sed_naive_backslashify"`
else
# Allow warning below.