[Spice-devel] [spice-server] build-sys: Add tests/pki to EXTRA_DIST - OT Makefiles...

Frediano Ziglio fziglio at redhat.com
Thu Mar 23 10:56:03 UTC 2017


> 
> On Fri, Mar 10, 2017 at 09:22:13AM -0500, Frediano Ziglio wrote:
> > > 
> > > This fixes make distcheck
> > > ---
> > > This applies on top of the make check-valgrind patch I sent earlier, I
> > > can
> > > rebase on top of git master if needed
> > > 
> > > Christophe
> > > 
> > >  server/tests/Makefile.am | 7 ++++++-
> > >  1 file changed, 6 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/server/tests/Makefile.am b/server/tests/Makefile.am
> > > index 86977ab..8237138 100644
> > > --- a/server/tests/Makefile.am
> > > +++ b/server/tests/Makefile.am
> > > @@ -2,7 +2,12 @@ NULL =
> > >  
> > >  @VALGRIND_CHECK_RULES@
> > >  VALGRIND_SUPPRESSIONS_FILES =
> > >  $(top_srcdir)/server/tests/valgrind/glib.supp
> > > -EXTRA_DIST = $(VALGRIND_SUPPRESSIONS_FILES)
> > > +EXTRA_DIST =				\
> > > +	$(VALGRIND_SUPPRESSIONS_FILES)	\
> > > +	pki/ca-cert.pem			\
> > > +	pki/server-cert.pem		\
> > > +	pki/server-key.pem		\
> > > +	$(NULL)
> > 
> > Very OT consideration. This alignment at the end is really
> > hard to maintain and there are different styles and alignment
> > in a single file.
> 
> Apart from the OT comment, is this ok to push ? :)
> 
> Christophe
> 

Yes, just was depending on valgrind

Acked-by: Frediano Ziglio <fziglio at redhat.com>

Thanks for fixing

Frediano


More information about the Spice-devel mailing list