Hi Tom,
On 9 October 2018 at 05:35, Tom Rini trini@konsulko.com wrote:
On Mon, Oct 08, 2018 at 09:40:51PM -0600, Simon Glass wrote:
Hi Tom,
On 7 October 2018 at 12:20, Tom Rini trini@konsulko.com wrote:
To make testing with clang support easier, add sandbox/clang-7 combination to our testing matrix. To facilitate this, switch to using the "sources" method that the travis.yml file supports to list additional repositories and add the official one for llvm-7. Due to buildman not supporting using clang at this time add logic to manually build a single sandbox configuration in the expected output directory so that we can still invoke all of our tests.
Cc: Simon Glass sjg@chromium.org Signed-off-by: Tom Rini trini@konsulko.com
.travis.yml | 27 ++++++++++++++++++++------- 1 file changed, 20 insertions(+), 7 deletions(-)
Reviewed-by: Simon Glass sjg@chromium.org
What prevents buildman supporting clang?
Well, I couldn't convince buildman to accept "/usr/bin/clang-7" as the full and entire path to the toolchain to use. Ideas? Thanks!
No, sadly it looks like we need to add special support for clang - as per README.clang. We need to set CC and CROSS_COMPILE, at least.
Regards, Simon