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

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

 



> 
> 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@xxxxxxxxxx>

Thanks for fixing

Frediano
_______________________________________________
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]