GNU bug report logs - #18906
2.4.3: fix for loop

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 18906 in the body.
You can then email your comments to 18906 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Arkadiusz Miśkiewicz <arekm <at> maven.pl>
To: bug-libtool <at> gnu.org
Subject: 2.4.3: fix for loop
Date: Thu, 30 Oct 2014 23:29:56 +0100

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 )

Message #8 received at 18906 <at> debbugs.gnu.org (full text, mbox):

From: Mike Frysinger <vapier <at> gentoo.org>
To: Arkadiusz Miśkiewicz <arekm <at> maven.pl>
Cc: 18906 <at> debbugs.gnu.org
Subject: Re: bug#18906: 2.4.3: fix for loop
Date: Fri, 31 Oct 2014 21:44:17 -0400

[Message part 1 (text/plain, inline)]

On 30 Oct 2014 23:29, Arkadiusz Miśkiewicz wrote:
> Fix for loop in cmdline_wrap test.

thanks, works for me !
-mike

[signature.asc (application/pgp-signature, inline)]

Message #13 received at 18906-close <at> debbugs.gnu.org (full text, mbox):

From: "Gary V. Vaughan" <gary <at> vaughan.pe>
To: Arkadiusz Miśkiewicz <arekm <at> maven.pl>
Cc: 18906-close <at> debbugs.gnu.org
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 <arekm <at> maven.pl> 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)

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.