Re: [PECL-DEV] filesystem notification extension

On 30-07-2024 15:29, Christoph M. Becker wrote:
> On 30.07.2024 at 14:43, Derick Rethans wrote:
>
>> On Tue, 30 Jul 2024, Frederik Bosch wrote:
>>
>>> On 27-07-2024 16:52, Christoph M. Becker wrote:
>>>
>>>> I have only rough knowledge of `pecl` (the command) since I'm mostly
>>>> working on Windows, and I don't know anything about Cargo.  So I
>>>> don't have the faintest idea whether that requirement would be a
>>>> stopper to add the extension to PECL.  Do we have precendent for
>>>> that requirement?
>> No, I don't think we don't. I would probably guess that a lot of people
>> would have a C compiler, and not a whole rust toolchain installed as
>> well.
>>
>> For the new PECL installer (PIE, https://github.com/php/pie) supporting
>> this "style" hasn't been taken under consideration.
>>
>> I would also say that it is unexpected to suddenly also have to download
>> a gazilion crates (it took 432MB!), on top of a whole rust compile
>> stack. Server environments will not likely to have this pre-installed
>> either.
> Oh, this appears to be a problem.  However, I believe at some time we
> need to support extensions written in Rust (to my knowledge there is
> Rust support even in the Linux kernel now).  And what to do with
> skywalking (and maybe more packages) already using Rust?
>
> But this is unlikely to be the right thread, and maybe even not the most
> suitable mailing list.  I wonder where to continue the discussion?
>
> Cheers,
> Christoph
Hi Christoph, Derick,

While I totally understand this feels like you get a node_modules folder 
for free when running cargo build, I did want to share my extension 
because there might be other people interested in using filesystem 
notifications. Personally, I use this to detect changes in #[Attributes] 
in my PHP classes.

If you decide to refuse the extensions for PECL, I'd be disappointed, 
but I could also understand. I nonetheless agree with Christoph, as a 
developer this was a very good experience for creating an extension.

Hopefully, you can postpone the decision on Rust-based extensions after 
my FS Notify extension request. Thanks anyhow for having a look!

Regards,
Frederik