valentin.gatienbaron added a comment.

  I showed in D6847 <https://phab.mercurial-scm.org/D6847> the same change but 
implemented in exchange._processcompared. Tests pass.
  I think it'd make for a simpler final state: with the current change, the 
client sees that the bookmark is going to move sideways, decides this is fine, 
requests that the server validates that the bookmark is indeed moving sideways 
(which it does), but in the end the server rejects the move. In the suggested 
change, the client sees that the bookmark is going sideways and rejects it. 
This should be consistent with the way new heads or new branches or diverging 
rewrites are prevented.

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6776/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D6776

To: idlsoft, #hg-reviewers, durin42
Cc: pulkit, durin42, valentin.gatienbaron, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to