<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 26, 2020 at 12:41 PM Danila Malyutin 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-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">For the record,<br>
Phabricator is even worse for that purpose as it doesn't really guarantee that what's been reviewed is actually what will be committed.<br>
In fact, I've got bit by that before svn->git transition (error in reapplying the patch on svn for submission).<br>
GitHub PRs solve this problem. In fact, there wouldn't be any need for the "getting commit rights" process and all those pings from the new contributors, since it'd be easy to make PR mergeable by anyone as long once they are approved.<br></blockquote><div><br></div><div>I think merging a PR on GitHub requires permission on the repo: a new contributor would require someone with "commit rights" to click on the merge button for them.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Although some of the friction disappeared after the move to git and after I discovered moz-phab.<br>
<br>
--<br>
Danila<br>
<br>
> -----Original Message-----<br>
> From: llvm-dev [mailto:<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>] On Behalf Of Nicolai<br>
> Hahnle via llvm-dev<br>
> Sent: Thursday, June 25, 2020 18:10<br>
> To: Nikita Popov <<a href="mailto:nikita.ppv@gmail.com" target="_blank">nikita.ppv@gmail.com</a>><br>
> Cc: LLVM Dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>; Clang Dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>><br>
> Subject: Re: [llvm-dev] [cfe-dev] Phabricator Maintenance<br>
> <br>
> On Thu, Jun 25, 2020 at 11:43 AM Nikita Popov via llvm-dev <llvm-<br>
> <a href="mailto:dev@lists.llvm.org" target="_blank">dev@lists.llvm.org</a>> wrote:<br>
> > On Thu, Jun 25, 2020 at 11:22 AM Zachary Turner via llvm-dev <llvm-<br>
> <a href="mailto:dev@lists.llvm.org" target="_blank">dev@lists.llvm.org</a>> wrote:<br>
> >> What this means for LLVM is that everyone will have to completely stop using<br>
> history rewriting operations.  No more rebase, squash, amend, etc.<br>
> ><br>
> > This is also incorrect. Most GitHub projects I work on use a rebase-oriented<br>
> workflow without issue, so clearly that's possible...<br>
> ><br>
> > The only issue in this area that I'm aware of is that you can't easily see what<br>
> changed between two revisions of a pull request if you both a) rebase onto<br>
> master and b) amend commits at the *same* time. For review purposes it is<br>
> most useful if additional changes are not squashed into existing commits, but<br>
> pushed on top. The squash is best performed when landing the changes.<br>
> <br>
> Disagree. I usually want to be able to review the actual commits that will be<br>
> landed, not some imagined combination that may or may not line up with what<br>
> the author will end up squashing things to.<br>
> <br>
> Remember, the goal is for authors to be able to have a review of multiple<br>
> commits in a patch series, i.e.<br>
> <br>
>    Patch 1<br>
>    Patch 2<br>
> <br>
> ... and then make changes to both of those patches based on reviews.<br>
> If you just push delta commits, then in the best case, if people properly use `git<br>
> commit --squash`, you get:<br>
> <br>
>    Patch 1<br>
>    Patch 2<br>
>    squash! Patch 1<br>
>    squash! Patch 2<br>
> <br>
> or something along those lines, and things just become really messy to keep<br>
> track of. The Phabricator Stacks UI isn't ideal for this, but it tends to work quite a<br>
> bit better than what GitHub has.<br>
> <br>
> Cheers,<br>
> Nicolai<br>
> --<br>
> Lerne, wie die Welt wirklich ist,<br>
> aber vergiss niemals, wie sie sein sollte.<br>
> _______________________________________________<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://urldefense.com/v3/__https://lists.llvm.org/cgi-" rel="noreferrer" target="_blank">https://urldefense.com/v3/__https://lists.llvm.org/cgi-</a><br>
> bin/mailman/listinfo/llvm-<br>
> dev__;!!A4F2R9G_pg!OD1Fs4auqBkrZx526L0axvXPTf5YH9MrHI_UReeJz9VA6Nu-<br>
> VQi8Ibwu6fVE0w0$<br>
_______________________________________________<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>