Skip to content

Commit c26197e

Browse files
authored
Fix the publish script to ignore witx more (#3486)
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
1 parent 5c3155f commit c26197e

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

scripts/publish.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,6 +270,12 @@ fn bump_version(krate: &Crate, crates: &[Crate], patch: bool) {
270270
};
271271

272272
for other in crates {
273+
// If `other` isn't a published crate then it's not going to get a
274+
// bumped version so we don't need to update anything in the
275+
// manifest.
276+
if !other.publish {
277+
continue;
278+
}
273279
if !is_deps || !line.starts_with(&format!("{} ", other.name)) {
274280
continue;
275281
}

0 commit comments

Comments
 (0)