On Tue, 2018-08-07 at 12:57 -0400, Jan Vesely wrote:
> 
> 
> On Tue, Aug 7, 2018 at 12:28 PM, Juan A. Suarez Romero <jasua...@igalia.com> 
> wrote:
> > On Tue, 2018-08-07 at 16:12 +0100, Emil Velikov wrote:
> > > On 6 August 2018 at 11:17, Juan A. Suarez Romero <jasua...@igalia.com> 
> > > wrote:
> > > > LLVM 6.0 requires GCC 4.9, which is not available in main Travis
> > > > repository.
> > > > 
> > > 
> > > We're not building LLVM 6.0 here, hence the commit message is a bit 
> > > confusing.
> > > Please add more information on the topic - even a bug report if there is 
> > > one.
> > > 
> > 
> > We are using now LLVM 6.0, for "make Vulkan" and "make Gallium Drivers
> > RadeonSI".
> > 
> > Previously we were using LLVM 6.0 for "make Gallium ST Clover LLVM-6.0", 
> > which
> > already had this ubuntu-toolchain-r-test.
> > 
> > Without this, we get an error like in 
> > https://travis-ci.org/mesa3d/mesa/jobs/413159413
> 
> I think Emil's point was that we actually need libstc++4.9, because llvm-lib 
> link to it, rather than the gcc compiler itself.
> 

Ah, I see. I'll update the commit message to say that libstdc++4.9 is required,
instead of GCC 4.9.

Emil: with this change, is this R-b from you?

        J.A.

> Jan
>  
> > 
> > 
> > 
> >         J.A.
> > 
> > > With that
> > > Reviewed-by: Emil Velikov <emil.veli...@collabora.com>
> > > 
> > > -Emil
> > > 
> > 
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to