<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 21, 2020 at 1:45 AM Christian Kühnel via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi folks,</div><div><br></div>Another thought on the topic is tooling support and tool integration: <div>There is a hughe ecosystem around Github and very little around Phabricator.<div><br><div>It took me 2 days to set up build jobs for the 10.x release branch [1]. There are nice build integrations for Github and it was smooth sailing. Setting up a build job for pull requests would just be a few clicks now.</div><div><br></div><div>In contrast I've been spending weeks on setting up a proper integration of pre-merge tests [2] with Phabricator. And I'm far from having a proper solution and I know some corner cases we will never be able to solve properly with Phabricator. The root cause is: On Phabricator you review patches that might or might not be related to a git revision and the process for merging/landing might introduce even more changes. This is a fundamental problem with the data model and nothing that can be fixed easily.</div><div><br></div><div>In Github pull requests there is always a git commit that you can just feed to the build server. And you can be sure of what really gets merged. You review, build and test exactly the change that gets merged afterwards.</div></div></div></div></div></div></blockquote><div><br></div><div>How would that be true? Given that upstream keep changing during the period of review? The commit is going to have to be rebased to head and that may involve making changes.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>So from the build server perspective, Github is clearly the better solution.<br></div><div><br></div><div><div><div>[1] <a href="https://buildkite.com/llvm-project/llvm-release-builds" target="_blank">https://buildkite.com/llvm-project/llvm-release-builds</a> </div></div><div>[2] <a href="https://github.com/google/llvm-premerge-checks/blob/master/docs/user_doc.md" target="_blank">https://github.com/google/llvm-premerge-checks/blob/master/docs/user_doc.md</a></div></div><div>Best,</div><div>Christian</div><div><br><br></div></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail_quote"></div></blockquote></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>