This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Sync config.[guess|sub] from FSF GCC mainline


On Mon, 23 May 2016, Nick Clifton wrote:

>  I am checking in the attached patch to sync our top level config.guess 
>  and config.sub files with their FSF GCC counterparts.

 This part looks like a regression to me, breaking character translation 
intended (single-argument `tr' without `-d' is I believe also not 
universally defined):

diff --git a/config.guess b/config.guess
index 802e5f1..b422c43 100755
--- a/config.guess
+++ b/config.guess
@@ -2,7 +2,7 @@
 # Attempt to guess a canonical system name.
 #   Copyright 1992-2016 Free Software Foundation, Inc.
 # This file is free software; you can redistribute it and/or modify it
 # under the terms of the GNU General Public License as published by
@@ -268,42 +272,42 @@ case
 	# A Tn.n version is a released field test version.
 	# A Xn.n version is an unreleased experimental baselevel.
 	# 1.2 uses "1.2" for uname -r.
-	echo ${UNAME_MACHINE}-dec-osf`echo ${UNAME_RELEASE} | sed -e
's/^[PVTX]//' | tr 'ABCDEFGHIJKLMNOPQRSTUVWXYZ' 'abcdefghijklmnopqrstuvwxyz'`
+	echo ${UNAME_MACHINE}-dec-osf`echo ${UNAME_RELEASE} | sed -e
's/^[PVTX]//' | tr ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz`

-- see the `tr' invocation at the end.  I spotted it by chance by reading 
the patch and I take it it is an accident which happened while doing a 
mechanical clean-up.

 Would you therefore please look into it and see if my suspicion is right?



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]