<div dir="ltr">I went for years without using arc.  The workflow was basically:<div><br></div><div>1) git format-patch -U999999 HEAD~1</div><div>2) Go to the phabricator website and click Create Patch</div><div>3) Click Browse and find the patch file that was emitted in step 1.</div><div>4) Copy/paste the description of the patch from step 1 into the box.</div><div>5) Add reviewers.</div><div><br></div><div>It doesn't sound like the proposed script saves you all that many steps.  </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 20, 2020 at 6:16 PM 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>