buildman/toolchain.py: fix toolchain directory
authorTrevor Woerner <trevor@toganlabs.com>
Wed, 21 Nov 2018 08:31:12 +0000 (03:31 -0500)
committerSimon Glass <sjg@chromium.org>
Wed, 5 Dec 2018 13:01:35 +0000 (06:01 -0700)
commitd82f539ab900c52a669a4f8407a39b8fb125f0ab
tree15b85d415149cf7cd7f6a5af1947e4c03944a257
parentbcb514ac02e8b8b823b7db4faa00ddcb8e78876a
buildman/toolchain.py: fix toolchain directory

The hexagon toolchain (4.6.1) from kernel.org, for example, was packaged in
a way that is different from most toolchains. The first entry when unpacking
most toolchain tarballs is:

gcc-<version>-nolib/<targetarch>-<system>

e.g.:

gcc-8.1.0-nolibc/aarch64-linux/

The first entry of the hexagon toolchain, however, is:

gcc-4.6.1-nolibc/

This causes the buildman logic in toolchain.py::ScanPath() to not be able to
find the "*gcc" executable since it looks in gcc-4.6.1-nolib/{.|bin|usr/bin}
instead of gcc-4.6.1/hexagon-linux/{.|bin|usr/bin}. Therefore when buildman
tries to download a set of toolchains that includes hexagon, the script fails.

This update takes the second line of the tarball unpacking (which works for
all the toolchains I've tested from kernel.org) and parses it to take the
first two elements, separated by '/'. It makes this logic a bit more robust.

Signed-off-by: Trevor Woerner <trevor@toganlabs.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
tools/buildman/toolchain.py