Re: [PATCH spice-server] build: Rename spice-server-enums.tmpl.[ch] to spice-server-enums.[ch].tmpl

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 06, 2018 at 11:46:33AM -0300, Eduardo Lima (Etrunko) wrote:
> This is a preparation for meson build, which has built-in support for
> generating enums, but requires the template files to be renamed. It uses
> the basename of template files to generate the output, and in this case
> it would be the same file for both '.c' and '.h'.
> 
> Reference http://mesonbuild.com/Gnome-module.html#gnomemkenums

Hmm the generated files which should have the same base name are
currently spice-server-enums.c and spice-server-enums.h, so this is ok,
and from the link you gave, 'spice-server-enums' would be the first arg
to gnome.mkenums().
Then the link you give lists 2 separate arguments for the templates, c_template
and h_template.
So after reading the link, I'm not sure why this patch is needed?

Christophe

> 
> Signed-off-by: Eduardo Lima (Etrunko) <etrunko@xxxxxxxxxx>
> ---
>  server/Makefile.am                                           | 12 ++++++------
>  .../{spice-server-enums.tmpl.c => spice-server-enums.c.tmpl} |  0
>  .../{spice-server-enums.tmpl.h => spice-server-enums.h.tmpl} |  0
>  3 files changed, 6 insertions(+), 6 deletions(-)
>  rename server/{spice-server-enums.tmpl.c => spice-server-enums.c.tmpl} (100%)
>  rename server/{spice-server-enums.tmpl.h => spice-server-enums.h.tmpl} (100%)
> 
> diff --git a/server/Makefile.am b/server/Makefile.am
> index 5d5590af..c1f241ac 100644
> --- a/server/Makefile.am
> +++ b/server/Makefile.am
> @@ -201,11 +201,11 @@ endif
>  libspice_server_la_LIBADD = libserver.la
>  libspice_server_la_SOURCES =
>  
> -spice-server-enums.c: spice-server.h spice-server-enums.tmpl.c
> -	$(AM_V_GEN)glib-mkenums --template $(srcdir)/spice-server-enums.tmpl.c $< > $@
> +spice-server-enums.c: spice-server.h spice-server-enums.c.tmpl
> +	$(AM_V_GEN)glib-mkenums --template $(srcdir)/spice-server-enums.c.tmpl $< > $@
>  
> -spice-server-enums.h: spice-server.h spice-server-enums.tmpl.h
> -	$(AM_V_GEN)glib-mkenums --template $(srcdir)/spice-server-enums.tmpl.h $< > $@
> +spice-server-enums.h: spice-server.h spice-server-enums.h.tmpl
> +	$(AM_V_GEN)glib-mkenums --template $(srcdir)/spice-server-enums.h.tmpl $< > $@
>  
>  EXTRA_DIST =					\
>  	spice-bitmap-utils.tmpl.c		\
> @@ -213,8 +213,8 @@ EXTRA_DIST =					\
>  	glz-encode-match.tmpl.c			\
>  	glz-encode.tmpl.c			\
>  	spice-server.syms			\
> -	spice-server-enums.tmpl.h		\
> -	spice-server-enums.tmpl.c		\
> +	spice-server-enums.h.tmpl		\
> +	spice-server-enums.c.tmpl		\
>  	$(NULL)
>  
>  BUILT_SOURCES = $(spice_built_sources)
> diff --git a/server/spice-server-enums.tmpl.c b/server/spice-server-enums.c.tmpl
> similarity index 100%
> rename from server/spice-server-enums.tmpl.c
> rename to server/spice-server-enums.c.tmpl
> diff --git a/server/spice-server-enums.tmpl.h b/server/spice-server-enums.h.tmpl
> similarity index 100%
> rename from server/spice-server-enums.tmpl.h
> rename to server/spice-server-enums.h.tmpl
> -- 
> 2.14.3
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]