- Notifications
You must be signed in to change notification settings - Fork83
Emscripten no longer exportHEAPU8 by default after v4.0.7#278
-
I have tried to setup |
BetaWas this translation helpful?Give feedback.
All reactions
😕 1
I addedHEAPU8 and HEAP32 to src/extra_exported_runtime_methods.json, and I think that fixes the build with EMSDK 4.0.7+.
@subframe7536 I think the change in your sqwab fork would have worked except it was applied to the wrong file, as@khanhiceteasuggests. Note that sqwab already has a mechanism to add or edit files by putting them under overlay.wa-sqlite, which might be easier than changing the workflow.
Replies: 4 comments 1 reply
-
Found that |
BetaWas this translation helpful?Give feedback.
All reactions
-
https://github.com/subframe7536/sqwab/actions/runs/15345257994/job/43179989242#step:9:223
|
BetaWas this translation helpful?Give feedback.
All reactions
-
AFAIK, all you need to do is build with:
That resolved this issue for our build (unrelated to wa-sqlite, but we found out about this breaking change from this thread). |
BetaWas this translation helpful?Give feedback.
All reactions
-
Thanks, I have already add these variables: https://github.com/subframe7536/sqwab/actions/runs/15345257994/workflow#L56-L67 But still fail to build. |
BetaWas this translation helpful?Give feedback.
All reactions
-
I addedHEAPU8 and HEAP32 to src/extra_exported_runtime_methods.json, and I think that fixes the build with EMSDK 4.0.7+. @subframe7536 I think the change in your sqwab fork would have worked except it was applied to the wrong file, as@khanhiceteasuggests. Note that sqwab already has a mechanism to add or edit files by putting them under overlay.wa-sqlite, which might be easier than changing the workflow. |
BetaWas this translation helpful?Give feedback.
All reactions
❤️ 1