-
Notifications
You must be signed in to change notification settings - Fork 5
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
Generate bindings from a directory #80
Conversation
⏱ Workflow Timer ⏱😥 The run time for "Continuous Integration" has regressed a bit by 55s (27.29%) 😥 The current run time is 4mins 17s while |
crates/cli/Cargo.toml
Outdated
@@ -29,6 +29,8 @@ serde = { version = "1.0.145", features = ["derive"] } | |||
serde_json = "1.0.85" | |||
webc = "3.0.1" | |||
wasmer-pack = { version = "0.5.0", path = "../wasmer-pack" } | |||
wapm-targz-to-pirita = { git = "ssh://[email protected]/wasmerio/pirita.git", version = "0.1.0" } |
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 we may want to publish as well the wapm_targz_to_pirita
crate to crates.io? (since the webc crate is already published)
Note: we'll need to use the same license as the webc crate
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.
Yep, that's the plan.
We can't publish it at the moment because wapm-targz-to-pirita
uses a fork of the tar
crate to work around alexcrichton/tar-rs#297. We'll probably need to publish our tar
fork or come up with some workaround which avoids calling _set_perms()
.
5ed1c99
to
656ddd4
Compare
Fixes #79.