[Mesa-dev] [PATCH] android, autotools: st/mesa: fix location of float64_glsl.h

[Mauro Rossi]

Hi Jordan,

On Wed, Mar 6, 2019 at 10:01 PM Jordan Justen <jordan.l.justen at intel.com> wrote:
>
> On 2019-03-04 16:11:33, Jordan Justen wrote:
> > On 2019-03-04 15:21:26, Mauro Rossi wrote:
> > > Hi,
> > > On Mon, Mar 4, 2019 at 7:50 PM Mauro Rossi <issor.oruam at gmail.com> wrote:
> > > >
> > > > Il lun 4 mar 2019, 18:59 Dylan Baker <dylan at pnwbakers.com> ha scritto:
> > > >>
> > > >> I wrote c812, and I'd be very happy to revert it. The problem that I ran into
> > > >> was putting it in a place that android, meson, and the autotools dist tarball
> > > >> were happy with. I'm hoping that we can remove autotools in the very near future
> > > >> and just revert c812 at that time. Does that seem like a reasonable solution?
> > > >>
> > > >> Dylan
> > >
> > > Hi Dylan,
> > > I have checked on Android.mk and reverting you commit c812c740e6
> > > would be very complex.
> > >
> > > The best way is to proceed now is with my patch to un-break the Android build
> > >
> > > Then as a reference, the future solution may require chages to sources:
> > >
> > > #include "compiler/glsl/{header}.h"
> > > to become
> > > #include "glsl/{header}.h"
> > >
> > > in order to reach the Android generated files with
> > >  $(call generated-sources-dir-for,STATIC_LIBRARIES,libmesa_glsl,,)
> > > as equivalent of -I$(top_builddir)/src/compiler
> >
> > Just wanted to point out that Jason is moving this under glsl in:
> >
> > https://gitlab.freedesktop.org/mesa/mesa/merge_requests/371
>
> Mauro,
>
> Jason pushed MR 371 in 9ab1b1d0227499b7ff6a61fdebe75693212a67f5.
>
> > So, either that will fix this, or else cause it to break in a new,
> > different way. :)
>
> Which was it? :)
>
> -Jordan

The android build is ok for me, without need for android, autotools patch
I tested also build of iris

Mauro