<p dir="ltr">I think that trying to create a ordering/rev number between independent git repositories is fundamentally unreliable.</p>
<p dir="ltr">If we want to keep llvm and clang in lock step we should probably probably just have them in the same repository like <a href="https://github.com/llvm-project/llvm-project">https://github.com/llvm-project/llvm-project</a>.</p>
<p dir="ltr">Cheers,<br>
Rafael </p>
<div class="gmail_quote">On Jun 27, 2016 11:10 AM, "Renato Golin" <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 27 June 2016 at 15:39, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> So, I probably missed something, but what was the main objection to<br>
> just using submodules? This would put llvm inside clang instead of the<br>
> other way around. When changing an API one currently has to<br>
<br>
I don't think the consensus was to change the order of inclusion (llvm<br>
into clang), but to *not* change anything else at this stage.<br>
<br>
That's one of the reasons the umbrella project with sub-modules was<br>
the most accepted solution, because we can later change the inclusion<br>
order (if we all agree, etc), without changing the underlying storage.<br>
<br>
<br>
> * Change llvm.<br>
> * Change clang and in the same atomic commit change what revision the<br>
> submodule points to.<br>
<br>
Having one repository inside another was rejected due to the problems<br>
it brings for development, validation and release. James has just<br>
pointed a few of those problems for development.<br>
<br>
An umbrella project with a commit hook and an auto-update would make<br>
sure all commits are synchronised correctly. Though, indeed, this will<br>
mean we'll still have the trouble of buildbots picking up one commit<br>
and not the other, I don't think this is a big enough problem that we<br>
should mess up everyone's workflow.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div>