Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This adds the rest of the emscripten macros that wrap javascript code… #6234

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Conversation

dsamo
Copy link
Contributor

@dsamo dsamo commented Apr 5, 2024

… to emscripten.cfg

Namely:
EM_ASM_DOUBLE
EM_ASM_INT
EM_ASM_PTR
MAIN_THREAD_EM_ASM_DOUBLE
MAIN_THREAD_EM_ASM_INT
MAIN_THREAD_EM_ASM_PTR
MAIN_THREAD_ASYNC_EM_ASM

@firewave
Copy link
Collaborator

firewave commented Apr 5, 2024

Half of the actions are not being run. Something's weird going on here.

@dsamo
Copy link
Contributor Author

dsamo commented Apr 5, 2024

@firewave My changes do not seem related to the errors. Should I do something from my side?

@chrchr-github
Copy link
Collaborator

If you wish, you could add yourself to the AUTHORS file. That would trigger a rerun.

@dsamo
Copy link
Contributor Author

dsamo commented Apr 5, 2024

@chrchr-github done

@dsamo
Copy link
Contributor Author

dsamo commented Apr 5, 2024

There is now an error in coverage build due to 502 server error - Rebased to retrigger

dsamo added 2 commits April 5, 2024 16:10
… to emscripten.cfg

Namely:
EM_ASM_DOUBLE
EM_ASM_INT
EM_ASM_PTR
MAIN_THREAD_EM_ASM_DOUBLE
MAIN_THREAD_EM_ASM_INT
MAIN_THREAD_EM_ASM_PTR
MAIN_THREAD_ASYNC_EM_ASM
@firewave
Copy link
Collaborator

firewave commented Apr 5, 2024

There is now an error in coverage build due to 502 server error - Rebased to retrigger

That happens from time to time. We can just re-run that step when it happens. No need for a re-trigger.

@chrchr-github chrchr-github merged commit 2f0bb71 into danmar:main Apr 5, 2024
64 checks passed
@dsamo dsamo deleted the patch-1 branch April 5, 2024 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants