On Thu, Oct 29, 2015 at 06:40:38PM -0200, Eduardo Lima (Etrunko) wrote: > Signed-off-by: Eduardo Lima (Etrunko) <etrunko@xxxxxxxxxx> > --- > Makefile.am | 5 +++++ > git.mk | 2 +- > m4/.gitignore | 0 > src/Makefile.am | 8 ++++++++ > 4 files changed, 14 insertions(+), 1 deletion(-) > delete mode 100644 m4/.gitignore > > diff --git a/Makefile.am b/Makefile.am > index d0e9ab4..9646119 100644 > --- a/Makefile.am > +++ b/Makefile.am > @@ -48,6 +48,11 @@ MAINTAINERCLEANFILES = \ > $(srcdir)/m4/lt~obsolete.m4 \ > $(NULL) > > +GITIGNOREFILES = \ > + AUTHORS \ This one should be in xxxCLEANFILES (MAINTAINER maybe?) > + build-aux/test-driver \ Not sure about this one, but I suspect it belongs to MAINTAINERCLEANFILES as well as the other build-aux/ files are listed in it. > + $(NULL) > + > dist-hook: gen-ChangeLog gen-AUTHORS > > # Generate the ChangeLog file (with all entries since the switch to git) > diff --git a/git.mk b/git.mk > index 0b26b23..277e6e3 100644 > --- a/git.mk > +++ b/git.mk > @@ -123,7 +123,7 @@ $(srcdir)/.gitignore: Makefile.am $(top_srcdir)/git.mk > $(gsettings__enum_file) \ > ; do echo /$$x; done; \ > fi; \ > - if test -f $(srcdir)/po/Makefile.in.in; then \ > + if test -f $(top_srcdir)/po/Makefile.in.in; then \ Same interrogation as Fidencio here. Are you updating git.mk from 'upstream', or was it changed for a different reason? > for x in \ > po/Makefile.in.in \ > po/Makefile.in \ > diff --git a/m4/.gitignore b/m4/.gitignore > deleted file mode 100644 > index e69de29..0000000 > diff --git a/src/Makefile.am b/src/Makefile.am > index 1ebc24e..5ce08bc 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -218,4 +218,12 @@ debug_helper_LDFLAGS = $(GLIB2_LIBS) -Wl,--subsystem,windows > debug_helper_CFLAGS = $(GLIB2_CFLAGS) > endif > > +GITIGNOREFILES = \ > + view/.deps \ > + view/.dirstamp \ > + view/.libs \ > + view/*.lo \ > + view/*.o \ > + $(NULL) This hunk does not look correct, the whole point of git.mk is to avoid listing these explicitly. What happens here is that git.mk does not know it should recurse in view/ (I don't know if it's supposed to guess or not). One way around it was to add a Makefile.am in view/ only containing -include $(top_srcdir)/git.mk (+ the associated changes in configure.ac and src/Makefile.am). I don't know if there's a better way of handling this. Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list