[PATCH 1 of 8] update: add pre-applyupdate hook

Durham Goode durham at fb.com
Mon May 6 15:16:47 CDT 2013


On 5/6/13 12:52 PM, "Matt Mackall" <mpm at selenic.com> wrote:

>On Mon, 2013-05-06 at 12:36 -0700, Durham Goode wrote:
>> # HG changeset patch
>> # User Durham Goode <durham at fb.com>
>> # Date 1367429725 25200
>> #      Wed May 01 10:35:25 2013 -0700
>> # Node ID 4ddd3b2b3407fe3344fd800ea11ab377a34b8ae6
>> # Parent  bfd128c3fbb830b332fec89eddc82b0b9aa31b64
>> update: add pre-applyupdate hook
>> 
>> This adds a hook after update actions are calculated but before they are
>> applied. This allows an extension to download any data necessary for the
>> update if necessary.
>
>Given that 'hooks' are a public interface (on par with the command line)
>and 'actions' are rather ad-hoc guts of the merge code, I think this is
>probably not the right way to expose this.
>

Ok.  I think I can achieve my goal by wrapping merge.applyupdates, so I'll
remove this patch from the series entirely.



More information about the Mercurial-devel mailing list