dedup progenitor, and update crucible and propolis revs - #2005
Conversation
Created using spr 1.3.4
|
This is going to need oxidecomputer/propolis#262 as well. |
Created using spr 1.3.4
leftwo
left a comment
There was a problem hiding this comment.
You will also need to update the packaging side as well, with new links to the CI produced
archives for propolis and crucible, as we want the compiled Omicron to match the tar.gz file that becomes part of the installed image.
Check out the package-manifest.toml file for details.
You will have to wait for both crucible and propolis to finish their CI runs on main/master before you will be able to get the zip archive and SHA hash needed here.
Created using spr 1.3.4
Created using spr 1.3.4
|
Is the helios failure just a flake? |
I think it might be genuine. I'm not entirely sure. I'm going to dig into it later today. |
|
Looks like bytecodealliance/rustix#467 |
|
Ah OK, that's easy enough to patch in omicron. Will put up a fix for it. |
Created using spr 1.3.4
Created using spr 1.3.4
Created using spr 1.3.4
Created using spr 1.3.4
This is PR 3 of N in this series. For the earlier ones, see:
Currently, we specify the progenitor dependency as both:
And
Cargo (due to rust-lang/cargo#7497) doesn't dedup these dependencies even
though they resolve to the same branch and hash. This causes issues with
cargo doccolliding with itself, as it tries to document both:Fix this by always using the branch name. (We choose this option over
not specifying the branch so to maintain uniformity in cases where a
non-default branch must be used.)
This also requires a crucible update, since that didn't specify a branch
either. And the crucible update necessitates a propolis update to get the versions aligned.
In the future it would be nice to have a lint which ensures that git
dependencies always have the branch name in them.