<div><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"></div></div></div><div><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 3, 2020 at 1:44 PM Louis Dionne via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">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"><br>
<br>
> On Feb 20, 2020, at 14:25, Johannes Doerfert <<a href="mailto:johannesdoerfert@gmail.com" target="_blank">johannesdoerfert@gmail.com</a>> wrote:<br>
> <br>
> On 02/20, Louis Dionne via llvm-dev wrote:<br>
>> I know there has been significant discussion about "moving" from<br>
>> Phabricator to GitHub reviews and pull requests, etc. I'm not<br>
>> suggesting that we do anything in terms of global LLVM policy.<br>
>> However, as a maintainer of libc++, I commit __a lot__ of other<br>
>> people's code for them. It would be a huge time saver for me if I<br>
>> could nicely suggest to contributors (not force them) to use PRs<br>
>> instead of Phabricator for their contributions. It would also handle<br>
>> commit attribution properly, which is a pain right now.<br>
> <br>
> Don't take this as me telling you it is "actually simple". I am<br>
> interested what about the contribution is problematic? If the libc++<br>
> system doesn't have more requirements than the rest of LLVM there might<br>
> be ways to make it less painful. FWIW, here is what I do, and I know not<br>
> everyone wants to use `arc`. Ina script this could potentially reduce<br>
> the pain. Again, this is not meant to tell you it is simple or your<br>
> problems are not real.<br>
> <br>
> arc patch DXXXX<br>
> git pull --rebase origin master<br>
> arc amend<br>
> arcfilter        // see below<br>
> git llvm push master<br>
> <br>
> <br>
> arcfilter () { git log -1 --pretty=%B | awk '/Reviewers:|Subscribers:/{p=1} /Reviewed By:|Differential Revision:/{p=0} !p && !/^Summary:/' | git commit --amend -F - }<br>
<br>
Thanks, this indeed solves some of my problems, however not entirely. When people submit contributions without an email address, I still have to do some digging to find out how to attribute the change. This shouldn't be something I even have to think about.<br></blockquote><div><br></div></div></div></div><div><div dir="ltr"><div class="gmail_quote"><div>I usually ask the patch author in the review how they want the attribution.</div></div></div></div><div><div dir="ltr"><div class="gmail_quote"><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">
<br>
Louis<br>
<br>
> <br>
>> Would it be possible to allow GitHub PRs to be submitted on the<br>
>> monorepo so as to let individual sub-projects deal with it however<br>
>> they please? I've spoken to numerous people involved in libc++<br>
>> development and they would like to start submitting PRs (and for the<br>
>> others, we'll still accept Phabricator reviews). Perhaps it is<br>
>> possible to setup some kind of filter such that PRs touching only<br>
>> libcxx/ and libcxxabi/ can be submitted, but otherwise they're closed<br>
>> by the bot?<br>
> <br>
> TBH, I feel this is yet another way of splitting the community and in<br>
> the end complicating things even more. I mean, since recently if you<br>
> want to ask a question there were the *-dev lists and the IRC. Now we<br>
> have discourse, discord on top of that with some people monitoring only<br>
> one of these and others required to monitor both. Duplicating the way we<br>
> do reviews is similarly going to require people that want to be informed<br>
> to duplicate their lookups.<br>
> <br>
> Cheers,<br>
>  Johannes<br>
> <br>
<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>
</div>