Re: Tru64/OSF: BIN_SH revisited
Stepan Kasal [EMAIL PROTECTED] writes: These two changes would make the following patch; OK to commit? Yes, except you need to modify the Autoconf documentation accordingly.
Re: Tru64/OSF: BIN_SH revisited
Hello, 0a) Revert the change to set BIN_SH in Autoconf, [...] IMHO this is a very good idea. First, BIN_SH is set in macro AS_BOURNE_COMPATIBLE, but it doesn't belong there: this macro contains things like set -o posix, which i) have an immediate effect inside the current instance ii) are not inhereted in subsequent calls to the shell (Because of (ii), AS_BOURNE_COMPATIBLE is called from _AS_RUN.) Second, if this change 0b) Additionally, prepend /usr/bin/posix/ to the path walk of _AS_DETECT_BETTER_SHELL, [...] make things work on OSF, then the BIN_SH assignment becomes redundant cruft, right? We shall remove it now, before is is conserved by a release. (If the problem re-appears when 2.60 will be widely used, we can then add BIN_SH to the right place (the detect better shell stuff), and with a comment explaining the situation where it was necessary.) Does this sound convincing? Moreover, I think that the line DUALCASE=1; export DUALCASE doesn't belong to AS_BOURNE_COMPATIBLE either, the reason is the same as above; I think it shall go to AS_SHELL_SANITIZE. These two changes would make the following patch; OK to commit? Have a nice day, Stepan 2006-03-17 Stepan Kasal [EMAIL PROTECTED] * lib/m4sugar/m4sh.m4 (AS_BOURNE_COMPATIBLE): Removed BIN_SH=xpg4, moved DUALCASE=1 ... (AS_SHELL_SANITIZE): ... here. Index: lib/m4sugar/m4sh.m4 === RCS file: /cvsroot/autoconf/autoconf/lib/m4sugar/m4sh.m4,v retrieving revision 1.162 diff -u -r1.162 m4sh.m4 --- lib/m4sugar/m4sh.m4 16 Mar 2006 13:33:18 - 1.162 +++ lib/m4sugar/m4sh.m4 17 Mar 2006 15:20:12 - @@ -172,8 +172,6 @@ else case `(set -o) 2/dev/null` in *posix*) set -o posix;; esac fi -BIN_SH=xpg4; export BIN_SH # for Tru64 -DUALCASE=1; export DUALCASE # for MKS sh ]) @@ -320,6 +318,7 @@ ## - ## AS_BOURNE_COMPATIBLE +DUALCASE=1; export DUALCASE # for MKS sh # PATH needs CR _AS_CR_PREPARE
Re: Tru64/OSF: BIN_SH revisited
Hi Paul, Stepan, * Paul Eggert wrote on Fri, Mar 17, 2006 at 08:55:14AM CET: Ralf Wildenhues [EMAIL PROTECTED] writes: 0a) Revert the change to set BIN_SH in Autoconf, in order to be consistent. 0b) Additionally, prepend /usr/bin/posix/ to the path walk of _AS_DETECT_BETTER_SHELL, to choose the right shell right away, inside Autoconf. This sounds good to me. The first patch below does (0b), the second (3). I'm tending toward (0b), unsure whether to add (0a) for good measure. What do you think? (0b) is good. I see your point about (0a) but let's go ahead and try it, for Autoconf. Thanks. OK. I've applied (0b) only for now, and dropped the Libtool patch. Thanks for the review! * Stepan Kasal wrote on Fri, Mar 17, 2006 at 04:27:09PM CET: Second, if this change 0b) Additionally, prepend /usr/bin/posix/ to the path walk of _AS_DETECT_BETTER_SHELL, [...] make things work on OSF, then the BIN_SH assignment becomes redundant cruft, right? That depends on whether anything below (spawned by) configure ever calls `sh' unadorned. We shall remove it now, before is is conserved by a release. Without deciding over this change: iff you apply it, please also adjust autoconf.texi which currently states that BIN_SH is set. Moreover, I think that the line DUALCASE=1; export DUALCASE doesn't belong to AS_BOURNE_COMPATIBLE either, the reason is the same as above; I think it shall go to AS_SHELL_SANITIZE. FWIW, I'm don't know about this one either. Cheers, Ralf
Re: Tru64/OSF: BIN_SH revisited
Hello, On Fri, Mar 17, 2006 at 11:20:49AM -0800, Paul Eggert wrote: Yes, except you need to modify the Autoconf documentation accordingly. I checked in the version attached to this mail. Stepan 2006-03-17 Stepan Kasal [EMAIL PROTECTED] * lib/m4sugar/m4sh.m4 (AS_BOURNE_COMPATIBLE): Removed BIN_SH=xpg4, moved DUALCASE=1 ... (AS_SHELL_SANITIZE): ... here. * doc/autoconf.texi (Special Shell Variables) BIN_SH: Do not say that it is set. Index: lib/m4sugar/m4sh.m4 === RCS file: /cvsroot/autoconf/autoconf/lib/m4sugar/m4sh.m4,v retrieving revision 1.162 diff -u -r1.162 m4sh.m4 --- lib/m4sugar/m4sh.m4 16 Mar 2006 13:33:18 - 1.162 +++ lib/m4sugar/m4sh.m4 17 Mar 2006 15:20:12 - @@ -172,8 +172,6 @@ else case `(set -o) 2/dev/null` in *posix*) set -o posix;; esac fi -BIN_SH=xpg4; export BIN_SH # for Tru64 -DUALCASE=1; export DUALCASE # for MKS sh ]) @@ -320,6 +318,7 @@ ## - ## AS_BOURNE_COMPATIBLE +DUALCASE=1; export DUALCASE # for MKS sh # PATH needs CR _AS_CR_PREPARE Index: doc//autoconf.texi === RCS file: /cvsroot/autoconf/autoconf/doc/autoconf.texi,v retrieving revision 1.967 diff -u -r1.967 autoconf.texi --- doc/autoconf.texi 17 Mar 2006 03:59:05 - 1.967 +++ doc/autoconf.texi 17 Mar 2006 17:19:17 - @@ -11091,7 +11091,6 @@ @evindex BIN_SH In Tru64, if @env{BIN_SH} is set to @code{xpg4}, subsidiary invocations of the standard shell conform to Posix. -Autoconf-generated scripts export this variable when they start up. @item CDPATH @evindex CDPATH
Re: Tru64/OSF: BIN_SH revisited
Ralf Wildenhues [EMAIL PROTECTED] writes: 0a) Revert the change to set BIN_SH in Autoconf, in order to be consistent. 0b) Additionally, prepend /usr/bin/posix/ to the path walk of _AS_DETECT_BETTER_SHELL, to choose the right shell right away, inside Autoconf. This sounds good to me. The first patch below does (0b), the second (3). I'm tending toward (0b), unsure whether to add (0a) for good measure. What do you think? (0b) is good. I see your point about (0a) but let's go ahead and try it, for Autoconf. Thanks.