diff mbox

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

Message ID alpine.DEB.2.00.1605271403420.9344@tp.orcam.me.uk
State New
Headers show

Commit Message

Maciej W. Rozycki May 27, 2016, 1:05 p.m. UTC

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 mbox


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?