Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Feb 2015 13:01:40 -0800
From:      Justin Hibbits <jhibbits@freebsd.org>
To:        Benjamin Kaduk <bjk@freebsd.org>
Cc:        "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, Ian Lepore <ian@freebsd.org>
Subject:   Re: svn commit: r279349 - head/sys/kern
Message-ID:  <CAHSQbTDHdWBqutBB1KLJapJ741R-dzBkMbFsLa=27O5akSvy%2BQ@mail.gmail.com>
In-Reply-To: <alpine.GSO.1.10.1502271550580.3953@multics.mit.edu>
References:  <201502270256.t1R2uxnv085328@svn.freebsd.org> <20150227082257.3fb1081c@bender.Home> <20150227202646.I2088@besplex.bde.org> <CAJ5_RoCj_D06WK%2BUVrLXNouu252X-6oaaj68RJoQhDzx_wJyAA@mail.gmail.com> <CAPyFy2AwRzvksD_5EcEU5EEvzR7tB1FOpAYzWc2nLtCnyGzaRQ@mail.gmail.com> <E4396923-9EAD-4FFA-82B9-6831B385CA8E@bsdimp.com> <1425052560.1281.21.camel@freebsd.org> <alpine.GSO.1.10.1502271550580.3953@multics.mit.edu>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Feb 27, 2015 at 12:52 PM, Benjamin Kaduk <bjk@freebsd.org> wrote:
> On Fri, 27 Feb 2015, Ian Lepore wrote:
>
>> I wonder if it would be useful to have the system itself mail the
>> summary line of every new review that's created to some list?  It would
>> probably need to be several lists -- I sure don't want to see every
>> ports phab that gets created, but getting a single mail for every src
>> review created wouldn't be too much.
>
> There's no harm in having such a list exist (assuming it is not mandatory
> for anyone to be on, which I suppose could be another proposal), and if
> some people would get use from it, it seems like a net benefit.
>
> Are you volunteering to ask phabric-admins if there's a hook to send mail
> on review creation and postmaster to make the mailing lists?
>
> -Ben
>

Whoever (not me!) volunteers, there's Phabricator Herald, to do things
like adding people (or groups) to reviews (if everyone joins a src
group, it'll Just Work).  LLVM uses this with its code reviews.

https://secure.phabricator.com/book/phabricator/article/herald/

- Justin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAHSQbTDHdWBqutBB1KLJapJ741R-dzBkMbFsLa=27O5akSvy%2BQ>