On Thu, 29 Nov 2018 at 15:51, Eric Engestrom <eric.engest...@intel.com> wrote: > > Signed-off-by: Eric Engestrom <eric.engest...@intel.com> > --- > The 32-bit linux one is tested, but the other two are "I think this > should be right".
There's a small nit about the 32 bit linux build. But regardless, the series is Reviewed-by: Emil Velikov <emil.veli...@collabora.com> If any of the examples off, we can correct when we spot it. These are a really good starting point. > +ar = '/usr/bin/ar' AFAICT the archive produced by ar shouldn't vary across CPU architectures. Although varying filename limitations (imposed by the file system) are common. > +llvm-config = '/usr/bin/llvm-config' llvm-config will vary greatly across distros, but on Arch it should be ....llvm-config32, Things may work with as-is, but I'm not sure how reliable/good idea it is. HTH -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
