<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 22, 2013 at 11:21 AM, Duncan Thomas <span dir="ltr"><<a href="mailto:duncan.thomas@gmail.com" target="_blank">duncan.thomas@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 22 November 2013 12:27, Elena Ezhova <<a href="mailto:eezhova@mirantis.com">eezhova@mirantis.com</a>> wrote:<br>

> But what if I want to update some module that consists of ten or even more<br>
> files (like rpc or db) and each of these files has quite a long change log?<br>
> In that case the commit message may turn out to be really long even if only<br>
> commit ids and names are included.<br>
<br>
<br>
</div>A message that is too long is definitely a better problem to have than<br>
a message that is missing important details.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">If we are talking about merging only part of oslo into a consuming project, then we can't just keep track of the "last" revision that was merged, because that won't necessarily include all of the changes. Elena, were you planning to keep a separate revision for each entry under openstack/common (not every file, just the files and directories at that level)?</div>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
To turn the question round, how can a reviewer review a change that<br>
includes ten or even more files without any information on what<br>
changed and why? rpc and db are extremely difficult imports to review,<br>
and I've found problems in the last two I looked at.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Problems in the code, or in the way the code was merged?</div><div class="gmail_default" style="font-size:small">
<br></div><div class="gmail_default" style="font-size:small">Doug</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>