On 10/19/2012 12:14 PM, Eric Blake wrote: > On 10/18/2012 09:33 PM, Eric Blake wrote: >> On 10/18/2012 05:20 PM, Cole Robinson wrote: >>> AUTHORS.in tracks the maintainers, as well as some folks who were >>> previously in AUTHORS but don't have a git commit with proper >>> attribution. >>> >>> Generated output is sorted alphabetically and lacks pretty spacing, so >>> tweak AUTHORS.in to follow the same format. >>> >>> Additionally, drop the syntax-check rule that previously validated >>> AUTHORS against git log. >>> --- >>> >>> I couldn't get the latter chunk of sed magic working... and since nothing >>> really makes me reach my frustration limit faster than Makefiles and >>> shell trickery, I just stuck with the basic changes. >> >> Fair enough. I guess if I want it any fancier, I'll actually have to >> test my stuff :) >> >> ACK > > Squash this in, so that people doing incremental upgrades aren't bitten > by a nasty-gram from automake about AUTHORS not existing: > > diff --git i/cfg.mk w/cfg.mk > index eed7fd8..d3c96ba 100644 > --- i/cfg.mk > +++ w/cfg.mk > @@ -699,6 +699,7 @@ ifeq (0,$(MAKELEVEL)) > _clean_requested = $(filter %clean,$(MAKECMDGOALS)) > ifeq (1,$(_update_required)$(_clean_requested)) > $(info INFO: gnulib update required; running ./autogen.sh first) > + $(shell touch $(srcdir)/AUTHORS $(srcdir)/ChangeLog) > maint.mk Makefile: _autogen > endif > endif >
Thanks Eric, pushed with that change, and the switch to `` over $() - Cole -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list