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

Backport Linker API #10

Merged
merged 2 commits into from
Oct 16, 2024
Merged

Backport Linker API #10

merged 2 commits into from
Oct 16, 2024

Conversation

evacchi
Copy link
Contributor

@evacchi evacchi commented Sep 30, 2024

Backport the API in #9 against Chicory 0.0.12 so that it can be used without waiting for a new upstream release, while we finalize things for 1.0.0. This is exactly the same as #9, except I am using the old 0.0.12 APIs to instantiate modules.

@evacchi
Copy link
Contributor Author

evacchi commented Sep 30, 2024

ough, there might be a bug 🤔 I'll create a reproducer if any

@evacchi
Copy link
Contributor Author

evacchi commented Oct 1, 2024

uhmm, it might not be a bug, it's looking for an extism:host/env.log_debug which is not defined in extism-runtime.wasm

@bhelx
Copy link
Contributor

bhelx commented Oct 4, 2024

uhmm, it might not be a bug, it's looking for an extism:host/env.log_debug which is not defined in extism-runtime.wasm

when is the last time we updated the runtime.wasm? i can't recall if i setup a process for that.

@evacchi
Copy link
Contributor Author

evacchi commented Oct 6, 2024

it was a little old but I realized the real culprit was that we never implemented a bunch of host functions (like log_* or config_{get,set} etc) 😬

@evacchi
Copy link
Contributor Author

evacchi commented Oct 8, 2024

fixed with #11

Signed-off-by: Edoardo Vacchi <evacchi@users.noreply.github.com>
@evacchi evacchi force-pushed the backported-linking branch from 6269863 to e5d0629 Compare October 9, 2024 17:38
Signed-off-by: Edoardo Vacchi <evacchi@users.noreply.github.com>
@evacchi
Copy link
Contributor Author

evacchi commented Oct 16, 2024

Let's merge this to unblock the demo at https://github.com/evacchi/xtp-kafka-demo
We can always revert.

@evacchi evacchi merged commit 967a52a into extism:main Oct 16, 2024
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.

2 participants