<div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>I think it would be great to document "how to revert a patch" in `GettingStarted` page.</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr">Best Regards,<div>Xing</div><div><div><br></div></div></div></div></div></div></div></div></div></div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 29, 2019 at 12:30 AM Jordan Rupprecht 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"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi all,<div>Last week I had to revert a change during my build cop shift, and discovered the usual script I used, git-svnrevert, only works w/ git-svn repos, and not the pure github repo, meaning I had to:</div><div>1) Find the git commit hash I needed to revert (given the svn revision that broke it)</div><div>2) Run "git revert" myself, and then I forgot to include the svn id (the git revert default message just includes the git commit hash, and I forgot to change that)</div><div><br></div><div>It doesn't make sense to do that manually, so I committed r357180 to make a new git subcommand do that: "git llvm revert r123456" should now work. "git llvm revert -n r123456" will do nothing but print the commands (if you don't trust it yet). Also, since it might be useful outside of the revert flow, a second subcommand "git llvm svn-lookup <git-hash>" will pull the svn id out of the git commit message.<br></div><div><br></div><div>If you're developing with the pure github repo (not the git-svn method, and not the even older pure-svn method), please try it out next time you have to revert. Patches welcome if you think something could be improved.</div><div><br></div><div>-- Jordan</div></div></div></div>
_______________________________________________<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>