Re: [PATCH 2 of 3 🚂] changegroup: use the `postclose` API on transaction

[Gregory Szorc]

> On Nov 7, 2014, at 8:48, Pierre-Yves David <pierre-yves.david at ens-lyon.org> wrote:
> 
> 
> 
>> On 11/07/2014 04:41 PM, Gregory Szorc wrote:
>> -  remote: changegroup hook:
>> 
>> Does this mean that clients pushing will no longer see output from changegroup hooks? That's not BC.
> 
> No, the context is important.
> 
>     remote: transaction abort!
>     remote: rollback completed
>  -  remote: changegroup hook:
> 
> This mean we'll no longer run the changegroup hook after aborted transaction. Rephrased, we do not run changegroup hook for non-added changegroup.

Mobile phone formatting bites again. Thanks for correcting me.