OpenTTD

Tasklist

FS#6414 - [PATCH] Honor user set cc, cxx

Attached to Project: OpenTTD
Opened by Wouter van Kesteren (woutershep2) - Saturday, 23 January 2016, 01:24 GMT
Type Patch
Category Build system
Status New
Assigned To No-one
Operating System All
Severity Medium
Priority Normal
Reported Version trunk
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

As it stands --cc-build and --cxx-build is useless when --build is set. Same goes for CC and CXX env variables when --host is given.
Attached is a patch to address this issue, it was made against 1.5.3 but i looked at trunk and saw the same broken logic, so i set the reported version to trunk.

Actual:
./configure --build=x86_64-pc-linux-gnu --cc-build=x86_64-pc-linux-gnu-cc --cxx-build=x86_64-pc-linux-gnu-c++ ...
checking build system type... x86_64-pc-linux-gnu
checking build cc... x86_64-pc-linux-gnu-gcc
checking build c++... x86_64-pc-linux-gnu-g++

Expected:
./configure --build=x86_64-pc-linux-gnu --cc-build=x86_64-pc-linux-gnu-cc --cxx-build=x86_64-pc-linux-gnu-c++ ...
checking build system type... x86_64-pc-linux-gnu
checking build cc... x86_64-pc-linux-gnu-cc
checking build c++... x86_64-pc-linux-gnu-c++

No actual code was added/removed, the checks for the explicitly user-provided $4 and $5 where simply moved before the guessing $3 check.
   compiler-detection.patch (4.5 KiB)
This task depends upon

Loading...