Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:106700
MIME-Version: 1.0
References: <CAF+90c-5EwcrVfuzjunhRecAdSBiKb7ATDN0_xU2aDeoj6gz3g@mail.gmail.com>
 <480CCF73-6CF8-4F7E-B274-B785731ECE48@zend.com> <55B76F88-E657-4108-9215-9E508E882CFC@zend.com>
In-Reply-To: <55B76F88-E657-4108-9215-9E508E882CFC@zend.com>
Date: Fri, 23 Aug 2019 19:05:03 +0200
Message-ID: <CAF+90c-mQmcH3ZF+u9QXu39SAg=GSCE1nhjSGWvomsdADJQ0Hg@mail.gmail.com>
To: Zeev Suraski <zeev@zend.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000009b0b870590cbcff7"
Subject: Re: [PHP-DEV] Remove special Zend karma
From: nikita.ppv@gmail.com (Nikita Popov)

--0000000000009b0b870590cbcff7
Content-Type: text/plain; charset="UTF-8"

On Fri, Aug 23, 2019 at 11:25 AM Zeev Suraski <zeev@zend.com> wrote:

>
> On 23 Aug 2019, at 10:56, Zeev Suraski <zeev@zend.com> wrote:
>
>
>
> On 23 Aug 2019, at 10:33, Nikita Popov <nikita.ppv@gmail.com> wrote:
>
>
> Hi internals,
>
>
> We currently have separate karma for the Zend/TSRM directories in php-src.
>
> I think this separation has become more annoying than useful at this point.
>
> Most changes from newer contributors are approved on GitHub first, and we
>
> often have to grant some form of Zend karma for tests if nothing else
>
> anyway.
>
>
> Any objections to dropping this and have a single php-src karma for the
>
> whole tree (with the exception of php_version.h, which is used to prevent
>
> bad merges)?
>
>
>
> No objections here - the separation is fairly meaningless nowadays.
>
>
> On second reading, do you mean to merge the Zend/TSRM into one, or so away
> with both altogether?  I initially read it as the former but based on the
> reasoning I think you probably meant the latter..?  Personally, given the
> complexity I think this level of granularity makes sense - do we really
> have that many folks that should have direct access there?
>
> Zeev
>

Zend/TSRM already have shared karma, I'm suggesting to merge it into
php-src karma. Complexity is pretty meaningless here, as changes need to be
reviewed anyway. Zend/ isn't different from other code in that respect. We
also have plenty of other complex code outside of Zend, such as ext/opcache.

Nikita

--0000000000009b0b870590cbcff7--