On Sun, Mar 03, 2019 at 07:10:24PM +0000, Frediano Ziglio wrote: > Python script generates code and header together however allowed > to save only one of them. > Allows to save both of them together to reduce number of time > we call Python script. > > Signed-off-by: Frediano Ziglio <fzig...@redhat.com> > --- > common/Makefile.am | 16 ++++++---------- > common/client_marshallers.h | 2 +- > common/meson.build | 19 +++++++++++++++---- > spice_codegen.py | 12 ++++++++++-- > 4 files changed, 32 insertions(+), 17 deletions(-) > > diff --git a/common/Makefile.am b/common/Makefile.am > index 3318009..3da5bad 100644 > --- a/common/Makefile.am > +++ b/common/Makefile.am > @@ -112,21 +112,17 @@ MARSHALLERS_DEPS = > \ > generated_client_demarshallers.c: $(top_srcdir)/spice.proto > $(MARSHALLERS_DEPS) > $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-demarshallers --client --include common/messages.h $< $@ >/dev/null > > -generated_client_marshallers.h: $(top_srcdir)/spice.proto $(MARSHALLERS_DEPS) > - $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-marshallers -P --client --include common/messages.h -H $< $@ > >/dev/null > - > -generated_client_marshallers.c: $(top_srcdir)/spice.proto $(MARSHALLERS_DEPS) > - $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-marshallers -P --include client_marshallers.h --client $< $@ > >/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 \ > + --generated-header generated_client_marshallers.h $< $@ >/dev/null > > generated_server_demarshallers.c: $(top_srcdir)/spice.proto > $(MARSHALLERS_DEPS) > $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-demarshallers --server --include common/messages.h $< $@ >/dev/null > > STRUCTS = -M String -M Rect -M Point -M DisplayBase -M Fill -M Opaque -M > Copy -M Blend -M Blackness -M Whiteness -M Invers -M Rop3 -M Stroke -M Text > -M Transparent -M AlphaBlend -M Composite > -generated_server_marshallers.c: $(top_srcdir)/spice.proto $(MARSHALLERS_DEPS) > - $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-marshallers $(STRUCTS) --server --include common/messages.h $< $@ > >/dev/null > - > -generated_server_marshallers.h: $(top_srcdir)/spice.proto $(MARSHALLERS_DEPS) > - $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-marshallers $(STRUCTS) --server --include common/messages.h -H $< > $@ >/dev/null > +generated_server_marshallers.c generated_server_marshallers.h: > $(top_srcdir)/spice.proto $(MARSHALLERS_DEPS) > + $(AM_V_GEN)$(PYTHON) $(top_srcdir)/spice_codegen.py > --generate-marshallers $(STRUCTS) --server --include common/messages.h \ > + --generated-header generated_server_marshallers.h $< $@ >/dev/null > > EXTRA_DIST = \ > $(CLIENT_MARSHALLERS) \ > diff --git a/common/client_marshallers.h b/common/client_marshallers.h > index f082934..b67b98e 100644 > --- a/common/client_marshallers.h > +++ b/common/client_marshallers.h > @@ -21,9 +21,9 @@ > > #include <spice/protocol.h> > > +#include "messages.h" > #include "common/generated_client_marshallers.h" > #include "marshaller.h" > -#include "messages.h"
Is this hunk intentionally in this patch?
>
> SPICE_BEGIN_DECLS
>
> diff --git a/common/meson.build b/common/meson.build
> index 156297b..9575568 100644
> --- a/common/meson.build
> +++ b/common/meson.build
> @@ -65,8 +65,14 @@ spice_common_dep = declare_dependency(link_with :
> spice_common_lib,
> if spice_common_generate_client_code
> targets = [
> ['client_demarshallers', spice_proto,
> 'generated_client_demarshallers.c', ['--generate-demarshallers', '--client',
> '--include', 'common/messages.h', '@INPUT@', '@OUTPUT@']],
> - ['client_marshalers', spice_proto, 'generated_client_marshallers.c',
> ['--generate-marshallers', '-P', '--client', '--include',
> 'client_marshallers.h', '@INPUT@', '@OUTPUT@']],
> - ['client_marshallers_h', spice_proto, 'generated_client_marshallers.h',
> ['--generate-marshallers', '-P', '--client', '--include',
> 'common/messages.h', '-H', '@INPUT@', '@OUTPUT@']],
> + ['client_marshallers', spice_proto,
> + ['generated_client_marshallers.c', 'generated_client_marshallers.h'],
> + ['--generate-marshallers',
> + '-P', '--client', '--include', 'client_marshallers.h',
> + '--generated-header', '@OUTPUT1@',
> + '@INPUT@', '@OUTPUT0@'
> + ]
> + ]
> ]
>
> spice_common_client_sources = [
> @@ -116,8 +122,13 @@ if spice_common_generate_server_code
>
> targets = [
> ['server_demarshallers', spice_proto,
> 'generated_server_demarshallers.c', ['--generate-demarshallers', '--server',
> '--include', 'common/messages.h', '@INPUT@', '@OUTPUT@']],
> - ['server_marshallers', spice_proto, 'generated_server_marshallers.c',
> ['--generate-marshallers', '--server'] + structs_args + ['--include',
> 'common/messages.h', '@INPUT@', '@OUTPUT@']],
> - ['server_marshallers_h', spice_proto, 'generated_server_marshallers.h',
> ['--generate-marshallers', '--server'] + structs_args + ['--include',
> 'common/messages.h', '-H', '@INPUT@', '@OUTPUT@']],
> + ['server_marshallers', spice_proto,
> + ['generated_server_marshallers.c', 'generated_server_marshallers.h'],
> + ['--generate-marshallers',
> + '--generated-header', '@OUTPUT1@',
> + '--server'] + structs_args + ['--include', 'common/messages.h',
> '@INPUT@', '@OUTPUT0@'
> + ]
> + ],
> ]
>
> spice_common_server_sources = []
> diff --git a/spice_codegen.py b/spice_codegen.py
> index 7e22092..4664740 100755
> --- a/spice_codegen.py
> +++ b/spice_codegen.py
> @@ -175,6 +175,8 @@ parser.add_option("--ptrsize", dest="ptrsize",
> help="set default pointer size", default="4")
> parser.add_option("--license", dest="license",
> help="license to use for generated file(s) (LGPL/BSD)",
> default="LGPL")
> +parser.add_option("--generated-header", dest="generated_header",
> metavar="FILE",
> + help="Name of the file to generate the header")
Bit unsure about the option name. --header-name maybe? Or just infer it
from the .c file name?
apart from this,
Reviewed-by: Christophe Fergeau <cferg...@redhat.com>
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/spice-devel
