GNU bug report logs - #63067
30.0.50; configure prints error with CONFIG_SHELL=/bin/sh

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 63067 in the body.
You can then email your comments to 63067 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
Date: Tue, 25 Apr 2023 20:19:45 +0800

On a Solaris 10 system, run:

  % CONFIG_SHELL=/bin/sh ./configure CC="/opt/SUNwspro/bin/cc -m64"

this will result in several errors, which I've now fixed.  However,
configure then complains:

checking for variable-length arrays... yes
./configure: !: not found
./configure: !: not found
./configure: !: not found
./configure: !: not found
./configure: !: not found
./configure: !: not found

which seems to be harmless, but is probably a bug somewhere in gnulib.
The Autoconf manual says:

     Using ‘!’ is not portable.  Instead of:

          if ! cmp -s file file.new; then
            mv file.new file
          fi

     use:

          if cmp -s file file.new; then :; else
            mv file.new file
          fi

so this should not appear in configure:

  func_gl_gnulib_m4code_260941c0e5dc67ec9e87d1fb321c300b ()
  {
    if ! $gl_gnulib_enabled_260941c0e5dc67ec9e87d1fb321c300b; then

      gl_gnulib_enabled_260941c0e5dc67ec9e87d1fb321c300b=true
      if case $host_os in mingw*) false;; *) :;; esac; then
        func_gl_gnulib_m4code_open
      fi
    fi
  }
  func_gl_gnulib_m4code_cloexec ()
  {
    if ! $gl_gnulib_enabled_cloexec; then

      gl_gnulib_enabled_cloexec=true
    fi
  }
  func_gl_gnulib_m4code_dirfd ()
  {
    if ! $gl_gnulib_enabled_dirfd; then

Message #8 received at 63067 <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Po Lu <luangruo <at> yahoo.com>
Cc: 63067 <at> debbugs.gnu.org
Subject: bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
Date: Tue, 25 Apr 2023 19:52:26 -0700

> On a Solaris 10 system, run:
> 
>   % CONFIG_SHELL=/bin/sh ./configure CC="/opt/SUNwspro/bin/cc -m64"
> 
> this will result in several errors, which I've now fixed.

My suggestion is to not worry about /bin/sh there. I.e., just run:

  ./configure CC="/opt/SUNwspro/bin/cc -m64"

and let 'configure' re-exec itself with a better shell. Or, if you'd 
rather pick the shell yourself, choose /bin/bash or /bin/ksh or 
/usr/xpg4/bin/sh instead of /bin/sh. The idea is to not bother testing 
with Solaris 10 /bin/sh as there's not much point nowadays to forcing 
execution of that ancient non-POSIX shell when configuring Emacs.

Message #13 received at 63067-done <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 63067-done <at> debbugs.gnu.org
Subject: Re: bug#63067: 30.0.50; configure prints error with
 CONFIG_SHELL=/bin/sh
Date: Thu, 27 Apr 2023 18:54:43 +0800

Paul Eggert <eggert <at> cs.ucla.edu> writes:

> My suggestion is to not worry about /bin/sh there. I.e., just run:
>
>   ./configure CC="/opt/SUNwspro/bin/cc -m64"
>
> and let 'configure' re-exec itself with a better shell. Or, if you'd
> rather pick the shell yourself, choose /bin/bash or /bin/ksh or
> /usr/xpg4/bin/sh instead of /bin/sh. The idea is to not bother testing
> with Solaris 10 /bin/sh as there's not much point nowadays to forcing
> execution of that ancient non-POSIX shell when configuring Emacs.

OK, but I still find it nicer for configure to work with any shell
supported by Autoconf.  So please don't object if I continue to fix
problems in configure as they appear.

Since this will be fixed the next time Gnulib is merged, I'm closing
this bug.

Previous Next

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