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

Frediano Ziglio fziglio at redhat.com
Fri Mar 10 16:32:55 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.
> > Sometimes you need to add a long line and you need to reindent
> > all macro.
> > Also you can't easily move lines as you have also to reindent
> > and tools (like diff) require additional parameter to ignore
> > this.
> > Why not something like
> > 
> > EXTRA_DIST = \
> > 	$(VALGRIND_SUPPRESSIONS_FILES) \
> > 	pki/ca-cert.pem \
> > 	pki/server-cert.pem \
> > 	pki/server-key.pem \
> > 	$(NULL)
> > 
> > I don't think is less readable.
> 
> Dunno, I prefer the nicely aligned \, and Makefile.am does not change
> that often. But yeah, that's just personal preference.
> 
> Christophe
> 

Yes, but take a look at server/Makefile.am :-)
At some point seems like a weird textual game!

Frediano


More information about the Spice-devel mailing list