Skip to content

Commit

Permalink
Fix the publish script to ignore witx more (#3486)
Browse files Browse the repository at this point in the history
This failed on CI [1] I think because the filesystem is traversed
differently than on my local system. This is relatively easily fixable
though where we shouldn't care about witx version when bumping version
requirements since we don't manage the publication of witx anyway.

[1]: https://github.com/bytecodealliance/wasmtime/runs/4038695579?check_suite_focus=true
  • Loading branch information
alexcrichton committed Oct 28, 2021
1 parent 5c3155f commit c26197e
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions scripts/publish.rs
Original file line number Diff line number Diff line change
Expand Up @@ -270,6 +270,12 @@ fn bump_version(krate: &Crate, crates: &[Crate], patch: bool) {
};

for other in crates {
// If `other` isn't a published crate then it's not going to get a
// bumped version so we don't need to update anything in the
// manifest.
if !other.publish {
continue;
}
if !is_deps || !line.starts_with(&format!("{} ", other.name)) {
continue;
}
Expand Down

0 comments on commit c26197e

Please sign in to comment.