logo       

[PATCH] Re: Automake-1.7.5: tests/dejagnu7.test: msg#00034

sysutils.automake.bugs

Subject: [PATCH] Re: Automake-1.7.5: tests/dejagnu7.test

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:

chris@Dachs:packages/BUILD$ runtest --status --version --unexistingflag

Illegal Argument "--unexistingflag"
try "runtest --help" for option list
chris@Dachs: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 <ccorn@xxxxxxxxxxxxxxx>

* 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






<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise