GNU bug report logs - #47382
runtest doesn't work with Solaris 10 /bin/sh

Previous Next

Package: dejagnu;

Reported by: Rainer Orth <ro <at> CeBiTec.Uni-Bielefeld.DE>

Date: Thu, 25 Mar 2021 10:34:01 UTC

Owned by: jcb62281 <at> gmail.com

Severity: normal

Done: Jacob Bachmeyer <jcb62281 <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Jacob Bachmeyer <jcb62281 <at> gmail.com>
To: Rainer Orth <ro <at> CeBiTec.Uni-Bielefeld.DE>
Cc: 47382 <at> debbugs.gnu.org
Subject: bug#47382: runtest doesn't work with Solaris 10 /bin/sh
Date: Thu, 08 Apr 2021 22:39:57 -0500
Rainer Orth wrote:
> * There's also
>
> Native configuration is /vol/src/gnu/dejagnu/dejagnu-1.6.3-rc2/config.guess: syn
> tax error at line 35: `me=$' unexpected
>
>   another POSIX shell construct, and less easily avoided because this
>   comes from upstream.
>   

As of commit 2ca7d52dfa1726aadd1363e399d9b41429332017, which will be in 
release candidate 3, configure now contains experimental logic to 
address this issue by patching the interpreter line in config.guess and 
config.sub when SHELL is not /bin/sh.

>   I'd already noticed another instance of the problem after I'd sent the
>   original PR: while e.g. autoconf-generated configure and related in
>   gcc are very careful to invoke config.guess only prefixed with $SHELL,
>   runtest.exp exec's it directly, again falling into the trap of POSIX
>   shell constructs (also $(), maybe more).
>   

The "exec" used in this case is Tcl's exec(n), which simply invokes the 
installed copy of config.guess (via the system) with the interpreter 
indicated in the file.  The patch now applied by configure should 
correct this issue.

> However, there are more errors still:
>
> [...]
>
> Running /vol/src/gnu/dejagnu/dejagnu-1.6.3-rc2/testsuite/report-card.all/onetest
> .exp ...
> spawn /bin/sh -c cd /vol/gcc/obj/dejagnu/dejagnu-1.6.3-rc2/testsuite/report-card
> .all/onetest && exec /vol/src/gnu/dejagnu/dejagnu-1.6.3-rc2/dejagnu report-card^
> M
> expr: syntax error
> ERROR: could not resolve command dejagnu-report-card
>
> There are obviously more hardcoded uses of /bin/sh here and elsewhere,
> but I haven't looked for those yet.

If this issue remains in rc3, I repeat my request for execution traces 
under both /bin/sh and /bin/ksh for the rc3 version of the dejagnu 
launcher script.  Please attach the output of "/bin/ksh -x ./dejagnu 
report-card" and "/bin/sh -x ./dejagnu report-card" with both run in the 
DejaGnu source directory when you get the chance if there are still issues.

I have delayed extending the "shell swap" logic in configure to the 
dejagnu launcher script because I am currently uncertain whether it will 
help solve any problems.


-- Jacob





This bug report was last modified 3 years and 352 days ago.

Previous Next


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