search

ghaerr / microwindows

The Nano-X Window System
Other
648 stars 92 forks source link

Update build for Emscripten v2.0.11 #57

Closed ghaerr closed 3 years ago

ghaerr commented 3 years ago

Removes EMTERPRIFY options and uses only WASM code and ASYNC=1, as older EMTERPRETER has been removed from EMSCRIPTEN.

All Win32 and Nano-X programs now build and operate correctly with latest Emscripten version.

Fixes race problem with web pages not completing .js or .data file loads before program exit.

Should fix #37 and probably #55.

To build the EMSCRIPTEN port (builds all demos and runs demo-aafont):

cp Configs/config.emscripten config
./make-emscripten
./emrun demo-aafont
pmp-p commented 3 years ago

xhello.html works fine on mozilla , but it did not for me on chrome/chromium/brave from 87.0.4280.88 till 87.0.4280.101 (tried with emsdk 2.0.10 and 2.0.13 ) could you confirm ? i could investigate a bit if needed

ghaerr commented 3 years ago

Hello @pmp-p,

Thanks for testing this out. I just tried it out using emscripten 2.0.11 and Chrome (unknown old version, and auto-updated to 88.0.4324.16) and xhello ran on both versions. Was it just xhello that is not working, or other demos as well?

I haven't had the time to setup the specific Chrome versions numbers your failure is occurring on, I am sorry.

Thank you!

pmp-p commented 3 years ago

Thanks for the confirmation , so there's a problem indeed but i'm used to it and i'ill look into it : I suspect a problem with a non-reproducible build. It happens often with emscripten + make because their clang script tend to pick up includes from build os in /usr/include instead of staying solely in wasm sysroot and erroring that they are not found.