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
feat: add awkward-cpp v22 #4101
feat: add awkward-cpp v22 #4101
Conversation
for more information, see https://pre-commit.ci
…yodide into agoose77/feat-add-awkward-cpp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @agoose77! Could you add some basic tests for awkward-cpp?
build: | ||
script: | | ||
export CMAKE_ARGS="${CMAKE_ARGS} -DEMSCRIPTEN=1" | ||
exports: whole_archive |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens if you don't set whole_archive
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It used to be the case that some symbols were missing. Right now I can't seem to test this locally; I'm getting
OSError: [Errno 7] Argument list too long: 'em++'
I don't recall this happening before.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's okay for now then then, we have a bug in our symbol logic that causes this Argument list too long
error when there are a lot of exported symbols.
Co-authored-by: Gyeongjae Choi <def6488@gmail.com>
…yodide into agoose77/feat-add-awkward-cpp
for more information, see https://pre-commit.ci
Description
Add the awkward-cpp package.
Checklists