watchexec/watchexec

panic on a 32bit ARM arch

Closed this issue · 7 comments

egorenar commented

Please see rmcgibbo/async-priority-channel#78

❯ watchexec --version
watchexec 1.24.1 (fb2829d1ab38 2023-12-14) +pid1
commit-hash: fb2829d1ab38e1edadef342e580615659455eef7
commit-date: 2023-12-14
build-date: 2023-12-14
release: 1.24.1
features: default,pid1
passcod commented

Ah, that's not great. It looks like the crate hasn't been very maintained, so might need to swap it out entirely to get support here :/

egorenar commented

there is a fix already :)
rmcgibbo/async-priority-channel#78

egorenar commented

my temporary fix:

--- a/crates/lib/Cargo.toml	2023-12-15 12:10:33.160927869 +0100
+++ b/crates/lib/Cargo.toml	2023-12-15 12:10:44.842927459 +0100
@@ -16,7 +16,7 @@
 edition = "2021"
 
 [dependencies]
-async-priority-channel = "0.1.0"
+async-priority-channel = { git = "https://github.com/rmcgibbo/async-priority-channel.git", rev = "refs/pull/79/head" }
 async-recursion = "1.0.5"
 atomic-take = "1.0.0"
 futures = "0.3.29"
--- a/crates/lib/src/config.rs	2023-12-15 12:16:57.059914388 +0100
+++ a/crates/lib/src/config.rs	2023-12-15 12:17:08.059914002 +0100
@@ -152,7 +152,7 @@
 	/// adjusting this value may help.
 	///
 	/// This is unchangeable at runtime and must be set before Watchexec instantiation.
-	pub event_channel_size: usize,
+	pub event_channel_size: u64,
 }
 
 impl Default for Config {
passcod commented

Oh neat! Thank you

passcod commented

I'll get a fix out as soon as upstream merges and releases :)

  • 👍1
egorenar commented

the fix has been merged.
and version 0.2.0 is available now.