[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] [mingw] fix cross-compile-with-wine case
From: |
Charles Wilson |
Subject: |
[PATCH] [mingw] fix cross-compile-with-wine case |
Date: |
Sat, 31 May 2008 13:11:58 -0400 |
* libltdl/config/ltmain.m4sh (func_to_host_path)
[host=mingw, build!=mingw|cygwin]: check that winepath
stdout is non-empty and exit code is 0, instead of checking
stderr.
Reported by: Roumen Petrov
---
Following the discussion here:
http://lists.gnu.org/archive/html/libtool-patches/2008-05/msg00087.html
and Roumen's tests, this seems like an obvious fix. Test suite
passes with no regressions on cygwin (not that this patch affects
cygwin, but it confirms that no silly syntax errors crept in).
Ok to push (preferably before any 2.2.6 release)?
Chuck
libltdl/config/ltmain.m4sh | 20 ++++++++++++--------
1 files changed, 12 insertions(+), 8 deletions(-)
diff --git a/libltdl/config/ltmain.m4sh b/libltdl/config/ltmain.m4sh
index 144bc76..44e36de 100644
--- a/libltdl/config/ltmain.m4sh
+++ b/libltdl/config/ltmain.m4sh
@@ -2555,17 +2555,21 @@ func_to_host_path ()
;;
* )
# Unfortunately, winepath does not exit with a non-zero
- # error code, so we are forced to check stderr for an
- # error message. On the other hand, if the command is not
- # found, the shell will set an exit code of 127. So we
- # must check for both, which explains the odd construction:
- func_to_host_path_winepath_stderr=`winepath -w "$1" >/dev/null
2>&1`
+ # error code, so we are forced to check the contents of
+ # stdout. On the other hand, if the command is not
+ # found, the shell will set an exit code of 127 and print
+ # *an error message* to stdout. So we must check for both
+ # error code of zero AND non-empty stdout, which explains
+ # the odd construction:
+ func_to_host_path_tmp1=`winepath -w "$1" 2>/dev/null`
func_to_host_path_winepath_exitcode=$?
if test "$func_to_host_path_winepath_exitcode" -eq 0 &&\
- test -z "${func_to_host_path_winepath_stderr}" ; then
- func_to_host_path_tmp1=`winepath -w "$1"`
- func_to_host_path_result=`echo "$func_to_host_path_tmp1" |\
+ test -n "${func_to_host_path_tmp1}" ; then
+ func_to_host_path_result=`echo "$func_to_host_path_tmp1"|\
$SED -e "$lt_sed_naive_backslashify"`
+ else
+ # Allow warning below.
+ func_to_host_path_result=""
fi
;;
esac
--
1.5.5.1
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [PATCH] [mingw] fix cross-compile-with-wine case,
Charles Wilson <=