[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: GNU Autoconf test version 2.59d available

From: Paul Eggert
Subject: Re: GNU Autoconf test version 2.59d available
Date: Wed, 07 Jun 2006 16:51:53 -0700
User-agent: Gnus/5.1008 (Gnus v5.10.8) Emacs/21.4 (gnu/linux)

Andreas B√ľning <address@hidden> writes:

> - The code that searches for the best shell (_AS_DETECT_BETTER_SHELL,
>   around line 234 in lib/m4sugar/m4sh.m4) doesn't use 
> $ac_executable_extensions,
>   so for systems which use .exe extensions no shell can be found here.
>   Please, correct this. If you like I'll write the necessary one or two lines
>   to patch this.

It might be more than one or two lines, I'm afraid.  However, thanks
for reporting it; I installed the following temporary workaround

By the way: on your host, does "test -x foo" succeed if foo.exe is

>   Btw, ac_executable_extension seems to be undocumented.

That's probably on purpose, as I expect that interface might change in
the future.

> I still had 2.59 installed in my $PATH

Yes, it's bad juju to try to mix autoconf versions in the same build.
That's always been true, but I expect people don't remember it since
it's been so long since the last release.  Maybe a NEWS warning is
in order....

Anyway, thanks again.

2006-06-07  Paul Eggert  <address@hidden>

        * lib/m4sugar/m4sh.m4 (_AS_DETECT_BETTER_SHELL): Look for
        $as_shell.exe too.  Problem reported by Andreas Buening in

--- lib/m4sugar/m4sh.m4 31 May 2006 09:44:39 -0000      1.190
+++ lib/m4sugar/m4sh.m4 7 Jun 2006 23:46:47 -0000       1.191
@@ -160,6 +160,11 @@ $2
 # --------------------
 # Try to be as Bourne and/or POSIX as possible.
+# FIXME: The assignment to BIN_SH is dubious; see
+# <>.
+# It might be better to remove it, but first please see
+# <>.
 [# Be Bourne compatible
 if test -n "${ZSH_VERSION+set}" && (emulate sh) >/dev/null 2>&1; then
@@ -217,6 +222,18 @@ m4_expand_once([m4_append([_AS_DETECT_SU
 # -----------------------
 # The real workhorse for detecting a shell with the correct
 # features.
+# FIXME: The '/usr/bin/posix' below works around a shell bug in OSF
+# <>
+# but this causes a regression on OpenServer 6.0.0
+# <>
+# The code should test for the OSF bug directly rather than look at
+# /usr/bin/posix here.
+# FIXME: The 'test -f "$as_shell.exe"' works around a problem in OS/2
+# <>
+# but we should replace the two test -f calls with a single AS_EXECUTABLE_P.
 [m4_wrap([m4_divert_text([M4SH-SANITIZE], [
@@ -237,8 +254,8 @@ if test "x$CONFIG_SHELL" = x; then
       for as_shell in $as_candidate_shells $SHELL; do
-        # Try only shells which exist, to save several forks.
-        AS_IF([test -f "$as_shell" &&
+        # Try only shells that exist, to save several forks.
+        AS_IF([{ test -f "$as_shell" || test -f "$as_shell.exe"; } &&
                         [("$as_shell") 2> /dev/null])],
@@ -1000,6 +1017,7 @@ esac[]dnl
 # ----------------
 # Find out ahead of time whether ``test -x'' can be used to distinguish
 # executables from other regular files.
+# FIXME: This should use "test -x /"; that's much faster.
 [# Find out whether ``test -x'' works.  Don't use a zero-byte file, as
 # systems may use methods other than mode bits to determine executability.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]