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>