<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>The core team took care of renaming the files over the weekend, so that reviews going forward should behave as normal. (i.e. Adds are truly additions and modifications are mods).  I looked at the review you referenced and that was file was a proposed addition before the name change.  Git/Gerrit as behaving as expected for this change.</div><div><br></div><div>As for making staged changes, that will run into problems as module paths will be out-of-sync and the change could not pass the gate.  Additionally, that would make it hard for changes to be applied or reverted atomically.</div><div><br></div><div>mark</div><div><br></div><div><div>On Jul 8, 2013, at 3:54 PM, Paul Michali <<a href="mailto:pcm@cisco.com">pcm@cisco.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Many people are converting their code, which is out for review, to neutron. One thing I'm seeing is the files which have been moved and renamed, are showing up as new files (not to pick on it, but as an example  <a href="https://review.openstack.org/#/c/33297/10/neutron/tests/unit/ml2/test_type_gre.py">https://review.openstack.org/#/c/33297/10/neutron/tests/unit/ml2/test_type_gre.py</a>) so it is hard to tell if there are other changes going on.<div><br></div><div>Should submitters ensure that the change set only has renaming changes (and maybe include a note in the patch set saying so), or is there a way that multiple patch sets can be used to make that clear (like one to rename in the text of the file, and a second one to move the file to a new location)?</div><div><br></div><div>Just wondering if there is a way to make reviewing easier…</div><div><br></div><div><br></div><div><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div>PCM (Paul Michali)</div><div><br></div><div>MAIL <a href="mailto:pcm@cisco.com">pcm@cisco.com</a></div><div>IRC   pcm_  (<a href="http://irc.freenode.net/">irc.freenode.net</a>)</div><div>TW   @pmichali</div></span>

</div>

<br></div></div>_______________________________________________<br>OpenStack-dev mailing list<br><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev<br></blockquote></div><br></body></html>