@php-wasm/universal : Add Phar support in php-wasm

[mho22]

Based on issue #1241.

This is an initial pull request version. I may need further guidance to ensure it is ready for merging.

[adamziel]

Thank you so much! Looks great to me. I'll hold on with merging until next week to make sure someone's online to revert just in case.

[mho22]

@adamziel I'm currently trying to run the composer PHAR file in php-wasm/web, but I'm encountering the following stack trace when running php.cli( [ 'php', 'composer.phar', 'install' ] );:

Uncaught (in promise) RuntimeError: null function or function signature mismatch
    at 032749f6:0xdfebf
    at 032749f6:0x7e623
    at 032749f6:0x2c68d9
    at 032749f6:0x6a6ec2
    at 032749f6:0x707bcc
    at 032749f6:0x74b9b4
    at 032749f6:0x2e9fbe
    at 032749f6:0x6eb13e
    at 032749f6:0x6ae8a1
    at ret.<computed> (php_8_2.js:43:72504)Caused by: Error
    at Asyncify.handleSleep (php_8_2.js:60:49)
    at _emscripten_sleep (php_8_2.js:43:39745)
    at 032749f6:0xddacd
    at 032749f6:0x6a6dbe
    at 032749f6:0x707bcc
    at 032749f6:0x74b9b4
    at 032749f6:0x2e9fbe
    at 032749f6:0x6eb13e
    at 032749f6:0x6ae8a1
    at ret.<computed> (php_8_2.js:43:72504)

with :

Uncaught RuntimeError: unreachable
    at 032749f6:0x6e6706
    at ret.<computed> (php_8_2.js:43:72504)
    at t.wasmExports.<computed> (index.js:522:18)
    at Object.doRewind (php_8_2.js:43:74168)
    at php_8_2.js:43:74741
    at callUserCallback (php_8_2.js:43:37432)
    at php_8_2.js:43:39676Caused by: Error
    at Asyncify.handleSleep (php_8_2.js:60:49)
    at _emscripten_sleep (php_8_2.js:43:39745)
    at 032749f6:0xddacd
    at 032749f6:0x6a6dbe
    at 032749f6:0x707bcc
    at 032749f6:0x74b9b4
    at 032749f6:0x2e9fbe
    at 032749f6:0x6eb13e
    at 032749f6:0x6ae8a1
    at ret.<computed> (php_8_2.js:43:72504)

This stack trace looks somewhat familiar but is obviously hard to read. Do you have any insights or suggestions on how to make this more debuggable?

[adamziel]

@mho22 the Node.js build should give you the same error with all the c function names. I would so love to start shipping the JSPI build to the browser ms and server side environments that support it as it would solve all such errors in one swoop.

[mho22]

@adamziel I made a mistake. I solved it and here is what returns php-wasm/web when I run php.cli( [ 'php', 'composer.phar', 'install' ] ) now :

 
 

With [#1093] implementation :

 
 

And here is what returns php-wasm/node when I run php.cli( [ 'php', 'composer.phar', 'install' ] ) :

 
 

This looks promising for the pull request I think.

[mho22]

In my quest to correct the errors above I found out other elements to add into the Asyncify functions list :

"Curl_proxy_connect",\
"call_extract_if_dead",\
"Curl_conncache_foreach",\

Should I update the current pull request ?

[adamziel]

Should I update the current pull request ?

Yes please, especially since we've had another .wasm updating PR merged in the meantime and there are wasm merge conflicts.

[mho22]

@adamziel done 👌

[adamziel]

LGTM although GitHub still shows merge conflicts

[mho22]

@adamziel My apologies. The conflicts have been resolved. But test-e2e and build failed?

[adamziel]

It seems like they passed after rebasing. Thank you @mho22!