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.red...@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

Attachment: pgpIVOJbwtcTH.pgp
Description: PGP signature

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

Reply via email to