<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 4:11 AM, Flavio Percoco <span dir="ltr"><<a href="mailto:flavio@redhat.com" target="_blank">flavio@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Greetings,<br>
<br>
Based on the recent discussion that came out about not having enough<br>
information in the commit message when syncing oslo-incubator modules,<br>
I was thinking that besides encouraging people to write better commit<br>
messages, we could also improve the script we use to sync those<br>
modules.<br>
<br>
Some of the changes that I've been thinking of:<br>
<br>
   1) Store the commit sha from which the module was copied from.<br>
   Every project using oslo, currently keeps the list of modules it<br>
   is using in `openstack-modules.conf` in a `module` parameter. We<br>
   could store, along with the module name, the sha of the commit it<br>
   was last synced from:<br>
<br>
       module=log,commit<br>
<br>
       or<br>
<br>
       module=log<br>
       log=commit<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">The second form will be easier to manage. Humans edit the module field and the script will edit the others.</div>
<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
   2) Add an 'auto-commit' parameter to the update script that will<br>
   generate a commit message with the short log of the commits where<br>
   the modules being updated were modified. Soemthing like:<br>
<br>
       Syncing oslo-incubator modules<br>
<br>
       log.py:<br>
           commit1: short-message<br>
           commit2: short-message<br>
           commit3: short-message<br>
<br>
       lockutils:<br>
           commit4: short-message<br>
           commit5: short-message<br>
<br>
#1 will help with figuring out when was the last time a module was<br>
updated and what changes have been introduced since then.<br>
<br>
#2 won't be the recommended way for writing commit messages. Oslo<br>
incubator syncs can have different side-effects in every project.<br>
However, it could be useful for trivial syncs.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It looks like #2 would be a good way to start with commit messages, and just leave it up to the person doing the merge to add the "why" information.</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">

<br>
Thoughts?<br>
<br>
Cheers,<br>
FF<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
@flaper87<br>
Flavio Percoco<br>
</font></span><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>
<br></blockquote></div><br></div></div>