[Mesa-dev] [PATCH] travis: use LLVM 5.0 for Meson Vulkan

[Gert Wollny]

Am Freitag, den 15.12.2017, 17:17 +0100 schrieb Juan A. Suarez Romero:
> Travis CI has moved to LLVM 5.0, and meson is detecting automatically
> the available version.

Considering that LLVM 5.0 is obviously not properly made available in
the travis-ci environment, I've reported the problem with travis-
upstream: 

https://github.com/travis-ci/travis-ci/issues/8925

Apart from that I think Eric is right, one should be able to specify
the version instead of just taking the latest availabe version. The
autotools based builds all force a version, which helps work around the
other problem:  

The travis version of LLVM is in /usr/local, so it is definitely not a
packaged version, since this would go into /usr, and because the PATH
defines /usr/local/bin before /usr/bin, and the local (incomplete)
version takes precedence, so adding llvm-5.0-dev like this is probably
useless.

Gallium ST Clover with LLVM-5.0 passes because it uses the package
version by specifically picking llvm-config-5.0 which, as Emil pointed
out, is a Debian+derivatives feat.

Best, 
Gert

> 
> So just let's change to LLVM 5.0 for this case.
> ---
>  .travis.yml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 211df3ec1ef..20432361176 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -37,12 +37,12 @@ matrix:
>        addons:
>          apt:
>            sources:
> -            - llvm-toolchain-trusty-3.9
> +            - llvm-toolchain-trusty-5.0
>            packages:
>              # LLVM packaging is broken and misses these dependencies
>              - libedit-dev
>              # From sources above
> -            - llvm-3.9-dev
> +            - llvm-5.0-dev
>              # Common
>              - xz-utils
>              - libexpat1-dev