config.sub breaks configuration on Solaris 10

[Bruno Haible]

Hi,

This is a show-stopper: The current config.sub (since 2018-05-06) prevents
packages from being configured on Solaris 10.

I am testing the current grep snapshot
https://meyering.net/grep/grep-3.1.48-7eea.tar.xz
on Solaris 10/sparc, and the configuration fails like this:

 --------------------------------------------------------------------
checking for a BSD-compatible install... /opt/csw/bin/ginstall -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /opt/csw/bin/gmkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether make supports nested variables... (cached) yes
configure: error: cannot run /bin/sh ../build-aux/config.sub
 --------------------------------------------------------------------

When I look into the configure script, I see that it is running
  /bin/sh ../build-aux/config.sub sun4
and sees whether this succeeds or fails.

And indeed it fails:

 --------------------------------------------------------------------
$ /bin/sh ../build-aux/config.sub sun4
../build-aux/config.sub: -r: is not an identifier
 --------------------------------------------------------------------

The same thing with traces shows where the problem occurs:

 --------------------------------------------------------------------
$ /bin/sh -x ../build-aux/config.sub sun4
timestamp=2018-11-28
+ sed -e s,.*/,, 
+ echo ../build-aux/config.sub 
me=config.sub
usage=Usage: ../build-aux/config.sub [OPTION] CPU-MFR-OPSYS or ALIAS

Canonicalize a configuration name.

Options:
  -h, --help         print this help, then exit
  -t, --time-stamp   print date of last modification, then exit
  -v, --version      print version number, then exit

Report bugs and patches to <address@hidden>.
version=GNU config.sub (2018-11-28)

Copyright 1992-2018 Free Software Foundation, Inc.

This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
help=
Try `config.sub --help' for more information.
+ test 1 -gt 0 
+ break 
IFS=-
+ read -r field1 field2 field3 field4 
sun4
../build-aux/config.sub: -r: is not an identifier
 --------------------------------------------------------------------

The same issue has also been seen by other people:
http://mail-index.netbsd.org/pkgsrc-users/2018/10/05/msg027524.html

The attached patch fixes it for me.

Here's the ChangeLog entry, if you need one:


2018-12-16  Bruno Haible  <address@hidden>

        Fix fatal error on Solaris 10 with /bin/sh.
        * config.sub (read_raw): New variable.
        Use it in all 'read' commands.

config-sub-solaris-fix.diff
Description: Text Data