<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 2, 2017 at 7:16 AM, Denis Laxalde <span dir="ltr"><<a href="mailto:denis@laxalde.org" target="_blank">denis@laxalde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Augie Fackler a écrit :<span class="gmail-"><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On Nov 2, 2017, at 09:39, Martin von Zweigbergk via Mercurial-devel <<a href="mailto:mercurial-devel@mercurial-scm.org" target="_blank">mercurial-devel@mercurial-scm<wbr>.org</a>> wrote:<br>
<br>
Can we still change the behavior of "hg revert -i -r" to show a to-apply diff, not a to-revert diff? (There's a bug number I'm too lazy to look up from mobile.)<br>
</blockquote>
<br>
I thought we had already done the patch-reversing that we felt was required...<br>
</blockquote>
<br></span>
The last discussion ended with a status quo:<br>
<br>
<a href="https://www.mercurial-scm.org/pipermail/mercurial-devel/2016-November/090142.html" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/<wbr>pipermail/mercurial-devel/2016<wbr>-November/090142.html</a><br>
<br>
Since then, I set "experimental.revertalternatei<wbr>nteractivemode=false" to<br>
have a behavior that I find meaningful in most situations.</blockquote><div><br></div><div>So do I. The problem is that new users won't have that. If we're graduating --interactive now, then this seems like a good time (at the latest) to switch the default of that flag.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> The only case<br>
it does not work well is "hg revert -i -r .^" (which I think was a major<br>
motivation for the current behavior).</blockquote><div><br></div><div>I prefer the forward direction even when reverting to a parent. We could add --no-forward-patch flag or something, or we could add a "hg grab" that grabs the file content from a revision and is equivalent to "hg revert" except that the patch is always forward. But last time I suggested that, no one seemed interested, so I'm not very optimistic. So probably just switch the default of revertalternatei<wbr>nteractivemode to false?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> On the other hand, now "hg<br>
uncommit" in evolve extension has a --interactive flag that could be<br>
used instead.<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On Thu, Nov 2, 2017, 05:47 Yuya Nishihara <<a href="mailto:yuya@tcha.org" target="_blank">yuya@tcha.org</a>> wrote:<br>
On Thu, 2 Nov 2017 10:03:44 +0100, Denis Laxalde wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Augie Fackler a écrit :<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
# HG changeset patch<br>
# User Augie Fackler <<a href="mailto:augie@google.com" target="_blank">augie@google.com</a>><br>
# Date 1509399967 14400<br>
# Mon Oct 30 17:46:07 2017 -0400<br>
# Node ID a2fc2086ac3116c63ab890180697b2<wbr>695c50d9f4<br>
# Parent 3ce0e4b51f789eff195ec900a07c1f<wbr>a5e8d5c5f2<br>
revert: no longer mark --interactive as experimental<br>
<br>
We seem to have settled down on behavior changes here (nothing<br>
matching revset `keyword(revert) and keyword(interactive)` since 4.2<br>
was released), so let's go ahead and advertise this excellent feature.<br>
</blockquote>
<br>
+1<br>
</blockquote>
<br>
+1, too. Queued, thanks.<br>
______________________________<wbr>_________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@mercurial-scm.org" target="_blank">Mercurial-devel@mercurial-scm.<wbr>org</a><br>
<a href="https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/<wbr>mailman/listinfo/mercurial-dev<wbr>el</a><br>
______________________________<wbr>_________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@mercurial-scm.org" target="_blank">Mercurial-devel@mercurial-scm.<wbr>org</a><br>
<a href="https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/<wbr>mailman/listinfo/mercurial-dev<wbr>el</a><br>
</blockquote>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>