-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Automatic sysroot creation #41
Comments
we could use something like the This should allow us to work completely locally. Maybe we can even the miri script be a wrapper around another script that serves the base functionality that priroda and miri need? |
Hmm, we could do something It's possible that we'll need some custom setup stuff anyway - I'm not certain what |
I don't understand what this has to do with using miri master vs the miri pinned in the rustc submodule, but if there's a way to make
The two binaries |
Priroda depends on miri the library directly, not miri the executable. It is currently not possible to have executable dependencies. |
Could we ship the miri library together with the binary so priroda can just use the one from rustup instead of having a git dependency? |
That could happen, however the reason I want to depend on main |
I see, you are doing
but there is no way to do that for cargo-miri. From what I know, splitting a crate like cargo-miri into a lib and a bin crate and making the bin crate just a thin wrapper around the lib is pretty straight-forward, so that seems totally acceptable in terms of Miri-side overhead. |
Yeah, exactly. I'm sure I'll make the PR after I do #40 |
Although we depend on
miri
from https://github.com/rust-lang/miri directly, we require that the user installs a sysroot manually usingcargo miri setup
(since that comes from themiri
rustup component). This means we are locked to the version ofmiri
which is in https://github.com/rust-lang/rust. However, it would be better if we could instead use the rustc version which https://github.com/rust-lang/miri compiles with. This would allow us to update more frequently.In particular, if we make changes to miri (such as rust-lang/miri#1736) then we need to wait for the miri update to reach nightly to use those changes.
cc @RalfJung the easiest way for us to make this possible would be for
cargo miri [setup]
to be callable as a library, although I suppose we could also installcargo miri
directly from the repository. That does seem like a worse solution however.The text was updated successfully, but these errors were encountered: