<div dir="ltr">And make sure the Differential Revision line has exactly one space between the colon and the URL.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 28, 2021 at 8:01 AM 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"><br>
<br>
> -----Original Message-----<br>
> From: llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> On Behalf Of Paul C.<br>
> Anagnostopoulos via llvm-dev<br>
> Sent: Thursday, January 28, 2021 10:45 AM<br>
> To: <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> Subject: [llvm-dev] Pushing a patch for someone else<br>
> <br>
> Could someone explain the best way to push a patch for someone else? I<br>
> don't use Arcanist, but add the Differential Revision line by hand. Please<br>
> include the method of downloading the person's patch to my machine.<br>
<br>
It's a bit of a manual process given that we don't use branches<br>
and a pull-request model as GitHub favors, but it's not hard.<br>
I don't think I've done this myself since the move to github so<br>
I might be missing a step, hopefully others will chime in to fix<br>
my mistakes.<br>
<br>
Assuming the patch is in a Phabricator review, there should be a <br>
"Download Raw Diff" link on the right-hand side at the top.  This,<br>
oddly enough, should download a diff file, which you can then apply<br>
to your working tree with "patch" or similar utility.<br>
<br>
What I do then is copy the summary into a text file, making sure <br>
the formatting looks reasonable.  You want to add two lines to it:<br>
<br>
Patch by <name of author><br>
<br>
Differential Revision: <a href="https://reviews.llvm.org/Dxxxxx" rel="noreferrer" target="_blank">https://reviews.llvm.org/Dxxxxx</a><br>
<br>
filling in the author and actual review number.  Make sure there's<br>
a blank line before the Differential Revision line.<br>
<br>
Then you commit this as usual, but add the --author option to<br>
provide (I think) the email address of the author.  (This might be<br>
redundant with the "Patch by" information, so the current practice<br>
might be to leave out "Patch by", not sure.)  And of course -F to<br>
specify the file where you put the commit message.<br>
<br>
Once it's committed, do the usual build/test/push.<br>
<br>
HTH,<br>
--paulr<br>
<br>
> <br>
> Needless to say, I've never done this before.<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://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__;!!JmoZiZGBv3RvKRSx!tzZMDG77DN4la7BtUuhj60VZ9kl3C0IjWwiVXob43-<br>
> plBKeksKvrSBLMQZZzx0ZT0Q$<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>