[Mesa-dev] [PATCH 1/2] build: Don't cross-compile GLSL builtin compiler

[Thierry Reding]

On Fri, Sep 14, 2012 at 08:37:43AM -0700, Matt Turner wrote:
> On Fri, Sep 14, 2012 at 12:22 AM, Thierry Reding
> <thierry.reding at avionic-design.de> wrote:
> > The builtin_compiler binary is used during the build process to generate
> > code for the builtin GLSL functions. Since this binary needs to be run
> > on the build host, it must not be cross-compiled.
> >
> > This patch fixes the build system to compile a second version of the
> > source files and the builtin_compiler binary itself for the build
> > system. It does so by defining the CC_FOR_BUILD and CXX_FOR_BUILD
> > variables, which are searched for by the configure script and point to
> > the location of native C and C++ compilers.
> > ---
> 
> I wish you'd sent this to the list a bit more promptly since the last
> update to bug 44618 eight months ago. As it is now, I've got a large
> (100) patch series in progress that touches these Makefile.am's.

I wish the same. At first I was waiting for the automake conversion to
progress further and then I got side-tracked with a bunch of other more
pressing issues. I guess that's just life...

> Overall, it looks pretty good. I'm very glad to learn about the AX_
> macros. I feel much better adding those to Mesa's build system than
> macros like we have now (see MESA_PIC_FLAGS).
> 
> I'd like it if this could be made a patch on top of my
> automake-gallium series,
> http://cgit.freedesktop.org/~mattst88/mesa/log/?h=automake-gallium

I'll give it a shot. Let's hope this doesn't take me another eight
months. =)

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20120914/dee1463c/attachment.pgp>