Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:74870
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.179 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <alpine.DEB.2.10.1406121249290.4247@whisky.home.derickrethans.nl>
References: <92.50.02901.F0978935@osu1php.osuosl.org>
	<alpine.DEB.2.10.1406111716340.4247@whisky.home.derickrethans.nl>
	<1E1B3A6C-E598-42C1-9C67-6ACBAF594FB3@ajf.me>
	<alpine.DEB.2.10.1406121249290.4247@whisky.home.derickrethans.nl>
Date: Thu, 12 Jun 2014 14:14:12 +0200
Message-ID: <CAH-PCH5fBA4iUx1eJDoS1TNqdq5V89K49rW5bPr1gLdKO+3k_Q@mail.gmail.com>
To: Derick Rethans <derick@php.net>
Cc: Andrea Faulds <ajf@ajf.me>, PHP Developers Mailing List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c15f3ab7f7cc04fba27f6c
Subject: Re: [PHP-DEV] [PR][php-src][#693][master][opened] -
 pcntl_signal_dispatch: Speed up by preventing system calls when unnecessary
From: tyra3l@gmail.com (Ferenc Kovacs)

--001a11c15f3ab7f7cc04fba27f6c
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Thu, Jun 12, 2014 at 1:51 PM, Derick Rethans <derick@php.net> wrote:

> On Wed, 11 Jun 2014, Andrea Faulds wrote:
>
> >
> > On 11 Jun 2014, at 17:17, Derick Rethans <derick@php.net> wrote:
> >
> > > Why is this not going to git-pulls@ ? Please don't make new PRs drown
> in
> > > the noise that internals (can be).
> >
> > They=E2=80=99re hardly *drowning* internals. Furthermore, said patches =
will
> > eventually be announced on internals anyway if they don=E2=80=99t get m=
erged.
> >
> > I say keep them here.
>
> I say you don't make that decision on your own. We have a list for it,
> put it on there.
>
> And right now, the subject lines are pretty useless too. On
> smaller devices, I see "[PHP-DEV] [PR][php-src][#693][master][opened] -
> pcntl_signa"
>
> I care neither about the repository, number, the branch, or whether it
> is opened at
> a glance, but merely the PR. This should go in the body of the
> mail. "[PHP-DEV] [PR]" is also not needed beacuse it needs to go onto
> it's already existing list.
>

[PHP-DEV] is not added by the script, but the mailing list
[PR] was added so people who aren't interested in PRs can easily filter it
same for the the repo name, I think you are right that the PR number and
status doesn't need to be in the subject, I will fix that.

--=20
Ferenc Kov=C3=A1cs
@Tyr43l - http://tyrael.hu

--001a11c15f3ab7f7cc04fba27f6c--