<div dir="ltr">I like the concept, and a quick glance at the script is essentially what I'd expect, though I haven't looked at it in depth. I'm quite okay with jumping through the current hoops needed to do those three items, but they are also my most common operations, so a script would definitely simplify things for me. Unfortunately, I don't currently have the time to invest in it myself.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 21 Jan 2020 at 02:16, Florian Hahn 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">Hi,<br>
<br>
One takeaway for me from the recent Phabricator vs Github PR discussions was that arc (arcanist) can be a pain to set up and may pose a hurdle for some contributors.<br>
<br>
I think those points could be addressed relatively easily by adding a llvm-patch script (or an even better name) that allows users to create and apply patches from <a href="http://reviews.llvm.org" rel="noreferrer" target="_blank">reviews.llvm.org</a> using Phabricators API. In my experience, the three most common uses cases are:<br>
<br>
1. Create a new review from the current HEAD commit in the working directory and let me optionally add subscribers/reviewers<br>
2. Update the diff for an existing review with the current HEAD commit in the working directory<br>
3. Download the latest diff for a revision and apply it to the working directory, using the commit message from Phabricator.<br>
<br>
Those should be fairly easy to implement and as a proof-of-concept I went ahead and put up a patch implementing 3. from the list above: <a href="https://reviews.llvm.org/D73075" rel="noreferrer" target="_blank">https://reviews.llvm.org/D73075</a> .<br>
<br>
 Please note that the script is probably a bit rough around the edges and I probably won’t have time to implement 1. and 2. in the near future on my own, but maybe someone would be interested in helping out.<br>
<br>
I think that could improve the experience for new contributors substantially: <br>
 * Create a new patch: `llvm-patch upload`<br>
 * Apply a patch from a review: `llvm-patch apply D12345`. <br>
<br>
WDYT?<br>
<br>
Cheers,<br>
Florian<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>