-
Notifications
You must be signed in to change notification settings - Fork 696
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
wasm-builder: Use riscv32emac-unknown-none-polkavm.json target #6419
base: master
Are you sure you want to change the base?
Conversation
5199aa3
to
005ec4a
Compare
How do I pass |
283e8b0
to
5205e58
Compare
Co-authored-by: Koute <[email protected]>
Signed-off-by: Jarkko Sakkinen <[email protected]>
Signed-off-by: Jarkko Sakkinen <[email protected]>
Signed-off-by: Jarkko Sakkinen <[email protected]>
I am confused that this even works. The Lines 1093 to 1094 in a1af8ed
|
Looks like the build runtime tests are failing, so yes, the derive and the linker crates should also be bumped. |
I posted this on Tuesday: I.e. still not knowing what to do (day three). |
Answered here: #6419 (comment) |
Please don't merge without addressing: |
Signed-off-by: Jarkko Sakkinen <[email protected]>
Signed-off-by: Jarkko Sakkinen <[email protected]>
This has now been addresses (finally) :-) I'll update next time to latest without asking. |
Signed-off-by: Jarkko Sakkinen <[email protected]>
Signed-off-by: Jarkko Sakkinen <[email protected]>
Bump did not fix the issue, it still gives: "Failed to write metrics: promwrite: sending remote write request: Post "placeholder": unsupported protocol scheme". |
So if I do
|
Description
Closes #6335.
Integration
N/A
Review Notes
RuntimeTarget
is converted to return path to the custom target JSON file