From: Stefano Lattarini Date: Tue, 10 Jan 2012 19:27:08 +0000 (+0100) Subject: tests: work around MinGW/MSYS issue in fd redirections X-Git-Tag: ng-0.5a~19^2~8 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=53339f86ceb32ff98b1ccfb05539645cb7e641fe;p=thirdparty%2Fautomake.git tests: work around MinGW/MSYS issue in fd redirections Some checks on $(AM_TESTS_FD_REDIRECT) were failing on MSYS, likely because system calls like "write(9, ...)" simply doesn't work for MinGW-compiled programs. Similar usages work for the shell scripts though, since /bin/sh is an MSYS program and thus is a lot more POSIX-ish than most MinGW-compiled programs. The best fix for this issue is to separate the checks using shell scripts as dummy test cases from the checks using compiled programs for the same purpose, and skip these latter checks on MinGW. This change fixes automake bug#10466. Report by Peter Rosin. * tests/parallel-tests-fd-redirect.test: Move checks using compiled C programs as test cases to ... * tests/parallel-tests-fd-redirect-exeext.test: ... this new test. * tests/list-of-tests.mk: Update. --- diff --git a/tests/list-of-tests.mk b/tests/list-of-tests.mk index 31d70a76b..8bc0ef05f 100644 --- a/tests/list-of-tests.mk +++ b/tests/list-of-tests.mk @@ -702,6 +702,7 @@ parallel-tests-log-compiler-2.test \ parallel-tests-dry-run-1.test \ parallel-tests-dry-run-2.test \ parallel-tests-fd-redirect.test \ +parallel-tests-fd-redirect-exeext.test \ parallel-tests-extra-programs.test \ parallel-tests-unreadable.test \ parallel-tests-subdir.test \ diff --git a/tests/parallel-tests-fd-redirect-exeext.test b/tests/parallel-tests-fd-redirect-exeext.test new file mode 100755 index 000000000..0f9d6da33 --- /dev/null +++ b/tests/parallel-tests-fd-redirect-exeext.test @@ -0,0 +1,114 @@ +#! /bin/sh +# Copyright (C) 2011, 2012 Free Software Foundation, Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2, or (at your option) +# any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# parallel-tests support: redirection of file descriptors with +# AM_TESTS_FD_REDIRECT, for tests which are binary executables +# We use some tricks to ensure that all code paths in `lib/am/check2.am' +# are covered, even on platforms where $(EXEEXT) would be naturally empty. +# See also the more generic test 'check-fd-redirect.test', and +# sister test 'parallel-tests-fd-redirect.test'. + +required='cc native' +am_parallel_tests=yes +. ./defs || Exit 1 + +cat >> configure.in << 'END' +AC_PROG_CC +# Calls like "write(9, ...)" are unlikely to work for MinGW-compiled +# programs. We must skip this test if this is the case. +am__ok=no +AC_LINK_IFELSE( + [AC_LANG_PROGRAM([[#include ]], + [[write (9, "foobar\n", 7); return 0;]])], + [AM_RUN_LOG([./conftest$EXEEXT 9>&1]) \ + && AM_RUN_LOG([(./conftest$EXEEXT 9>&1) | grep "^foobar"]) \ + && am__ok=yes]) +test $am__ok = yes || AS_EXIT([63]) +AM_CONDITIONAL([real_EXEEXT], [test -n "$EXEEXT"]) +test -n "$EXEEXT" || EXEEXT=.bin +AC_OUTPUT +END + +cat > Makefile.am << 'END' +AM_TESTS_FD_REDIRECT = 9>&1 +TESTS = $(check_PROGRAMS) +check_PROGRAMS = baz qux.test +qux_test_SOURCES = zardoz.c + +## Sanity check. +if !real_EXEEXT +check-local: + test -f baz.bin + test -f qux.test.bin +endif +END + +$ACLOCAL +$AUTOCONF +$AUTOMAKE -a + +cat > baz.c <<'END' +#include +#include +int main (void) +{ + ssize_t res = write (9, " bazbazbaz\n", 11); + if (res < 0) + perror("write failed"); + return res != 11; +} +END + +cat > zardoz.c <<'END' +#include +#include +int main (void) +{ + ssize_t res = write (9, " quxquxqux\n", 11); + if (res < 0) + perror("write failed"); + return res != 11; +} +END + +st=0; ./configure || st=$? +cat config.log # For debugging, as we do tricky checks in configure. +if test $st -eq 63; then + skip_ "fd redirect in compiled program unsupported" +elif test $st -eq 0; then + : Continue. +else + fatal_ "unexpected error in ./configure" +fi + +# Sanity checks. +st=0 +grep '^baz\.log:.*baz\$(EXEEXT)' Makefile || st=1 +grep '^\.test\$(EXEEXT)\.log:' Makefile || st=1 +grep '^qux\.log:' Makefile && st=1 +test $st -eq 0 || fatal_ "doesn't cover expected code paths" + +st=0 +$MAKE check >stdout || st=1 +cat stdout +cat baz.log +cat qux.log +test $st -eq 0 +grep "^ bazbazbaz$" stdout +grep "^ quxquxqux$" stdout +$EGREP '(bazbazbaz|quxquxqux)' *.log && Exit 1 + +: diff --git a/tests/parallel-tests-fd-redirect.test b/tests/parallel-tests-fd-redirect.test index 8b1637896..f22b31a70 100755 --- a/tests/parallel-tests-fd-redirect.test +++ b/tests/parallel-tests-fd-redirect.test @@ -15,36 +15,22 @@ # along with this program. If not, see . # parallel-tests support: redirection of file descriptors with -# AM_TESTS_FD_REDIRECT, even when using tests without suffix, or -# which are binary executables. -# We use some tricks to ensure that all code paths in `lib/am/check2.am' -# are covered, even on platforms where $(EXEEXT) would be naturally empty. +# AM_TESTS_FD_REDIRECT, even when using tests without suffix. +# The sister `parallel-tests-fd-redirect-exeext.test' do a similar +# check for tests that are binary executables. # See also the more generic test 'check-fd-redirect.test'. -required='cc native' am_parallel_tests=yes . ./defs || Exit 1 cat >> configure.in << 'END' -AC_PROG_CC -AM_CONDITIONAL([real_EXEEXT], [test -n "$EXEEXT"]) -test -n "$EXEEXT" || EXEEXT=.bin AC_OUTPUT END cat > Makefile.am << 'END' AM_TESTS_FD_REDIRECT = 9>&1 TEST_EXTENSIONS = .test .sh -TESTS = foo.sh bar $(check_PROGRAMS) -check_PROGRAMS = baz qux.test -qux_test_SOURCES = zardoz.c - -## Sanity check. -if !real_EXEEXT -check-local: - test -f baz.bin - test -f qux.test.bin -endif +TESTS = foo.sh bar END $ACLOCAL @@ -64,52 +50,21 @@ END chmod a+x foo.sh bar -cat > baz.c <<'END' -#include -#include -int main (void) -{ - ssize_t res = write (9, " bazbazbaz\n", 11); - if (res < 0) - perror("write failed"); - return res != 11; -} -END - -cat > zardoz.c <<'END' -#include -#include -int main (void) -{ - ssize_t res = write (9, " quxquxqux\n", 11); - if (res < 0) - perror("write failed"); - return res != 11; -} -END - ./configure # Sanity checks. st=0 grep '^bar\.log:.*bar' Makefile || st=1 -grep '^baz\.log:.*baz\$(EXEEXT)' Makefile || st=1 -grep '^\.test\$(EXEEXT)\.log:' Makefile || st=1 -$EGREP '^(foo|qux)\.log:' Makefile && st=1 +grep '^foo\.log:' Makefile && st=1 test $st -eq 0 || fatal_ "doesn't cover expected code paths" -st=0 -$MAKE check >stdout || st=1 +st=0; $MAKE check >stdout || st=1 cat stdout cat foo.log cat bar.log -cat baz.log -cat qux.log test $st -eq 0 grep "^ foofoofoo$" stdout grep "^ barbarbar$" stdout -grep "^ bazbazbaz$" stdout -grep "^ quxquxqux$" stdout -$EGREP '(foofoofoo|barbarbar|bazbazbaz|quxquxqux)' *.log && Exit 1 +$EGREP '(foofoofoo|barbarbar)' *.log && Exit 1 :