<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jun 7, 2018 at 1:10 PM, Eric Rescorla <span dir="ltr"><<a href="mailto:ekr@rtfm.com" target="_blank">ekr@rtfm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class=""></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
BZ, I'd actually like to ask you a question. When you are working<br>
on a series of changesets and someone askes you for a revision of<br>
one of those changesets, do you expect to add a new commit on top<br>
of the commit in the changeset, or rewrite it the existing commit?<br>
</blockquote>
<br></span>
It depends.<br>
<br>
Full disclosure, I use mq for the most part.<br></blockquote><div><br></div></span><div>I hear we're not supposed to do that :)</div></div></div></div></blockquote><div><br></div><div>FWIW, I don't use mq, but my workflow is almost identical to bz's.</div><div><br></div><div>If somebody asks me to make a change to a patch in the middle of the series, there are two cases:</div><div> - If I know that this area of the code is not touched by any patches on top of the one I want to modify, then I just commit the change on top of the series, and then use hg histedit to roll the change into the patch in the middle.</div><div> - Otherwise, I will "hg up" to the relevant patch, make the change, use "hg amend" to amend it into the patch, and then use "hg evolve" to rebase the other patches on top of the modified patch.</div><div> - If I fear that I might be making a mistake in this process and want to checkpoint my work, then I will commit this change separately, and then fold the changes once I'm done and everything works.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div></div></span><div>but I don't think we should require it as the only way to roll, especially because with Git it kind of messes with your history in an annoying way (in that it's hard to then get back to the history of each CL).<br></div></div></div></div></blockquote><div><br></div><div>As for myself, I'm not attached to have my local hg history exactly match the series-of-reviewable-units. However, if we do end up with a script to push an entire commit series to review (and I think such a thing is absolutely needed), simply converting one commit to one reviewable unit seems like the most straightforward implementation.<br></div><div> </div><div>Markus</div><br></div></div></div>