Bug 14871 - Non-unified-tree ordinary cross-compiler builds default to stupid names for tools
Summary: Non-unified-tree ordinary cross-compiler builds default to stupid names for t...
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: bootstrap (show other bugs)
Version: 3.4.0
: P3 minor
Target Milestone: 4.0.0
Assignee: Nathanael C. Nerode
URL:
Keywords: build
Depends on:
Blocks:
 
Reported: 2004-04-07 02:03 UTC by Nathanael C. Nerode
Modified: 2004-08-25 04:13 UTC (History)
1 user (show)

See Also:
Host: *
Target: *
Build: *
Known to work:
Known to fail:
Last reconfirmed: 2004-04-09 05:15:29


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nathanael C. Nerode 2004-04-07 02:03:53 UTC
From Makefile.tpl: 
 
USUAL_AS_FOR_TARGET = ` \ 
  if [ -f $$r/gas/as-new ] ; then \ 
    echo $$r/gas/as-new ; \ 
... 
    if [ '$(host)' = '$(target)' ] ; then \ 
      echo $(AS); \ 
    else \ 
       echo as | sed '$(program_transform_name)' ; \ 
^^^^^^^^^^^^^^^ 
This isn't really what we're looking for, since program_transform_name is 
for the *newly built* tools, and the user may be supplying something weird, 
but it says nothing about where to find preexisting tools.  We should trust 
configure's choice, which is found using NCN_STRICT_TARGET_TOOL, and is 
essentially $(target_alias)-as (unless it was set by the user in some 
other way, of course). 
 
I have a patch for this already. 
 
     fi; \ 
   fi` 
 
Similarly for the other tools.  :-P
Comment 1 GCC Commits 2004-04-09 05:15:19 UTC
Subject: Bug 14871

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	neroden@gcc.gnu.org	2004-04-09 05:15:10

Modified files:
	.              : ChangeLog Makefile.tpl Makefile.in configure 
	                 configure.in 

Log message:
	PR bootstrap/14871
	* Makefile.tpl: If we don't have built-in-tree target tools,
	use the ones found by configure rather than hacking around with
	program_transform_name.
	* configure.in: Give Makefile.tpl the information necessary
	to do that.
	* Makefile.in: Regenerate.
	* configure: Regenerate.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/ChangeLog.diff?cvsroot=gcc&r1=1.905&r2=1.906
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/Makefile.tpl.diff?cvsroot=gcc&r1=1.89&r2=1.90
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/Makefile.in.diff?cvsroot=gcc&r1=1.207&r2=1.208
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/configure.diff?cvsroot=gcc&r1=1.142&r2=1.143
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/configure.in.diff?cvsroot=gcc&r1=1.282&r2=1.283

Comment 2 Andrew Pinski 2004-05-26 13:46:14 UTC
This is fixed, right, please close it if it is.
Comment 3 Andrew Pinski 2004-08-25 04:13:35 UTC
Fixed in 3.5.0.