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
