diff mbox

[U-Boot,1/3] buildman: Allow 'gnueabihf' toolchains for ARM

Message ID 1492213672-20931-1-git-send-email-trini@konsulko.com
State Accepted
Commit 546a6f3a9b48fb891e5530528cc0696564438760
Delegated to: Tom Rini
Headers show

Commit Message

Tom Rini April 14, 2017, 11:47 p.m. UTC
Many toolchains for ARM use the 'gnueabihf' suffix rather than just
'gnueabi', so allow these to be used, but with a lower priority than
'gnueabi' ones.

Cc: Simon Glass <sjg@chromium.org>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
 tools/buildman/toolchain.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Simon Glass April 15, 2017, 4:06 p.m. UTC | #1
On 14 April 2017 at 17:47, Tom Rini <trini@konsulko.com> wrote:
> Many toolchains for ARM use the 'gnueabihf' suffix rather than just
> 'gnueabi', so allow these to be used, but with a lower priority than
> 'gnueabi' ones.
>
> Cc: Simon Glass <sjg@chromium.org>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  tools/buildman/toolchain.py | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
Tom Rini April 19, 2017, 1:03 p.m. UTC | #2
On Fri, Apr 14, 2017 at 07:47:50PM -0400, Tom Rini wrote:

> Many toolchains for ARM use the 'gnueabihf' suffix rather than just
> 'gnueabi', so allow these to be used, but with a lower priority than
> 'gnueabi' ones.
> 
> Cc: Simon Glass <sjg@chromium.org>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
index 47788762016b..5cf97ac8148e 100644
--- a/tools/buildman/toolchain.py
+++ b/tools/buildman/toolchain.py
@@ -120,8 +120,9 @@  class Toolchain:
             Priority of toolchain, PRIORITY_CALC=highest, 20=lowest.
         """
         priority_list = ['-elf', '-unknown-linux-gnu', '-linux',
-            '-none-linux-gnueabi', '-uclinux', '-none-eabi',
-            '-gentoo-linux-gnu', '-linux-gnueabi', '-le-linux', '-uclinux']
+            '-none-linux-gnueabi', '-none-linux-gnueabihf', '-uclinux',
+            '-none-eabi', '-gentoo-linux-gnu', '-linux-gnueabi',
+            '-linux-gnueabihf', '-le-linux', '-uclinux']
         for prio in range(len(priority_list)):
             if priority_list[prio] in fname:
                 return PRIORITY_CALC + prio