[cfe-dev] [llvm-dev] FYI: LLVM Phabricactor notifications.

[Aaron Ballman via cfe-dev]

On Fri, May 31, 2019 at 4:18 AM Manuel Klimek <klimek at google.com> wrote:
>
>
>
> On Fri, May 31, 2019 at 10:11 AM Aaron Ballman <aaron.ballman at gmail.com> wrote:
>>
>>
>>
>> On Fri, May 31, 2019, 10:02 AM Roman Lebedev via llvm-dev <llvm-dev at lists.llvm.org> wrote:
>>>
>>> On Fri, May 31, 2019 at 10:55 AM Manuel Klimek via llvm-dev
>>> <llvm-dev at lists.llvm.org> wrote:
>>> >
>>> > We should be able to control the noise of these
>>>
>>> >, but I also wonder whether we can switch to github reviews as part of the git move :)
>>> (Provocative offtopic. (I personally hope that never ever happens.))
>>
>>
>> Strong +1 to not switching away from phab.
>
>
> Aaron, do you have admin on phab yet? If yes, would you mind to config the notifications?
> (honestly, part of my comment was trying to figure out who wants it so badly that they'd be willing to help with maintenance - I have admin bits to hand out :)

I don't have admin on Phab but I would be happy to help in any way
needed. I'm only mildly familiar with the app (mostly had to deal with
basic config and setup), however, so I may not be the best admin.

~Aaron

>
>>
>>
>> ~Aaron
>>
>>>
>>> > (we just lost our long time phab maintainer on the team)
>>>
>>> > On Fri, May 31, 2019 at 12:44 AM Shoaib Meenai via llvm-dev <llvm-dev at lists.llvm.org> wrote:
>>> >>
>>> >> I believe (and I believe it was James who pointed this out on IRC) that Phabricator pulls in all refs, and GitHub stores PR commits under refs/pull.
>>> >>
>>> >> On 5/30/19, 3:37 PM, "llvm-dev on behalf of Tom Stellard via llvm-dev" <llvm-dev-bounces at lists.llvm.org on behalf of llvm-dev at lists.llvm.org> wrote:
>>> >>
>>> >>     On 05/30/2019 10:04 AM, Sachkov, Alexey via llvm-dev wrote:
>>> >>     > +llvm-dev
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > *From:* cfe-dev [mailto:cfe-dev-bounces at lists.llvm.org] *On Behalf Of *Bader, Alexey via cfe-dev
>>> >>     > *Sent:* Thursday, May 30, 2019 7:31 PM
>>> >>     > *To:* clang-dev developer list <cfe-dev at lists.llvm.org>
>>> >>     > *Subject:* [cfe-dev] FYI: LLVM Phabricactor notifications.
>>> >>     > *Importance:* Low
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > Hi,
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > I think some of contributors to the Clang received a notifications about some commits done in the past.
>>> >>     >
>>> >>     > I wanted to share my thoughts on why it might has happened.
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > I think the commits from this PR https://github.com/llvm/llvm-project/pull/13were pulled by Phabricator (probably with aim to review GitHub pull requests in https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=AD-hZ9rLcmVgzhl1TFTAvKbX-nrgHD75y9sqN5zTEsg&s=KaCscc77Z-fBuYo2dCuykCeHg35yKQXRwp6uVuIsPos&e= environment).
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > The PR has 2000+ commit, most of which are some old commits from the master branch, and when Phabricator pulled the commits from PR, it sent a notification to the commit author (or committer).
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > Probably we can do something to avoid this situation in the future.
>>> >>     >
>>> >>     >
>>> >>
>>> >>     + Manuel, Chandler
>>> >>
>>> >>     Any idea what happened here? Is phabricator setup to automatically import
>>> >>     pull requests?
>>> >>
>>> >>     -Tom
>>> >>
>>> >>     >
>>> >>     > Sorry for the inconvenience.
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > Alexey
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > --------------------------------------------------------------------
>>> >>     > Joint Stock Company Intel A/O
>>> >>     > Registered legal address: Krylatsky Hills Business Park,
>>> >>     > 17 Krylatskaya Str., Bldg 4, Moscow 121614,
>>> >>     > Russian Federation
>>> >>     >
>>> >>     > This e-mail and any attachments may contain confidential material for
>>> >>     > the sole use of the intended recipient(s). Any review or distribution
>>> >>     > by others is strictly prohibited. If you are not the intended
>>> >>     > recipient, please contact the sender and delete all copies.
>>> >>     >
>>> >>     >
>>> >>     >
>>> >>     > _______________________________________________
>>> >>     > LLVM Developers mailing list
>>> >>     > llvm-dev at lists.llvm.org
>>> >>     > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Ddev&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=AD-hZ9rLcmVgzhl1TFTAvKbX-nrgHD75y9sqN5zTEsg&s=42V9lvDIvDXSSvzhNIJLw7ddZMt54c2yNRxFpSMppEg&e=
>>> >>     >
>>> >>
>>> >>     _______________________________________________
>>> >>     LLVM Developers mailing list
>>> >>     llvm-dev at lists.llvm.org
>>> >>     https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Ddev&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=AD-hZ9rLcmVgzhl1TFTAvKbX-nrgHD75y9sqN5zTEsg&s=42V9lvDIvDXSSvzhNIJLw7ddZMt54c2yNRxFpSMppEg&e=
>>> >>
>>> >>
>>> >> _______________________________________________
>>> >> LLVM Developers mailing list
>>> >> llvm-dev at lists.llvm.org
>>> >> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>>> >
>>> > _______________________________________________
>>> > LLVM Developers mailing list
>>> > llvm-dev at lists.llvm.org
>>> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>>> _______________________________________________
>>> LLVM Developers mailing list
>>> llvm-dev at lists.llvm.org
>>> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev