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

[MyDeveloper Day via cfe-dev]

Sorry maybe I didn't explain what I saw..(again sorry if this is known
already and I'm stating the obvious to people who know better than me.)

The Revision contains rL in the repository field, as such the herald rule
will add the LLVM project.and the llvm-commit subscriber whenever the
revision is updated.

When the commit AutoCloses, it updates the Revision, as such if the
revision still has the rL repository field then I believe the rule will
fire again and add them. (post commit)

This revision was automatically updated to reflect the committed changes.

H270 Tag SVN LLVM revisions as LLVM <https://reviews.llvm.org/H270>
*Conditions*
Passed Repository is any of rCRT, rL, rLLD, rPLO, rT
Passed Revision title does not contain [private]
*Passed* Rule passed.
*Action: Add projects*
Added Projects Added a project: LLVM <https://reviews.llvm.org/tag/llvm/>.
*Action: Add subscribers*
Added Subscribers Added a subscriber: llvm-commits
<https://reviews.llvm.org/p/llvm-commits/>.

Its true I'm  a little surprised that the Herald rule hadn't fired earlier
and added the LLVM project and llvm-commit subscriber already, unless
somehow it had, but they were removed manually, but I see no trace of that
in the history.

Potentially you could extend the rule to say "Revision status is not any
of  Accepted", but I guess the whole point of that rule is to tell people
watching rL that a commit for a revision marked as being part of rL has
been changed

Unfortunately there isn't any connection between the Revision repository
and the actual repo its committed to.

I do notice LLVM doesn't use the "Owners" application in Phabricator, this
is an great way of ensuring code is automatically channeled to code
owners/reviews for a particular area via the "Affected packages" in a
hearld rule, and can be used to automate the adding of reviewers, blocking
reviews, adding projects (but alas it doesn't allow the setting of the
repository from what I can tell)

MyDeveloperDay (Paul)



On Mon, Jun 3, 2019 at 5:21 PM <paul.robinson at sony.com> wrote:

> As there is no mention of the repository being change the revisions feed (
> https://reviews.llvm.org/D62616) I suspect it was created that way, and
> its only as the commit fires that it gets added.  (it might be clearer if a
> herald rule so these are added at review creation, although anyone then
> removing them will get them readded at commit if they still have the
> incorrect repository.)
>
>
>
> Note the highlighted part of this quote from the revision-closed email:
>
>
>
> This revision was automatically updated to reflect the committed changes.
> Closed by commit rL362363: [CodeComplete] Add a bit more whitespace to
> completed patterns (authored by ibiryukov, committed by ).
> Herald added a project: LLVM.
> Herald added a subscriber: llvm-commits.
>
>
>
> The LLVM project was added at the time Phabricator saw the closing commit,
> not when the revision was created.
>
> If it had been added at the time the revision was created, all review
> emails would have gone to both lists. They did not.
>
> --paulr
>
>
>
> *From:* MyDeveloper Day [mailto:mydeveloperday at gmail.com]
> *Sent:* Monday, June 03, 2019 11:53 AM
> *To:* Robinson, Paul
> *Cc:* dlj at google.com; llvm-dev; clang developer list; Aaron Ballman
> *Subject:* Re: [llvm-dev] FYI: LLVM Phabricactor notifications.
>
>
>
> PaulR
>
>
>
> (sorry again if this is known knowledge)
>
>
>
> > There's no reason for Herald to be adding project LLVM/subscriber
> llvm-commits at the last second here.
>
>
>
> Its possible the rL (LLVM) had be added as the repository in the review on
> creation rather than rCFE, if thats the case then the herald rule "H270" is
> going to fire because it see the repository in the review, so add LLVM
> project and llvm-commits as a subscriber automatically. It won't care that
> this has gone into rCFE and not rL  (I mean it does go into rL via the
> cfe/trunk but I'm not sure if you want to notify for that)
>
>
>
> As there is no mention of the repository being change the revisions feed (
> https://reviews.llvm.org/D62616) I suspect it was created that way, and
> its only as the commit fires that it gets added.  (it might be clearer if a
> herald rule so these are added at review creation, although anyone then
> removing them will get them readded at commit if they still have the
> incorrect repository.)
>
>
>
> MyDeveloperDay (Paul)
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20190603/ba93a511/attachment.html>