This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Cleanup MIPS preconfigure script
- From: Steve Ellcey <sellcey at mips dot com>
- To: "Joseph S. Myers" <joseph at codesourcery dot com>
- Cc: Adhemerval Zanella <azanella at linux dot vnet dot ibm dot com>, <libc-alpha at sourceware dot org>
- Date: Tue, 9 Sep 2014 10:44:44 -0700
- Subject: Re: [PATCH] Cleanup MIPS preconfigure script
- Authentication-results: sourceware.org; auth=none
- References: <6b5ce132-2a69-4ee9-aaa5-d07792e72b50 at BAMAIL02 dot ba dot imgtec dot org> <alpine dot DEB dot 1 dot 10 dot 1409042322000 dot 27075 at tp dot orcam dot me dot uk> <1409871127 dot 2740 dot 76 dot camel at ubuntu-sellcey> <alpine dot DEB dot 1 dot 10 dot 1409050002390 dot 27075 at tp dot orcam dot me dot uk> <1409948236 dot 2740 dot 110 dot camel at ubuntu-sellcey> <Pine dot LNX dot 4 dot 64 dot 1409052152480 dot 20739 at digraph dot polyomino dot org dot uk> <1410279506 dot 2740 dot 158 dot camel at ubuntu-sellcey> <alpine dot DEB dot 1 dot 10 dot 1409091806340 dot 27075 at tp dot orcam dot me dot uk> <1410282834 dot 2740 dot 159 dot camel at ubuntu-sellcey> <540F3760 dot 5010203 at linux dot vnet dot ibm dot com> <Pine dot LNX dot 4 dot 64 dot 1409091723570 dot 15803 at digraph dot polyomino dot org dot uk>
On Tue, 2014-09-09 at 17:24 +0000, Joseph S. Myers wrote:
> On Tue, 9 Sep 2014, Adhemerval Zanella wrote:
>
> > And patch 0febba23ddabcd971be5259ee20236b9e3efa690 broke powerpc64 configure:
>
> Steve, you need to put preconfigure back inside a case statement so
> nothing runs for non-mips* machines.
>
Ack, I didn't realize that was necessary. Here is a patch to fix that,
should I go ahead and check it in? I also made the capitalization
changes that Maciej recommended.
Steve Ellcey
sellcey@mips.com
2014-09-09 Steve Ellcey <sellcey@mips.com>
* sysdeps/mips/preconfigure: Put code inside mips* case statement.
Fix capitalization of error message.
diff --git a/sysdeps/mips/preconfigure b/sysdeps/mips/preconfigure
index fb572d7..c118592 100644
--- a/sysdeps/mips/preconfigure
+++ b/sysdeps/mips/preconfigure
@@ -1,24 +1,29 @@
-abiflag=`$CC $CFLAGS $CPPFLAGS -E -dM -xc /dev/null | sed -n
's/^#define _MIPS_SIM \(.*\)/\1/p'`
-mips16flag=`$CC $CFLAGS $CPPFLAGS -E -dM -xc /dev/null | sed -n
's/^#define __mips16 \(.*\)/\1/p'`
+case "$machine" in
+mips*)
+ abiflag=`$CC $CFLAGS $CPPFLAGS -E -dM -xc /dev/null | sed -n
's/^#define _MIPS_SIM \(.*\)/\1/p'`
+ mips16flag=`$CC $CFLAGS $CPPFLAGS -E -dM -xc /dev/null | sed -n
's/^#define __mips16 \(.*\)/\1/p'`
-base_machine=mips
-if test "$abiflag" = "_ABIO32" ; then
- if test "$mips16flag" = "1" ; then
- machine=mips/mips32/mips16
+ base_machine=mips
+ if test "$abiflag" = "_ABIO32" ; then
+ if test "$mips16flag" = "1" ; then
+ machine=mips/mips32/mips16
+ else
+ machine=mips/mips32
+ fi
+ elif test "$abiflag" = "_ABIN32" ; then
+ machine=mips/mips64/n32
+ elif test "$abiflag" = "_ABI64" ; then
+ machine=mips/mips64/n64
else
- machine=mips/mips32
+ as_fn_error $? "Unable to determine ABI." "$LINENO" 5
fi
-elif test "$abiflag" = "_ABIN32" ; then
- machine=mips/mips64/n32
-elif test "$abiflag" = "_ABI64" ; then
- machine=mips/mips64/n64
-else
- as_fn_error $? "Unable to determine ABI." "$LINENO" 5
-fi
-# $config_machine is not really needed here but the slash after
$machine is
-# needed by the case statement in
sysdeps/unix/sysv/linux/mips/configure.ac.
-machine=$machine/$config_machine
+ # $config_machine is not really needed here but the slash after
+ # $machine is needed by the case statement in
+ # sysdeps/unix/sysv/linux/mips/configure.ac.
+ machine=$machine/$config_machine
-if test "$abiflag" != "_ABIO32" -a "$mips16flag" = "1"; then
- as_fn_error $? "mips16 is only supported with the O32 ABI." "$LINENO"
5
-fi
+ if test "$abiflag" != "_ABIO32" -a "$mips16flag" = "1"; then
+ as_fn_error $? "MIPS16 is only supported with the o32 ABI." "$LINENO"
5
+ fi
+ ;;
+esac