Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:126181
Feedback-ID: i8414410d:Fastmail
Precedence: bulk
MIME-Version: 1.0
Date: Thu, 26 Dec 2024 22:21:28 -0600
To: "php internals" <internals@lists.php.net>
Message-ID: <aaf24493-211c-495d-9620-b27f476bfc34@app.fastmail.com>
In-Reply-To: 
 <CAEKnhAFMw9h4oxLbze9eQH2q06OabEOuUc7uetoLGJrBmuygAg@mail.gmail.com>
References: 
 <CAEKnhAEFJKPWWL0EKu_JBPgwBEe+Pk3GDzR7HQm3J-g7=Wk0BA@mail.gmail.com>
 <7228b901-a6a9-4aa9-8425-5ad51f66a853@gmx.de>
 <CAEKnhAHx50FeHw=jpG7=2KmCGGn3CaP491dd6Lti3bxSAw0G1g@mail.gmail.com>
 <2556faa6-c233-4809-8999-afe1af40f4f1@app.fastmail.com>
 <CAEKnhAFMw9h4oxLbze9eQH2q06OabEOuUc7uetoLGJrBmuygAg@mail.gmail.com>
Subject: Re: [PHP-DEV] C Unit testing and mocking
Content-Type: text/plain
Content-Transfer-Encoding: 7bit
From: larry@garfieldtech.com ("Larry Garfield")

On Thu, Dec 26, 2024, at 6:25 AM, Jakub Zelenka wrote:

>> Point of order: The recently adopted 3rd party code policy does not apply to C tooling.  It mentions "PHP Tooling", which is defined as "PHP code run by PHP.net".  The website, docs tooling, etc.  It has no bearing on what C libraries or toolchains can or should be used in php-src itself.  (Whether that's unit testing, url parsers, HTML parsers, threading libraries, etc.)
>> 
>
> Ah ok, thanks for the clarification. This part (Definitions -> 
> Librarires) is slightly confusing in this regard though:
>
>> Refers to existing third party code packages or tools, either C extensions or PHP code, maintained by someone other than the PHP Internals team.
>
> Specifically that C extensions part made it sound like it also applies 
> to C code. But after reading the introduction, it's probably more as 
> you say, and it applies only to PHP code.
>
> Cheers
>
> Jakub

Ah, I can see how that could be interpreted that way, but that wasn't the intent.  "C extensions" in this case means things like the MongoDB PECL extension, or Redis PECL extension.  It's saying that if we wanted to start using Redis in the website somewhere (I dunno why, this is hypothetical), and that means using the Redis C PECL extension, that's allowed.

--Larry Garfield