Patchwork Fix powerpc-lynxos build

login
register
mail settings
Submitter Joseph S. Myers
Date Dec. 16, 2010, 12:06 a.m.
Message ID <Pine.LNX.4.64.1012160004140.4081@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/75707/
State New
Headers show

Comments

Joseph S. Myers - Dec. 16, 2010, 12:06 a.m.
In the course of testing another patch I found that powerpc-*-lynxos* 
included lynx.opt twice in extra_options and this broke the build.  It's a 
bug in the options awk scripts that it breaks the build (through duplicate 
field declarations of fields generated internally to describe target 
options without variables explicitly given in the .opt files), but also 
clearly the .opt file should not be listed twice.  I've committed this 
patch as obvious to remove the duplicate.

Patch

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 167883)
+++ ChangeLog	(working copy)
@@ -1,3 +1,8 @@ 
+2010-12-16  Joseph Myers  <joseph@codesourcery.com>
+
+	* config.gcc (powerpc-*-lynxos*): Don't add lynx.opt to
+	extra_options twice.
+
 2010-12-15  Joseph Myers  <joseph@codesourcery.com>
 
 	* doc/tm.texi.in (US_SOFTWARE_GOFAST): Don't document.
Index: config.gcc
===================================================================
--- config.gcc	(revision 167883)
+++ config.gcc	(working copy)
@@ -2162,7 +2162,6 @@ 
 	tmake_file="t-lynx rs6000/t-lynx"
 	extra_options="${extra_options} rs6000/sysv4.opt lynx.opt"
 	extra_parts="crtbegin.o crtbeginS.o crtend.o crtendS.o"
-	extra_options="${extra_options} lynx.opt"
 	thread_file=lynx
 	gnu_ld=yes
 	gas=yes