Message1110

Author tonfa
Recipients ThomasAH, alexis, bos, junkblocker, kupfer, mpm
Date 2006-05-02.12:51:07
Content
On 3/31/06, Alexis S. L. Carvalho <mercurial-bugs@selenic.com> wrote:
>
> Two questions about the changes to the commit function:
>
> - With these changes, the precommit hook will run while holding the wlock.
>   Is that a problem?

I don't think it matters, the user probably don't want the working
copy to be modified (eg update/adding/removing/...) in the precommit
hook.
>
> - Should the wlock be passed to the changes function?
>
It should if possible, I'll correct it if the patch is ok.

thanks,

Benoit
History
Date User Action Args
2006-05-02 12:51:22tonfasetrecipients: + mpm, bos, ThomasAH, alexis, junkblocker, kupfer
2006-05-02 12:51:22tonfalinkissue132 messages
2006-05-02 12:51:07tonfacreate