[Piglit] [PATCH] util: add includes when building for Wayland

Tapani Pälli tapani.palli at intel.com
Wed May 16 11:23:04 UTC 2018


On 05/11/2018 06:01 PM, vadym.shovkoplias wrote:
> This fixes following compile issue on OpenSUSE Tumbleweed:
> 
> piglit_wl_framework.c:33:10: fatal error: wayland-client.h:
> No such file or directory #include <wayland-client.h>

Yeah, this path does not propagate to UTIL_GL_INCLUDES ATM. I think this 
works in most systems because the header is found from default include 
path (/usr/include) but in case of Tumbleweed it seems 
'/usr/include/wayland'. Change looks good to me!

Reviewed-by: Tapani Pälli <tapani.palli at intel.com>

> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106477
> Signed-off-by: Sergii Romantsov <sergii.romantsov at globallogic.com>
> ---
>   tests/util/CMakeLists.txt | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/tests/util/CMakeLists.txt b/tests/util/CMakeLists.txt
> index 0508eb0..3fa67d7 100644
> --- a/tests/util/CMakeLists.txt
> +++ b/tests/util/CMakeLists.txt
> @@ -94,6 +94,9 @@ if(PIGLIT_USE_WAFFLE)
>   		list(APPEND UTIL_GL_SOURCES
>   			piglit-framework-gl/piglit_wl_framework.c
>   		)
> +		list(APPEND UTIL_GL_INCLUDES
> +			${WAYLAND_wayland-client_INCLUDEDIR}
> +		)
>   	endif()
>   	if(PIGLIT_HAS_X11)
>   		list(APPEND UTIL_GL_SOURCES
> 


More information about the Piglit mailing list