<div dir="ltr"><div>Mark, </div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">>> What do you mean by "dangerous code merging" in the subject? The body of</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">>> your mail doesn't make any reference to whatever "danger" you're seeing.</span><br style="font-family:arial,sans-serif;font-size:13px"></div>
<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div>I mean that cut and paste approach is really unsafe. For example some new member is able to change oslo code directly during syncing with some project,<div>
 and nobody will be able to catch such things.</div><div><br></div><div>I didn't catch any of such situation, but I saw a lot of attempts to change openstack/common/* directly.</div><div>(and it is really close situation..)</div>
<div><br></div><div><br></div><div><div><span style="font-family:arial,sans-serif;font-size:13px">>> The idea of using submodules has come a few times. I don't have a</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">>> fundamental objection to it, except any time I've seen submodules used</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">>> in a project they've been extremely painful for everyone involved.</span><br>
</div><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><font face="arial, sans-serif">oslo-incubator sync util and submodules solves the same problem, almost in same way: </font></div><div>
<font face="arial, sans-serif">sync util -> copy paste code from <hash></font></div><div><font face="arial, sans-serif">submodules -> just set <hash> of commit from what to use code</font></div><div><br>
</div><div><font face="arial, sans-serif">So I think the problem is not in submodules, problem is in approach of common code for different projects.</font></div><div><font face="arial, sans-serif">But IMHO it is much better to have problems around creating common code that is used by all projects, then to make</font></div>
<div><font face="arial, sans-serif">N different solutions for N different projects doing almost the same things. </font></div><div><br></div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">>> I'd be happy to look at a demo of a submodule based system for projects</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">>> to use code from oslo-incubator.</span><br style="font-family:arial,sans-serif;font-size:13px"></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br>
</span></div><div><font face="arial, sans-serif">Probably we should just try, and analyze what approach is better?</font></div><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">Best regards,</font></div>
<div><font face="arial, sans-serif">Boris Pavlovic</font></div><div><font face="arial, sans-serif">--</font></div><div><font face="arial, sans-serif">Mirantis Inc. </font></div><div><br></div><div><br></div></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Aug 8, 2013 at 2:39 PM, Mark McLoughlin <span dir="ltr"><<a href="mailto:markmc@redhat.com" target="_blank">markmc@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What do you mean by "dangerous code merging" in the subject? The body of<br>
your mail doesn't make any reference to whatever "danger" you're seeing.<br>
<div class="im"><br>
On Thu, 2013-08-08 at 14:16 +0400, Boris Pavlovic wrote:<br>
> Hi All,<br>
><br>
> Could somebody answer me, why we are merging oslo code in other projects<br>
> and don't use<br>
> git submodules (<a href="http://git-scm.com/book/en/Git-Tools-Submodules" target="_blank">http://git-scm.com/book/en/Git-Tools-Submodules</a>)<br>
<br>
</div>The idea of using submodules has come a few times. I don't have a<br>
fundamental objection to it, except any time I've seen submodules used<br>
in a project they've been extremely painful for everyone involved.<br>
<br>
I'd be happy to look at a demo of a submodule based system for projects<br>
to use code from oslo-incubator.<br>
<br>
Mark.<br>
<br>
<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>
</blockquote></div><br></div>