--- Begin Message ---
Subject: |
2.4.3: fix for loop |
Date: |
Thu, 30 Oct 2014 23:29:56 +0100 |
User-agent: |
KMail/1.13.7 (Linux/3.16.5; KDE/4.14.2; x86_64; ; ) |
Fix for loop in cmdline_wrap test.
--- libtool-2.4.3/tests/cmdline_wrap.at~ 2014-10-27 18:20:51.000000000
+0100
+++ libtool-2.4.3/tests/cmdline_wrap.at 2014-10-30 22:37:17.938397654 +0100
@@ -28,7 +28,7 @@
AT_SETUP([Run tests with low max_cmd_len])
AT_KEYWORDS([recursive expensive])
dnl If we already have failures, then reruns will fail too!
-fail_list=`for f in ?/fail ??/fail ???/fail ????/fail; do echo $f; end`
+fail_list=`for f in ?/fail ??/fail ???/fail ????/fail; do echo $f; done`
AT_CHECK([test -z "$fail_list" || (exit 77)])
m4_ifdef([AT_CAPTURE_FILE],
[AT_CAPTURE_FILE([testsuite.log])])
--
Arkadiusz Miśkiewicz, arekm / ( maven.pl | pld-linux.org )
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#18906: 2.4.3: fix for loop |
Date: |
Sun, 2 Nov 2014 17:48:36 +0000 |
Hi Arkadiusz,
> On Oct 30, 2014, at 10:29 PM, Arkadiusz Miśkiewicz <address@hidden> wrote:
>
>
> Fix for loop in cmdline_wrap test.
>
> --- libtool-2.4.3/tests/cmdline_wrap.at~ 2014-10-27 18:20:51.000000000
> +0100
> +++ libtool-2.4.3/tests/cmdline_wrap.at 2014-10-30 22:37:17.938397654
> +0100
> @@ -28,7 +28,7 @@
> AT_SETUP([Run tests with low max_cmd_len])
> AT_KEYWORDS([recursive expensive])
> dnl If we already have failures, then reruns will fail too!
> -fail_list=`for f in ?/fail ??/fail ???/fail ????/fail; do echo $f; end`
> +fail_list=`for f in ?/fail ??/fail ???/fail ????/fail; do echo $f; done`
> AT_CHECK([test -z "$fail_list" || (exit 77)])
> m4_ifdef([AT_CAPTURE_FILE],
> [AT_CAPTURE_FILE([testsuite.log])])
Nice catch! I was wondering why that wasn't working, and hadn't had the time to
look.
Applied and pushed.
Cheers,
--
Gary V. Vaughan (gary AT gnu DOT org)
--- End Message ---