libgo: better error messages for unknown GOARCH/GOOS

	PR go/113530

Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/557655

diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE
index 18281c6cd1e98f56da7bbb2450cd36339a864aa4..429904a2b8f9354f2da67cbe19320133c9138f44 100644 (file)
--- a/gcc/go/gofrontend/MERGE
+++ b/gcc/go/gofrontend/MERGE
@@ -1,4 +1,4 @@
-7ab229670f7dad1d79f33929f9a4f8e6e4a71526
+8c056e335cecec67d1d223a329b7ba4dac778a65
 
 The first line of this file holds the git revision number of the last
 merge done from the gofrontend repository.

diff --git a/libgo/Makefile.am b/libgo/Makefile.am
index c95dc2106cddec5c8a68db74399f3de38f94619f..3eccadbac67bfd62962a4c86db04ba2df297ea5b 100644 (file)
--- a/libgo/Makefile.am
+++ b/libgo/Makefile.am
@@ -497,6 +497,10 @@ s-version: Makefile
 zgoarch.go: s-zgoarch; @true
 s-zgoarch: Makefile goarch.sh
        rm -f zgoarch.go.tmp
+       if ! $(SHELL) $(srcdir)/goarch.sh $(GOARCH) family >/dev/null 2>/dev/null;  then \
+         $(SHELL) $(srcdir)/goarch.sh $(GOARCH) family; \
+         exit 1; \
+       fi
        echo "package goarch" > zgoarch.go.tmp
        echo >> zgoarch.go.tmp
        echo 'const GOARCH = "'$(GOARCH)'"' >> zgoarch.go.tmp

diff --git a/libgo/Makefile.in b/libgo/Makefile.in
index 8dcb6d6a35485f8731948afeffd174301f524b77..0a2b7ee55301015cb4a2e968ef8b34a40e0b6475 100644 (file)
--- a/libgo/Makefile.in
+++ b/libgo/Makefile.in
@@ -2716,6 +2716,10 @@ s-version: Makefile
 zgoarch.go: s-zgoarch; @true
 s-zgoarch: Makefile goarch.sh
        rm -f zgoarch.go.tmp
+       if ! $(SHELL) $(srcdir)/goarch.sh $(GOARCH) family >/dev/null 2>/dev/null;  then \
+         $(SHELL) $(srcdir)/goarch.sh $(GOARCH) family; \
+         exit 1; \
+       fi
        echo "package goarch" > zgoarch.go.tmp
        echo >> zgoarch.go.tmp
        echo 'const GOARCH = "'$(GOARCH)'"' >> zgoarch.go.tmp

diff --git a/libgo/configure b/libgo/configure
index db371bc76f107c3794a3349b7646d04983212aab..20bff2bd6beca1c18a0a2ae43388285ffbedf83e 100755 (executable)
--- a/libgo/configure
+++ b/libgo/configure
@@ -14107,6 +14107,10 @@ fi
 
 
 
+if test "${GOOS}" = "unknown"; then
+   as_fn_error $? "\"could not determine GOOS from ${host}\"" "$LINENO" 5
+fi
+
 USE_DEJAGNU=no
 case ${host} in
   *-*-rtems*) USE_DEJAGNU=yes ;;
@@ -14339,6 +14343,10 @@ esac
 
 
 
+if test "${GOARCH}" = "unknown"; then
+   as_fn_error $? "\"could not determine GOARCH from ${host}\"" "$LINENO" 5
+fi
+
  if test "$GOARCH" = "386" -o "$GOARCH" = "amd64" -o "$GOARCH" = "amd64p32"; then
   LIBGO_IS_X86_TRUE=
   LIBGO_IS_X86_FALSE='#'

diff --git a/libgo/configure.ac b/libgo/configure.ac
index e8d66f8415d9110720878c768a93a61ec08baf96..22158ac7f5d28f33a9407ef4de45f7ce172a9175 100644 (file)
--- a/libgo/configure.ac
+++ b/libgo/configure.ac
@@ -209,6 +209,10 @@ AM_CONDITIONAL(LIBGO_IS_BSD, test $is_darwin = yes -o $is_dragonfly = yes -o $is
 AC_SUBST(GOOS)
 AC_SUBST(ALLGOOS)
 
+if test "${GOOS}" = "unknown"; then
+   AC_MSG_ERROR("could not determine GOOS from ${host}")
+fi
+
 dnl Test whether we need to use DejaGNU or whether we can use the
 dnl simpler gotest approach.  We can only use gotest for a native
 dnl build.
@@ -376,6 +380,10 @@ AC_SUBST(GOARCH)
 AC_SUBST(ALLGOARCH)
 AC_SUBST(ALLGOARCHFAMILY)
 
+if test "${GOARCH}" = "unknown"; then
+   AC_MSG_ERROR("could not determine GOARCH from ${host}")
+fi
+
 AM_CONDITIONAL(LIBGO_IS_X86, test "$GOARCH" = "386" -o "$GOARCH" = "amd64" -o "$GOARCH" = "amd64p32")
 
 FUNCTION_DESCRIPTORS=false