On 08/05/15 16:57, Segher Boessenkool wrote:
On Fri, May 08, 2015 at 09:57:45AM -0400, Lennart Sorensen wrote:
I think people can legitimately argue that if someone specifies CROSS_COMPILE then that is what they want to use, and you should NOT go looking at other options at all that might exist in that case.
After all if I make a typo in my CROSS_COMPILE line I would rather get an error that it can't find that compiler, than have it run of and find some other cross compiler I have installed and use that.
So probably the correct thing would be:
if CROSS_COMPILE is set then use CROSS_COMPILE else do the list search instead.
Yes, that is the no-surprises user-in-control way of doing things. Which is my recommendation as well.
Thanks for the review guys. I'll send an updated version of the patch to the list shortly.
ATB,
Mark.