[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