Re: [Mesa-dev] [PATCH mesa] wayland-egl: fix wayland cflags

2017-10-26 Thread Tobias Klausmann 

On 10/26/17 6:25 PM, Eric Engestrom wrote:
> Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h 
> include"
> Suggested-by: Daniel Stone 
> Signed-off-by: Eric Engestrom 
> ---
>  src/egl/wayland/wayland-egl/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/egl/wayland/wayland-egl/Makefile.am 
> b/src/egl/wayland/wayland-egl/Makefile.am
> index 730fd03ddbe18b71232f..7dde9552f33d6d8936ce 100644
> --- a/src/egl/wayland/wayland-egl/Makefile.am
> +++ b/src/egl/wayland/wayland-egl/Makefile.am
> @@ -3,7 +3,7 @@ pkgconfig_DATA = wayland-egl.pc
>  
>  AM_CFLAGS = $(DEFINES) \
>   $(VISIBILITY_CFLAGS) \
> - $(WAYLAND_SERVER_CFLAGS)
> + $(WAYLAND_CLIENT_CFLAGS)
>  
>  lib_LTLIBRARIES = libwayland-egl.la
>  noinst_HEADERS = wayland-egl-backend.h


Oh,

yep this is better! I should have changed it when fixing the build a few
weeks ago.

Acked-by: Tobias Klausmann 


___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH mesa] wayland-egl: fix wayland cflags

2017-10-26 Thread Emil Velikov 
On 26 October 2017 at 17:25, Eric Engestrom  wrote:
> Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h 
> include"
> Suggested-by: Daniel Stone 
> Signed-off-by: Eric Engestrom 
> ---
>  src/egl/wayland/wayland-egl/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/egl/wayland/wayland-egl/Makefile.am 
> b/src/egl/wayland/wayland-egl/Makefile.am
> index 730fd03ddbe18b71232f..7dde9552f33d6d8936ce 100644
> --- a/src/egl/wayland/wayland-egl/Makefile.am
> +++ b/src/egl/wayland/wayland-egl/Makefile.am
> @@ -3,7 +3,7 @@ pkgconfig_DATA = wayland-egl.pc
>
>  AM_CFLAGS = $(DEFINES) \
> $(VISIBILITY_CFLAGS) \
> -   $(WAYLAND_SERVER_CFLAGS)
> +   $(WAYLAND_CLIENT_CFLAGS)
>
In reality there's no distinction (nor documentation) if the server of
the client should provide wayland-egl.h last time I've checked.
But I guess it doesn't really matter since we're heading towards
(re)moving wayland-egl from Mesa.

Acked-by: Emil Velikov 

-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH mesa] wayland-egl: fix wayland cflags

2017-10-26 Thread Daniel Stone 
On 26 October 2017 at 17:25, Eric Engestrom  wrote:
> Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h 
> include"
> Suggested-by: Daniel Stone 

Acked-by: Daniel Stone 
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH mesa] wayland-egl: fix wayland cflags

2017-10-26 Thread Eric Engestrom 
Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h 
include"
Suggested-by: Daniel Stone 
Signed-off-by: Eric Engestrom 
---
 src/egl/wayland/wayland-egl/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/egl/wayland/wayland-egl/Makefile.am 
b/src/egl/wayland/wayland-egl/Makefile.am
index 730fd03ddbe18b71232f..7dde9552f33d6d8936ce 100644
--- a/src/egl/wayland/wayland-egl/Makefile.am
+++ b/src/egl/wayland/wayland-egl/Makefile.am
@@ -3,7 +3,7 @@ pkgconfig_DATA = wayland-egl.pc
 
 AM_CFLAGS = $(DEFINES) \
$(VISIBILITY_CFLAGS) \
-   $(WAYLAND_SERVER_CFLAGS)
+   $(WAYLAND_CLIENT_CFLAGS)
 
 lib_LTLIBRARIES = libwayland-egl.la
 noinst_HEADERS = wayland-egl-backend.h
-- 
Cheers,
  Eric

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev