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 <config-patches@gnu.org>.
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 <br...@clisp.org>
Fix fatal error on Solaris 10 with /bin/sh.
* config.sub (read_raw): New variable.
Use it in all 'read' commands.
diff --git a/build-aux/config.sub b/build-aux/config.sub
index 4670805..fd224f3 100755
--- a/build-aux/config.sub
+++ b/build-aux/config.sub
@@ -110,8 +110,16 @@ case $# in
exit 1;;
esac
+# Determine whether 'read' supports the '-r' flag.
+# /bin/sh on Solaris 10 does not.
+if (echo x | read -r f) 2>/dev/null; then
+ read_raw='-r'
+else
+ read_raw=
+fi
+
# Split fields of configuration type
-IFS="-" read -r field1 field2 field3 field4 <<EOF
+IFS="-" read $read_raw field1 field2 field3 field4 <<EOF
$1
EOF
@@ -918,7 +926,7 @@ case $basic_machine in
;;
*-*)
- IFS="-" read -r cpu vendor <<EOF
+ IFS="-" read $read_raw cpu vendor <<EOF
$basic_machine
EOF
;;
_______________________________________________
config-patches mailing list
config-patches@gnu.org
https://lists.gnu.org/mailman/listinfo/config-patches
