[Spice-devel] [spice-protocol 4/4] build-sys: Install code generation data/scripts

[Frediano Ziglio]

> 
> spice-common generate code from these python scripts and the installed
> SPICE .proto. This commit installs them during make install, and adds a
> codegendir variable to the pkgconfig file so that spice-common can
> locate the scripts.
> ---
>  python_modules/Makefile.am | 17 ++++++++---------
>  spice-protocol.pc.in       |  2 ++
>  2 files changed, 10 insertions(+), 9 deletions(-)
> 
> diff --git a/python_modules/Makefile.am b/python_modules/Makefile.am
> index 50e1a71..3a4515a 100644
> --- a/python_modules/Makefile.am
> +++ b/python_modules/Makefile.am
> @@ -1,16 +1,15 @@
>  NULL =
>  
> -PYTHON_MODULES =				\
> -	__init__.py				\
> -	codegen.py				\
> -	demarshal.py				\
> -	marshal.py				\
> -	ptypes.py				\
> -	spice_parser.py				\
> +python_codegendir = $(libdir)/spice-protocol/python_modules
> +dist_python_codegen_DATA =		\
> +	__init__.py			\
> +	codegen.py			\
> +	demarshal.py			\
> +	marshal.py			\
> +	ptypes.py			\
> +	spice_parser.py			\
>  	$(NULL)
>

Looks like there are some space changes. Are they required? The patch looks much bigger.

Frediano

  
> -EXTRA_DIST = $(PYTHON_MODULES)
> -
>  DISTCLEANFILES = *.pyc
>  
>  -include $(top_srcdir)/git.mk
> diff --git a/spice-protocol.pc.in b/spice-protocol.pc.in
> index c7e76cd..173781a 100644
> --- a/spice-protocol.pc.in
> +++ b/spice-protocol.pc.in
> @@ -1,5 +1,7 @@
>  prefix=@prefix@
> +exec_prefix=@exec_prefix@
>  includedir=@includedir@
> +codegendir=@libdir@/spice-protocol
>  
>  Name: spice-protocol
>  Description: SPICE protocol headers
> --
> 2.4.3