Switch & upgrade to quickjs-ng 7.0.0

[p-bakker]

quickjs-ng 7.0.0 has been released, so would be nice to update this project to that version

Additionally, it seems that quickjs-ng has much more traction than quickjs, so maybe make quickjs-ng the default?

[justjake]

I don’t know how CMake works, so I’m reluctant to add a dependency on CMake to this project. If you want to maintain the quickjs-ng variants I’m happy to accept a PR updating the build process to work with quickjs-ng 7

[p-bakker]

Afraid I probably know less than you about how CMake works... :-/

[saghul]

@justjake Hey there! Can you point me to where you build quickjs? I can help.

[justjake]

@saghul id appreciate the support. I template Variant.mk into each variant, then run make per variant to produce the final wasm and/or js files

[justjake]

Here’s the make rule for quickjs:

quickjs-emscripten/templates/Variant.mk

Line 151 in aa48b61

$(BUILD_QUICKJS)/%.o: $(QUICKJS_ROOT)/%.c $(WASM_SYMBOLS) | $(EMCC_SRC)

[justjake]

I think one issue I encountered trying to adapt newer quickjs-ng is that it’s harder to enable malloc_usable_size and other memory accounting features in quickjs-ng for Emscripten environment. Emscripten appears to provide a malloc_usable_size but perhaps the original authors thought it’s a useless feature in web? Or maybe it has bugs? But between cmake being intimidating and some of the ifdef changes I gave up on quickjs-ng

[saghul]

I'll look into this.