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

[Dylan Baker]

Quoting Emil Velikov (2017-12-15 09:35:03)
> On 15 December 2017 at 16:49, Eric Engestrom <eric.engestrom at imgtec.com> wrote:
> > On Friday, 2017-12-15 17:17:55 +0100, Juan A. Suarez Romero wrote:
> >> Travis CI has moved to LLVM 5.0, and meson is detecting automatically
> >> the available version.
> >>
> >> So just let's change to LLVM 5.0 for this case.
> >
> > I thought we were using 3.9 on purpose, to test our minimum required
> > llvm version?
> >
> > I think the proper fix would be to add a `-D llvm-version=3.9` option
> > and use that in the dependency() check.
> >
> Allow me to suggest and alternative solution:
> 
> Feed in the llvm-config name (aka LLVM_CONFIG) like we do for scons
> and autotools.
> It will provide some nice consistency, plus it will allow for easier
> selection as path and version.
> 
> Not to mention that distributions (Debian for example) allow multiple
> versions to coexist, naming the binary - llvm-config-$version.
> 
> -Emil

I don't object to that solution, but it would require a change to upstream
meson, which currently doesn't have support for passing an explicit binary to
the dependency call. It would/will take some time for that to move through
review and would not be available until 0.45.0 at the earliest. Since 0.44.0
was released less than a week ago I would not expect 0.45 until mid January at
the earliest.

I think the easiest immediate solution would be to set the path environment
variable to not include /usr/local, or to use a symlink to mask the version in
/usr/local with the version that we want to use.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171215/f7decced/attachment.sig>