Looks fine to me, Reviewed-By: Gert Wollny <gw.foss...@gmail.com>
Am Mittwoch, den 28.02.2018, 23:18 +0200 schrieb Andres Gomez: > Travis CI has moved to LLVM 5.0, and meson is detecting automatically > the available version in /usr/local/bin based on the PATH env > variable > order preference. > > As for 0.44.x, Meson cannot receive the path to the llvm-config > binary > as a configuration parameter. See > https://github.com/mesonbuild/meson/issues/2887 and > https://github.com/dcbaker/meson/commit/7c8b6ee3fa42f43c9ac7dcacc61a7 > 7eca3f1bcef > > We want to use the custom (APT) installed version. Therefore, let's > make Meson find our wanted version sooner than the one at > /usr/local/bin > > Once this is corrected, we would still need a patch similar to: > https://lists.freedesktop.org/archives/mesa-dev/2017-December/180217. > html > > v2: Create the link only to the specificly wanted LLVM version > (Gert). > > Cc: Eric Engestrom <eric.engest...@imgtec.com> > Cc: Dylan Baker <dy...@pnwbakers.com> > Cc: Emil Velikov <emil.veli...@collabora.com> > Cc: Juan A. Suarez Romero <jasua...@igalia.com> > Cc: Gert Wollny <gw.foss...@gmail.com> > Cc: Jon Turney <jon.tur...@dronecode.org.uk> > Signed-off-by: Andres Gomez <ago...@igalia.com> > Reviewed-and-Tested-by: Eric Engestrom <eric.engest...@imgtec.com> > Reviewed-by: Dylan Baker <dy...@pnwbakers.com> > Reviewed-by: Juan A. Suarez <jasua...@igalia.com> > --- > .travis.yml | 30 ++++++++++++++++++++++++++---- > 1 file changed, 26 insertions(+), 4 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index 0ec08e5bff7..823111ca539 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -34,6 +34,8 @@ matrix: > - LABEL="meson Vulkan" > - BUILD=meson > - MESON_OPTIONS="-Ddri-drivers= -Dgallium-drivers=" > + - LLVM_VERSION=4.0 > + - LLVM_CONFIG="llvm-config-${LLVM_VERSION}" > addons: > apt: > sources: > @@ -573,8 +575,28 @@ script: > scons $SCONS_TARGET && eval $SCONS_CHECK_COMMAND; > fi > > - - if test "x$BUILD" = xmeson; then > - export CFLAGS="$CFLAGS -isystem`pwd`"; > - meson _build $MESON_OPTIONS; > - ninja -C _build; > + - | > + if test "x$BUILD" = xmeson; then > + > + # Travis CI has moved to LLVM 5.0, and meson is detecting > + # automatically the available version in /usr/local/bin based > on > + # the PATH env variable order preference. > + # > + # As for 0.44.x, Meson cannot receive the path to the > + # llvm-config binary as a configuration parameter. See > + # https://github.com/mesonbuild/meson/issues/2887 and > + # https://github.com/dcbaker/meson/commit/7c8b6ee3fa42f43c9ac7 > dcacc61a77eca3f1bcef > + # > + # We want to use the custom (APT) installed version. > Therefore, > + # let's make Meson find our wanted version sooner than the one > + # at /usr/local/bin > + # > + # Once this is corrected, we would still need a patch similar > + # to: > + # https://lists.freedesktop.org/archives/mesa-dev/2017-Decembe > r/180217.html > + test -f /usr/bin/$LLVM_CONFIG && ln -s /usr/bin/$LLVM_CONFIG > $HOME/prefix/bin/llvm-config > + > + export CFLAGS="$CFLAGS -isystem`pwd`" > + meson _build $MESON_OPTIONS > + ninja -C _build > fi _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev