[PATCH 6 of 6] check-concurrency: expose the feature as 'server.check-concurrency'

Augie Fackler raf at durin42.com
Tue Jun 13 18:47:22 EDT 2017


> On Jun 11, 2017, at 16:48, Pierre-Yves David <pierre-yves.david at ens-lyon.org> wrote:
> 
> I slept over this before sending an updated patch and I think there is small bit still missing. The latest proposal is a bit ambiguous in term of value meaning.
> 
> The initial proposal had a "verb" and an "object":
> 
>  check-concurrency=stric # "check" is "strict"
>  check-concurrency=related # "check" if "related".
> 
> The latest one is a bit less clear, since we kinda lost the
> "verb".
> 
>  concurrent-push-mode=strict # "mode" is "strict" (okay)
>  concurrent-push-mode=related # "mode" is "related" what does this mean?
> 
> Maybe we should switch the "logic" of the "value":
> 
>  concurrent-push-mode=related # "unrelated" push can be concurrent
> 
> Or makes it more verbose:
> 
>  concurrent-push-mode=allow-unrelated
> 
> (or)
> 
>  concurrent-push-mode=deny-related
> 
> What do you all think?

That seems like an improvement to me.


More information about the Mercurial-devel mailing list