Can I get a review for this?
https://bugs.openjdk.java.net/browse/JDK-8068927
http://cr.openjdk.java.net/~dlong/8068927/webrev/
thanks,
dl
On 1/13/2015 10:56 AM, Dean Long wrote:
On 1/13/2015 1:08 AM, Andrew Haley wrote:
On 13/01/15 08:44, Dean Long wrote:
I came up with a simpler version, where I replace "aarch64-" with
"arm-", run autoconf-config.sub, then replace "arm-" back to
"aarch64-".
Thanks. That sounds good to me.
Andrew.
Here's the patch. If it looks good, I can file a bug and push it to
the staging repo.
dl
diff -r b052cb38b985 common/autoconf/build-aux/config.sub
--- a/common/autoconf/build-aux/config.sub Thu Dec 11 15:05:06
2014 -0800
+++ b/common/autoconf/build-aux/config.sub Tue Jan 13 13:57:23
2015 -0500
@@ -41,25 +41,8 @@
case $1 in
-- ) # Stop option processing
shift; break ;;
- aarch64-gnu )
- sub_args="$sub_args aarch64-unknown-gnu"
- shift; ;;
- aarch64-linux )
- sub_args="$sub_args aarch64-unknown-linux-gnu"
- shift; ;;
- aarch64-*-linux )
- os=`echo $1 | sed 's/aarch64-\(.*\)-linux/\1/'`
- config="aarch64-unknown-linux-gnu"
- sub_args="$sub_args $config"
- shift; ;;
- aarch64-*-gnu )
- os=`echo $1 | sed 's/aarch64-\(.*\)-gnu.*$/\1/'`
- config="aarch64-unknown-gnu"
- sub_args="$sub_args $config"
- shift; ;;
- aarch64-*-linux-* )
- os=`echo $1 | sed 's/aarch64-\(.*\)-linux-.*$/'`
- config="aarch64-unknown-linux-gnu"
+ aarch64-* )
+ config=`echo $1 | sed 's/^aarch64-/arm-/'`
sub_args="$sub_args $config"
shift; ;;
- ) # Use stdin as input.
@@ -74,9 +57,7 @@
result=`. $DIR/autoconf-config.sub $sub_args "$@"`
exitcode=$?
-if [ "x$os" != "x" ] ; then
- result=`echo $result | sed "s/-unknown-/-$os-/"`
-fi
+result=`echo $result | sed "s/^arm-/aarch64-/"`
echo $result
exit $exitcode
