From c26197ef8c6e78bd41ea76cd5235a7fb78c95e32 Mon Sep 17 00:00:00 2001 From: Alex Crichton Date: Thu, 28 Oct 2021 14:39:06 -0500 Subject: [PATCH] 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 --- scripts/publish.rs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/scripts/publish.rs b/scripts/publish.rs index 848d319915..e4aa342246 100644 --- a/scripts/publish.rs +++ b/scripts/publish.rs @@ -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; }