On 2019-04-13 at 01:36, Mauro Rossi <issor.or...@gmail.com> wrote:
> Just a message to Eric,
>
> as per our previous private thread,
Sorry, I completely forgot to actually send the fix :facepalm:
>
> I've checked that the Android build works,
> but we use libLLVM70 name in library dependency.
I don't understand the second part is this sentence?
>
> Please adapt and apply the patch to mesa dev branch,
> to fix the breakage
>
> \ prior to " in the LLVM version string value, just touching
> Android.mk solves the problem for me
Yes, this is the correct fix (because of the `eval`, if anyone's wondering).
Reviewed-by: Eric Engestrom <e...@engestrom.ch>
Unfortunately, I'm on holiday for a week now. Can you push the fix yourself?
Otherwise, Tapani can :)
>
> Mauro
>
> On Sat, Apr 13, 2019 at 1:27 AM Mauro Rossi <issor.or...@gmail.com> wrote:
> >
> > Fixes the following building errors:
> >
> > external/mesa/src/gallium/drivers/r600/r600_pipe_common.c:1290:14:
> > error: expected ')'
> > ", LLVM " MESA_LLVM_VERSION_STRING
> > ^
> > <command line>:8:34: note: expanded from here
> > ^
> > external/mesa/src/gallium/drivers/r600/r600_pipe_common.c:1287:10:
> > note: to match this '('
> > snprintf(rscreen->renderer_string, sizeof(rscreen->renderer_string),
> > ^
> > 1 error generated.
> >
> > Fixes: 05b114e ("simplify LLVM version string printing")
> > Signed-off-by: Mauro Rossi <issor.or...@gmail.com>
> > ---
> > Android.mk | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/Android.mk b/Android.mk
> > index 09139e86d1..b835eb64e9 100644
> > --- a/Android.mk
> > +++ b/Android.mk
> > @@ -97,13 +97,13 @@ define mesa-build-with-llvm
> > $(if $(filter $(MESA_ANDROID_MAJOR_VERSION), 4 5), \
> > $(warning Unsupported LLVM version in Android
> > $(MESA_ANDROID_MAJOR_VERSION)),) \
> > $(if $(filter 6,$(MESA_ANDROID_MAJOR_VERSION)), \
> > - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0307
> > -DMESA_LLVM_VERSION_STRING="3.7")) \
> > + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0307
> > -DMESA_LLVM_VERSION_STRING=\"3.7\")) \
> > $(if $(filter 7,$(MESA_ANDROID_MAJOR_VERSION)), \
> > - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> > -DMESA_LLVM_VERSION_STRING="7.0")) \
> > + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> > -DMESA_LLVM_VERSION_STRING=\"7.0\")) \
> > $(if $(filter 8,$(MESA_ANDROID_MAJOR_VERSION)), \
> > - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> > -DMESA_LLVM_VERSION_STRING="7.0")) \
> > + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> > -DMESA_LLVM_VERSION_STRING=\"7.0\")) \
> > $(if $(filter 9,$(MESA_ANDROID_MAJOR_VERSION)), \
> > - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0309
> > -DMESA_LLVM_VERSION_STRING="3.9")) \
> > + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0309
> > -DMESA_LLVM_VERSION_STRING=\"3.9\")) \
> > $(eval LOCAL_SHARED_LIBRARIES += libLLVM70)
> > endef
> >
> > --
> > 2.20.1
> >
> _______________________________________________
> 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
