From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Git configuration |
Date: | 2010-07-27 16:36:10 |
Message-ID: | 4C4F0AFA.7010609@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Le 27/07/2010 14:12, Dave Page a écrit :
> On Tue, Jul 27, 2010 at 1:10 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>> Le 27/07/2010 13:08, Dave Page a écrit :
>>> On Tue, Jul 27, 2010 at 11:16 AM, Guillaume Lelarge
>>> <guillaume(at)lelarge(dot)info> wrote:
>>>> Le 27/07/2010 11:37, Dave Page a écrit :
>>>>> On Tue, Jul 27, 2010 at 10:30 AM, Guillaume Lelarge
>>>>> <guillaume(at)lelarge(dot)info> wrote:
>>>>>
>>>>>> Did you fix the script? do you need help? I can probably take some time
>>>>>> to look at it.
>>>>>
>>>>> Not yet. Feel free. It's in commitmsg.py iirc:
>>>>>
>>>>> http://github.com/mhagander/pg_githooks
>>>>>
>>>>
>>>> Well, I could be completely wrong but I think you just need to change
>>>> one parameter in hook/policyenforce.ini: committerequalsauthor must be
>>>> equal to 0. If you do this, the hook won't try to match commiter's and
>>>> author's name, but it will still enforce commiters. Which is what we
>>>> want, right?
>>>
>>> That isn't the problem (and committerequalsauthor is already zero).
>>> The problem is that the commitmsg script sends the email "from" the
>>> author, not the committer.
>>>
>>
>> Oh OK. I thought I still couldn't use the --author CLI option. Now, I
>> understand the issue. I'll try to work on this today.
>
> Well you shouldn't, as it will send email in someone else's name until
> this is fixed.
>
Here is a patch to fix this. Can you try it? Thanks.
--
Guillaume
http://www.postgresql.fr
http://dalibo.com
Attachment | Content-Type | Size |
---|---|---|
0001-FROM-mail-address-is-the-commiter-s-one.patch | text/x-diff | 0 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2010-07-27 16:36:47 | Re: 1.10.5 ? |
Previous Message | Dave Page | 2010-07-27 13:01:36 | Re: 1.10.5 ? |