[PATCH wayland 4/5] build: Build a subset of libwayland_util when not building libraries

Peter Hutterer peter.hutterer at who-t.net
Wed Jun 17 14:27:02 PDT 2015


On Wed, Jun 17, 2015 at 03:13:51PM -0500, Derek Foreman wrote:
> The scanner doesn't need anything but wayland-util.c/.h so we can
> shrink wayland-util when not building the main libraries.
> 
> Signed-off-by: Derek Foreman <derekf at osg.samsung.com>
> ---
>  Makefile.am | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)
> 
> diff --git a/Makefile.am b/Makefile.am
> index 31adc97..845c116 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -34,20 +34,23 @@ else
>  wayland_scanner = wayland-scanner
>  endif
>  
> -libwayland_util_la_CFLAGS = $(FFI_CFLAGS) $(GCC_CFLAGS)
> +libwayland_util_la_CFLAGS = $(GCC_CFLAGS)
>  libwayland_util_la_SOURCES =			\
> -	src/connection.c			\
>  	src/wayland-util.c			\
> -	src/wayland-util.h			\
> -	src/wayland-os.c			\
> -	src/wayland-os.h			\
> -	src/wayland-private.h
> +	src/wayland-util.h
>  
>  noinst_LTLIBRARIES = libwayland-util.la
>  
>  if ENABLE_LIBRARIES
>  lib_LTLIBRARIES = libwayland-server.la libwayland-client.la
>  
> +libwayland_util_la_CFLAGS += $(FFI_CFLAGS)
> +libwayland_util_la_SOURCES +=			\
> +	src/connection.c			\
> +	src/wayland-os.c			\
> +	src/wayland-os.h			\
> +	src/wayland-private.h
> +
>  include_HEADERS =				\
>  	src/wayland-util.h			\
>  	src/wayland-server.h			\
> -- 
> 2.1.4

not a big fan of conditionally appending to helper libraries, we could split
this into two fixed helper libraries. but that's personal taste, so the
series is still
Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
   Peter



More information about the wayland-devel mailing list