<div dir="ltr"><div dir="ltr">On Fri, Jun 19, 2020 at 8:43 PM Hubert Tong <<a href="mailto:hubert.reinterpretcast@gmail.com">hubert.reinterpretcast@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><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 class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 19, 2020 at 1:26 PM Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</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="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 19 Jun 2020, 18:55 Hubert Tong, <<a href="mailto:hubert.reinterpretcast@gmail.com" target="_blank">hubert.reinterpretcast@gmail.com</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 class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 19, 2020 at 12:32 PM Anton Korobeynikov via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">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">Just my 2 cents here: we are working on enabling this as a part of<br>
bugzilla migration as PRs and issues are very tied inside GitHub. Stay<br>
tuned for updates!<br></blockquote><div>I am not aware that the previous long thread about usage of GitHub PRs in place of Phabricator reviews got anywhere near the point where the option of Phabricator reviews was being dropped. The original post on this thread indicated interest in not maintaining Phabricator. How does that affect the availability of Phabricator? Does this mean that the community is going to move to GitHub PRs because the choice of Phabricator is being taken away?<br></div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I don't think the choice is being taken away, but somebody who believes the cost is worth it has to be willing and able to take on the cost. I can see that that might feel the same if you'd prefer phab but can't shoulder the investment, but I think it's an important difference.</div><div dir="auto"><br></div><div dir="auto">We got Phab back in the day when I started to work on clang and decided that it's not a good use of my time to do email code reviews (and I had to fight a cultural battle to get it :) if somebody thinks the diff of GitHub PR to Phab is worth the ongoing investment & security risks, I'm very happy to hand it over.</div></div></blockquote><div>Thanks for having fought that cultural battle and your work since. I was responding mainly to say that we should not proceed with just assuming that Phabricator does not warrant further investment because GitHub PRs exist.<br></div></div></div></blockquote><div><br></div><div>I do agree. I would urge everyone to look at the incremental benefit as well as the incremental cost though. Phab being better overall (while also clearly being worse in some aspects) doesn't automatically imply that it's a good idea to keep it, given the cost & risk it poses.</div><div> </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 class="gmail_quote"><div></div><div> </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="auto"><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><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 class="gmail_quote"><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On Fri, Jun 19, 2020 at 3:45 PM Manuel Klimek via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
> -Chris' outdated email, +Chris' correct email :)<br>
><br>
> On Fri, Jun 19, 2020 at 2:01 PM Manuel Klimek <<a href="mailto:klimek@google.com" rel="noreferrer" target="_blank">klimek@google.com</a>> wrote:<br>
>><br>
>> Hi folks,<br>
>><br>
>> phabricator maintenance is a topic that has been lying dormant for a while now.<br>
>><br>
>> That subsequently creates a non-optimal user experience.<br>
>> For me personally, given that github provides a secure PR infrastructure, the additional effort required to keep Phab going is not an investment I'm personally willing to make. I understand that there are unique selling points for Phab in its UI compared to github PRs, but there are also significant downsides in the effort to integrate with Phab that github PRs make easier.<br>
>><br>
>> Thus, I see two options:<br>
>> 1. somebody volunteers to take on Phabricator maintenance and figures out a way to fund it, either through the LLVM foundation or some other means (I'd love for us at Google to pay for it directly and give folks outside Google access, but that is unfortunately a hard problem for a variety of reasons). I'd be happy to help to provide a DB snapshot for the migration, of course.<br>
>><br>
>> 2. We switch to github PRs<br>
>><br>
>> Thoughts?<br>
>> /Manuel<br>
>><br>
>><br>
>> On Thu, Jun 18, 2020 at 6:42 PM Raphael Isemann <<a href="mailto:teemperor@gmail.com" rel="noreferrer" target="_blank">teemperor@gmail.com</a>> wrote:<br>
>>><br>
>>> Friendly ping<br>
>>><br>
>>> Am Do., 9. Apr. 2020 um 16:04 Uhr schrieb Alexandre Ganea<br>
>>> <<a href="mailto:alexandre.ganea@ubisoft.com" rel="noreferrer" target="_blank">alexandre.ganea@ubisoft.com</a>>:<br>
>>> ><br>
>>> > cc Paul / MyDeveloperDay<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > De : llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> De la part de David Blaikie via llvm-dev<br>
>>> > Envoyé : April 8, 2020 10:21 PM<br>
>>> > À : Raphael “Teemperor” Isemann <<a href="mailto:teemperor@gmail.com" rel="noreferrer" target="_blank">teemperor@gmail.com</a>>; Manuel Klimek <<a href="mailto:klimek@google.com" rel="noreferrer" target="_blank">klimek@google.com</a>><br>
>>> > Cc : llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev@lists.llvm.org</a>><br>
>>> > Objet : Re: [llvm-dev] Outdated Phabricator version on <a href="http://reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">reviews.llvm.org</a> breaks Google authentication since today<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > hey Manuel - are you/do you know who's likely to be doing any upkeep on Phabricator these days? Might need an update for this...<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > - Dave<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > On Wed, Apr 8, 2020 at 5:57 AM Raphael “Teemperor” Isemann via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>>> ><br>
>>> > Hi all,<br>
>>> ><br>
>>> > I’m using my Google account to log into my Phabricator account on <a href="http://reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">reviews.llvm.org</a> . Since today that no longer works as I don’t seem to get any reply from <a href="http://reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">reviews.llvm.org</a> when I’m logged into my account. It tried logging out which fixes the issue of <a href="http://reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">reviews.llvm.org</a> not loading, but when I try to login I just get the following error:<br>
>>> ><br>
>>> > > Expected to retrieve an "account" email from Google Plus API call to identify account, but failed.<br>
>>> ><br>
>>> > After some searching it seems that this error is due to the Google Plus API being shutdown and the Phabricator folks replaced that logic (including this error message string) a year ago here [1]<br>
>>> ><br>
>>> > I assume we haven’t updated <a href="http://reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">reviews.llvm.org</a> to whatever latest Phabricator release contains that patch.<br>
>>> ><br>
>>> > Not sure who’s currently responsible for updating <a href="http://reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">reviews.llvm.org</a> so I thought I’ll just drop a mail to the list (and maybe save someone else from figuring out why their login is suddenly broken).<br>
>>> ><br>
>>> > [1] <a href="https://secure.phabricator.com/D20030" rel="noreferrer noreferrer" target="_blank">https://secure.phabricator.com/D20030</a><br>
>>> > _______________________________________________<br>
>>> > LLVM Developers mailing list<br>
>>> > <a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev@lists.llvm.org</a><br>
>>> > <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br>
<br>
<br>
-- <br>
With best regards, Anton Korobeynikov<br>
Department of Statistical Modelling, Saint Petersburg State University<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" rel="noreferrer" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>
</blockquote></div></div></div>
</blockquote></div></div>
</blockquote></div></div>