Re: filesystem notification extension
| From: | Frederik Bosch | Date: | Tue, 30 Jul 2024 14:43:28 +0000 |
| Subject: | Re: filesystem notification extension | ||
| References: | 1 2 3 4 5 6 7 | Groups: | php.pecl.dev |
| Request: | Send a blank email to pecl-dev+get-17602@lists.php.net to get a copy of this message | ||
On 30-07-2024 15:29, Christoph M. Becker wrote:
On 30.07.2024 at 14:43, Derick Rethans wrote: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, FrederikOn Tue, 30 Jul 2024, Frederik Bosch wrote: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,On 27-07-2024 16:52, Christoph M. Becker wrote: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.I have only rough knowledge ofpecl(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?