Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@couchdb.apache.org
Received-SPF: pass (nike.apache.org: domain of paul.joseph.davis@gmail.com
 designates 209.85.220.180 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <95997D68-0407-4C9E-A7A6-1C9FF79D5374@apache.org>
References: <20120617182914.2F9E811F6E@tyr.zones.apache.org>
 <CAJ_m3YDOe6EhTpF_tRcH2BVikUf6BKOgbn+vzcx1EDN33kOrcw@mail.gmail.com>
 <4B807C66-3C26-48E1-B3C4-A7C86DCD5C5B@apache.org>
 <CAJ_m3YCYDb_yfWsFkX7EN_yP9qawJ1O8uKBudCbD0f3ivk=m1g@mail.gmail.com>
 <95997D68-0407-4C9E-A7A6-1C9FF79D5374@apache.org>
From: Paul Davis <paul.joseph.davis@gmail.com>
Date: Sun, 17 Jun 2012 15:05:35 -0500
Message-ID: 
 <CAJ_m3YBNoEMb0b9aS6Qv=b2Mi6KQ6xomTmikf95dNR3CTxozcg@mail.gmail.com>
Subject: Re: git commit: Automate maintenance of the THANKS file
To: dev@couchdb.apache.org
Content-Type: text/plain; charset=ISO-8859-1

On Sun, Jun 17, 2012 at 3:02 PM, Jan Lehnardt <jan@apache.org> wrote:
>
> On Jun 17, 2012, at 21:56 , Paul Davis wrote:
>
>> On Sun, Jun 17, 2012 at 2:44 PM, Jan Lehnardt <jan@apache.org> wrote:
>>>
>>> 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! :)
>>>
>>
>> Its an extra step but not one that I find to be particularly onerous.
>> Given that we're already working on codifying merge practices I don't
>> see why we don't just add a check box for "includes commit adding
>> yourself to the THANKS file if this is your first contribution" that
>> we look for.
>
> That's a fair point, but this has annoyed me forever.
>
>>>> 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.
>>>
>>
>> I'm not sure its a good idea to have a file that is only built
>> correctly in special circumstances.
>
> I'm happy to add an rm -f $< to the target.
>
>
>>>> 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.
>>>
>>
>> I'm confused. You've removed one manually curated file only to add a
>> new one that just modifies the build of the first? Seems like a lot of
>> gymnastics.
>
> .mailmap solves more than just this.
>
>
>> In a perfect world I would be all in with you on this but
>> unfortunately a large number of people don't spend time checking their
>> user settings before pushing commits around. Instead of just adding
>> people to a file the first time they make a commit this means I have
>> to go and check that the THANKS file is generated properly and then
>> maybe update .mailmap if not and recheck that I got it correct.
>
> Fair enough, wanna revert?
>
> Cheers
> Jan
> --
>
>

Playing with it a bit to see if I can make it build correctly and also
just build the AUTHORS file. I'll leave it around for a bit but won't
promise that the first time I spend more than 30s screwing with
mailmap that I revert it.