[lldb-dev] [cfe-dev] [RFC] Deprecate email code reviews in favor of Phabricator

Aaron Ballman via lldb-dev lldb-dev at lists.llvm.org
Tue May 4 07:01:33 PDT 2021


On Tue, May 4, 2021 at 9:56 AM <paul.robinson at sony.com> wrote:
>
> > You're right that doing post-commit reviews on Phabricator is not
> > seamless---the rG link is not included anywhere.  Hopefully that could be
> > fixed with some Phabricator configuration tweaks, like sending the commit
> > email to the -commits list.
>
> The commit email has a URL: link, e.g. this recent one (which has no
> Dnnnnn review):
>
> URL: https://github.com/llvm/llvm-project/commit/b04148f77713c92ee57b33b7b858ad18ce8d78e3
>
> Does that take you to a different place than the rG link would?
> Seems like they ought to go to the same place.

That sends you to the GitHub commit for the change. The corresponding
rG link would be:

https://reviews.llvm.org/rGb04148f77713c92ee57b33b7b858ad18ce8d78e3

~Aaron


More information about the lldb-dev mailing list