On Tue, May 06, 2014 at 10:23:42PM +0000, Mario Domenech Goulart wrote: > Hi Michele, > > Many thanks. Attached is your patch signed-off (tested on Linux).
Hi guys, Thanks for your efforts. Unfortunately, this patch breaks the mingw-msys build. The msys tools all require a "faux Unix" path, like /c/mingw rather than a Windows-style path like c:/mingw or c:\mingw. We define TEST_DIR as `pwd -W`, which prints out a Windows-style path because this value is passed to programs, which use them in system calls (which are not emulated by mingw, so must be native paths). Could you try the attached patch instead? I've tested it on Debian (with zsh and csh), OpenIndiana, Haiku, Cygwin and Mingw-msys. What shell is Solaris really using? I would like to make my OpenIndiana mimick the "real" Solaris as much as possible (I'd install Solaris but I didn't manage to get it installed in a VM. And downloading it is a royal pain, too). I checked, but "sh" in OpenIndiana is ksh, which also knows about ${PWD}. Even csh seems to support ${PWD}. So it appears it's not exactly a bashism, but rather an extremely common extension to POSIX sh. So knowing exactly what Solaris uses for a shell would be very helpful. Cheers, Peter -- http://www.more-magic.net
>From 765eff9d0bce2601897f374934f3acd6e3a5fa51 Mon Sep 17 00:00:00 2001 From: Michele La Monaca <mikele.chic...@lamonaca.net> Date: Tue, 6 May 2014 13:17:57 +0200 Subject: [PATCH] fix private repository tests for Solaris Signed-off-by: Mario Domenech Goulart <mario.goul...@gmail.com> Signed-off-by: Peter Bex <peter....@xs4all.nl> --- tests/runtests.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/runtests.sh b/tests/runtests.sh index d2ffe72..5007f8a 100755 --- a/tests/runtests.sh +++ b/tests/runtests.sh @@ -398,10 +398,11 @@ $compile -e embedded3.c embedded4.scm echo "======================================== private repository test ..." mkdir -p tmp $compile private-repository-test.scm -private-repository -o tmp/xxx -tmp/xxx $PWD/tmp -PATH=$PWD/tmp:$PATH xxx $PWD/tmp +tmp/xxx ${TEST_DIR}/tmp +# This MUST be `pwd`: ${PWD} is not portable, and ${TEST_DIR} breaks mingw-msys +PATH=`pwd`/tmp:$PATH xxx ${TEST_DIR}/tmp # this may crash, if the PATH contains a non-matching libchicken.dll on Windows: -#PATH=$PATH:$PWD/tmp xxx $PWD/tmp +#PATH=$PATH:${TEST_DIR}/tmp xxx ${TEST_DIR}/tmp rm -fr rev-app rev-app-2 reverser/*.import.* reverser/*.so echo "======================================== reinstall tests" -- 1.7.10.4
_______________________________________________ Chicken-users mailing list Chicken-users@nongnu.org https://lists.nongnu.org/mailman/listinfo/chicken-users