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

Previous Next

Package: emacs;

Reported by: Po Lu <luangruo <at> yahoo.com>

Date: Tue, 25 Apr 2023 12:21:02 UTC

Severity: normal

Found in version 30.0.50

Done: Po Lu <luangruo <at> yahoo.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Po Lu <luangruo <at> yahoo.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#63067: closed (30.0.50; configure prints error with
 CONFIG_SHELL=/bin/sh)
Date: Thu, 27 Apr 2023 10:55:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Thu, 27 Apr 2023 18:54:43 +0800
with message-id <87cz3pbnnw.fsf <at> yahoo.com>
and subject line Re: bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
has caused the debbugs.gnu.org bug report #63067,
regarding 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
63067: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=63067
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
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 part 3 (message/rfc822, inline)]
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.


This bug report was last modified 2 years and 26 days ago.

Previous Next


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