[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] Re: Automake-1.7.5: tests/dejagnu7.test
From: |
Christian Cornelssen |
Subject: |
[PATCH] Re: Automake-1.7.5: tests/dejagnu7.test |
Date: |
Sun, 25 May 2003 17:50:43 +0200 (CEST) |
Hi,
On Sun, 25 May 2003, Alexandre Duret-Lutz wrote:
> Thanks for looking into this. Could you try the following command?
>
> runtest --status --version --unexistingflag
>
> I think it might be an easier way to check whether --status is supported
> without hard-coding any threshold.
>
> Here it simply displays the --version output, i.e.
>
> Expect version is 5.38.2
> Tcl version is 8.4
> Framework version is 1.4.3
>
> while I think you'll get a diagnostic about --unexistingflag.
>
Guessed right:
address@hidden:packages/BUILD$ runtest --status --version --unexistingflag
Illegal Argument "--unexistingflag"
try "runtest --help" for option list
address@hidden:packages/BUILD$ echo $?
1
But is it safe to expect future runtests to tolerate garbage after
--version? In general, --help and --version need only be recognized
when being the only argument.
Could you try "runtest --help | fgrep -e --status" instead?
I'd then suggest the following:
Index: ChangeLog
>From Christian Cornelssen <address@hidden>
* tests/dejagnu7.test: Check if runtest supports --status.
--- tests/dejagnu7.test.orig Fri Apr 18 11:17:30 2003
+++ tests/dejagnu7.test Sun May 25 17:44:44 2003
@@ -24,6 +24,9 @@
required=runtest
. ./defs || exit 1
+# Check whether DejaGNU supports --status
+runtest --help | $FGREP -e --status || exit 77
+
cat > failtcl << 'END'
#! /bin/sh
echo whatever
Regards,
Christian Cornelssen