On Jun 17, 2012, at 21:29 , Paul Davis wrote: > I'm not sure I like this so much. Playing around with it, its a bit > prone to screw ups.
I just don't want to maintain this file manually any more. It is error-prone and makes merging user-contributions a pain. I'm happy to have this implemented in any other way, but I think we should try to remove any mechanical steps from maintaining our source if we can. I hope you agree! :) > It also breaks if AUTHORS.gz exists before you > pull in new commits. We could solve that by forcing it to build every > time but that's a bit of a hack for not much gain. Can you explain how it breaks if AUTHORS.gz exists before the merge? If you mean THANKS.gz, my idea was that this is only relevant on packaging time (make distcheck) where THANKS.gz by definition does not exist. > Its also got Benoit in there twice since he made commits with slightly > different author/committer names which also seems awkward. The subsequent .mailmap commit fixes the dupes. The push emails seem to be delayed atm, I reported this to danielsh on #asfinfra. Cheers Jan -- > > On Sun, Jun 17, 2012 at 1:29 PM, <[email protected]> wrote: >> Updated Branches: >> refs/heads/master 9514ed3f5 -> 7646794ea >> >> >> Automate maintenance of the THANKS file >> >> THANKS now is automatically maintained for you. It uses git author >> data to append people who submitted patches in the past. >> >> For non-commit THANKS, you can still manually edit this file. >> >> This commit removes from the THANKS file the people that appear >> as git commit authors since we started committing them after >> commit 6c976bd. These removed entries get automatically re- >> added by `make`. >> >> >> Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo >> Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/7646794e >> Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/7646794e >> Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/7646794e >> >> Branch: refs/heads/master >> Commit: 7646794ea7a30d41a64c873d8b27ae830397d94a >> Parents: 9514ed3 >> Author: Jan Lehnardt <[email protected]> >> Authored: Sun Jun 17 20:06:21 2012 +0200 >> Committer: Jan Lehnardt <[email protected]> >> Committed: Sun Jun 17 20:11:31 2012 +0200 >> >> ---------------------------------------------------------------------- >> Makefile.am | 10 +++++++++- >> THANKS | 13 +++++-------- >> 2 files changed, 14 insertions(+), 9 deletions(-) >> ---------------------------------------------------------------------- >> >> >> http://git-wip-us.apache.org/repos/asf/couchdb/blob/7646794e/Makefile.am >> ---------------------------------------------------------------------- >> diff --git a/Makefile.am b/Makefile.am >> index 9dd22b8..1cbd865 100644 >> --- a/Makefile.am >> +++ b/Makefile.am >> @@ -79,7 +79,15 @@ README.gz: $(top_srcdir)/README >> -gzip -9 < $< > $@ >> >> THANKS.gz: $(top_srcdir)/THANKS >> - -gzip -9 < $< > $@ >> + sed -e 's/^#.*//' $< > $<.tmp # strip comments >> + sed -e '/^$$/d' $<.tmp > $<.tmp2 # strip empty lines >> + git shortlog -se 6c976bd..HEAD \ >> + | grep -v @apache.org \ >> + | sed -E 's/^[[:blank:]]{5}[[:digit:]]+[[:blank:]]/ * /' \ >> + >> $<.tmp2 # inject git authors >> + echo '\nFor a list of authors see the `AUTHORS` file.\n' >> $<.tmp2 >> + -gzip -9 < $<.tmp2 > $@ # zip >> + rm $<.tmp $<.tmp2 # cleanup >> >> check: dev check-js >> $(top_builddir)/test/etap/run $(top_srcdir)/test/etap >> >> http://git-wip-us.apache.org/repos/asf/couchdb/blob/7646794e/THANKS >> ---------------------------------------------------------------------- >> diff --git a/THANKS b/THANKS >> index f0dccb8..1e065f8 100644 >> --- a/THANKS >> +++ b/THANKS >> @@ -66,7 +66,6 @@ suggesting improvements or submitting changes. Some of >> these people are: >> * Lim Yue Chuan <[email protected]> >> * David Davis <[email protected]> >> * Klaus Trainer <[email protected]> >> - * Dale Harvey <[email protected]> >> * Juuso Väänänen <[email protected]> >> * Jeff Zellner <[email protected]> >> * Benjamin Young <[email protected]> >> @@ -92,12 +91,10 @@ suggesting improvements or submitting changes. Some of >> these people are: >> * Simon Leblanc <[email protected]> >> * Rogutės Sparnuotos <[email protected]> >> * Gavin McDonald <[email protected]> >> - * Ronny Pfannschmidt <[email protected]> >> - * Magnus Hoff <[email protected]> >> - * Alexander Dorofeev <[email protected]> >> - * Anthony S Baker <[email protected]> >> - * Ewan McDougall <[email protected]> >> - * Adam Lofts <[email protected]> >> >> -For a list of authors see the `AUTHORS` file. >> +# Dear committer who merges a commit from a non-committer: >> +# You don't have to manually maintain the THANKS file anymore (yay!). >> +# Non-committer authors get automatically appended to THANKS and >> +# moved into THANKS.gz by `make`. This note will be stripped as well. >> +# Authors from commit 6c976bd and onwards are auto-inserted. >> >>
