Re: [spice-common PATCH] codegen Makefile: add common/ to --include client_marshallers.h

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

 



> 
> Fixes out-of-tree builds.
> 
> For example spice-gtk out-of-tree build fails with:
>   ../subprojects/spice-common/common/generated_client_marshallers.h:19:10:
>       fatal error: client_marshallers.h: No such file or directory
>    #include "client_marshallers.h"
>             ^~~~~~~~~~~~~~~~~~~~~~
> 
> Signed-off-by: Uri Lublin <uril@xxxxxxxxxx>

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

Probably now you need a patch for spice-common in spice-gtk.

Maybe would be worth to add a out of tree build to CI ?

> ---
>  common/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/common/Makefile.am b/common/Makefile.am
> index 2dd6d04..a904dae 100644
> --- a/common/Makefile.am
> +++ b/common/Makefile.am
> @@ -114,7 +114,7 @@ generated_client_demarshallers.c generated_messages.h:
> $(top_srcdir)/spice.proto
>  	--generated-declaration-file generated_messages.h $< $@ >/dev/null
>  
>  generated_client_marshallers.c generated_client_marshallers.h:
>  $(top_srcdir)/spice.proto $(MARSHALLERS_DEPS)
> -	$(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py --generate-marshallers
> -P --include client_marshallers.h --client \
> +	$(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py --generate-marshallers
> -P --include common/client_marshallers.h --client \
>  	--generate-header $< $@ >/dev/null
>  
>  generated_server_demarshallers.c: $(top_srcdir)/spice.proto
>  $(MARSHALLERS_DEPS)

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




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