unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
       [not found] <87ildkcfxa.fsf.ref@yahoo.com>
@ 2023-04-25 12:19 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-04-26  2:52   ` Paul Eggert
  0 siblings, 1 reply; 3+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-04-25 12:19 UTC (permalink / raw)
  To: 63067

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






^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
  2023-04-25 12:19 ` bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-04-26  2:52   ` Paul Eggert
  2023-04-27 10:54     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Eggert @ 2023-04-26  2:52 UTC (permalink / raw)
  To: Po Lu; +Cc: 63067

> 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.





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh
  2023-04-26  2:52   ` Paul Eggert
@ 2023-04-27 10:54     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 0 replies; 3+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-04-27 10:54 UTC (permalink / raw)
  To: Paul Eggert; +Cc: 63067-done

Paul Eggert <eggert@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.





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-04-27 10:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <87ildkcfxa.fsf.ref@yahoo.com>
2023-04-25 12:19 ` bug#63067: 30.0.50; configure prints error with CONFIG_SHELL=/bin/sh Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-04-26  2:52   ` Paul Eggert
2023-04-27 10:54     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).