Tracking issuss of Bindings PHP

[godruoyi]

RFC: #2782

Tasks

• use custom exception: Err: Attempted to access uninitialized class object extphprs/ext-php-rs#262
• Investigate the possibility of releasing opendal to pecl, or explore other solutions that would allow users to avoid building from rust.
• fix php ci: fix: make php binding happy again #5761

[Xuanwo]

Is there a way to distribute native exetension so that users only need to add fields in composer instead? I expect users don't need to know about rust core details and just use opendal as a native php package.

[godruoyi]

Actually, we cannot do this yet. If we want to use "composer require" to install opendalphp, we need to submit the code to https://packagist.org/ and it should only contain PHP code instead of binary files. Generally speaking, we can only install PHP binary extensions through pecl. Perhaps we can consider how to publish opendal on pecl so that we can install opendal-php just like installing redis/swoole.

pecl install redis
pecl install swoole
pecl install opendal-php

[Xuanwo]

opendal-php is our project name, and users should use opendal in php ecosystem. Thus, we should support pecl install opendal. Others LGTM.

[Xuanwo]

Hi, @godruoyi, our php build failed on github action, would you like to take a look?

context: #3055

[godruoyi]

hi @Xuanwo, It seems that the issue is caused by upgrading to Ubuntu 22.04.3 (ubuntu-latest). In this version, we are using clang 14.0.0.

failed CI

And in Ubuntu 20.04, we were able to compile successfully. This version uses clang 12.0.0.
Passed CI

We can temporarily change the IMAGE in PHP CI to ubuntu-20.04 now(if yes, will push a pr soon), and then I need to take a look at ext-php-rs. We need it to support ubuntu-latest first then we can change the running environment of PHP CI in opendal to ubuntu latest. This may take some time because it seems that this repository has not merged any PR for a long time.

[Xuanwo]

So, the issue we encountered here is that ext-php-rs doesn't function on clang 14?

---

By the way, it works on my PC with clang 16:

:) clang -v
clang version 16.0.6
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-pc-linux-gnu/13.2.1
Found candidate GCC installation: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1
Selected GCC installation: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/13.2.1
Candidate multilib: .;@m64
Candidate multilib: 32;@m32
Selected multilib: .;@m64
:) php -v
PHP 8.2.10 (cli) (built: Sep  1 2023 11:07:36) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.10, Copyright (c) Zend Technologies

[godruoyi]

I don't have much knowledge about this case, need some time to take a look.

---

Are you running on 22.04.3? @Xuanwo

[Xuanwo]

Are you running on 22.04.3? @Xuanwo

I'm running on archlinux.

[Xuanwo]

Hi, @godruoyi, would you be interested in taking another look? I'm curious whether we can provide a pre-built release for PHP so that our users can use OpenDAL directly from PHP without needing to set up Rust.