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

fix hc wasm 2 #79

Merged
merged 2 commits into from
Jul 11, 2022
Merged

fix hc wasm 2 #79

merged 2 commits into from
Jul 11, 2022

Conversation

mhuesch
Copy link
Contributor

@mhuesch mhuesch commented Jul 11, 2022

this reverts part of #73.

my theory of what went wrong

Conductor ready.
Error: Failed to install app: Expected AdminResponse::AppBundleInstalled but got Error(InternalError("Conductor returned an error while using a ConductorApi: EntryDefStoreError(DnaError(WasmError(Compile(\"Error while importing \\\"__wbindgen_placeholder__\\\".\\\"__wbindgen_describe\\\": unknown import. Expected Function(FunctionType { params: [I32], results: [] })\"))))"))

(above output comes for commit d388d8c in social_sensemaker repo)

what to do

the best thing would be to fix the root cause.

but my hunch is that it would be complex & take too much time.

so for now, we will keep quickcheck back at 0.9.2 release. perhaps things will get fixed in the meantime 🤷‍♂️

mhuesch added 2 commits July 11, 2022 10:38
this is in hopes of avoiding an issue with getrandom (a dep of new QC)
which I don't want to figure out rn.
@mhuesch mhuesch merged commit 364213a into main Jul 11, 2022
@mhuesch mhuesch deleted the fix-hc-wasm-2 branch July 11, 2022 16:55
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.

1 participant