node: Control via `uds_windows` not `winpipe`
Recent versions of Windows support Unix Domain Sockets natively, see https://devblogs.microsoft.com/commandline/af_unix-comes-to-windows/.
By using that feature instead of Windows named pipes, the difference for handling communication via the control socket comparing Unix-like systems and Windows becomes smaller:
- No special paths like
\.\pipe\…have to be handled. - Not two I/O mechanisms are abstracted (named pipe and UDS) but just one.
winpiperelies on background threads whileuds_windowsdoes not.
Once the feature windows_unix_domain_sockets (which is tracked at
https://github.com/rust-lang/rust/issues/150487) stabilizes, it will
likely be possible to just drop the dependency uds_windows and use
the implementation in std::os::windows::net directly.
9 files changed
+54
-101
90cf37c4
→
ebf7d876
modified Cargo.lock
@@ -2229,6 +2229,15 @@ dependencies = [
|
|
|
|
|
|
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
|
|
|
|
|
@@ -2840,8 +2849,8 @@ dependencies = [
|
|
|
|
|
|
| + | |
|
|
| - | |
|
|
|
|
|
@@ -3071,7 +3080,7 @@ dependencies = [
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -4492,6 +4501,17 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
|
|
|
|
|
|
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
| + | |
|
|
|
|
|
modified Cargo.toml
@@ -69,6 +69,7 @@ snapbox = "0.4.3"
|
|
|
|
|
|
| + | |
|
|
|
|
|
modified crates/radicle-node/Cargo.toml
@@ -47,8 +47,8 @@ thiserror = { workspace = true, default-features = true }
|
|
|
|
|
|
| - | |
|
|
| + | |
|
|
|
|
|
modified crates/radicle-node/src/control.rs
@@ -6,9 +6,9 @@ use std::path::PathBuf;
|
|
|
|
|
|
| - | |
| + | |
|
|
| - | |
| + | |
|
|
|
|
|
@@ -33,7 +33,7 @@ pub enum Error {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -45,11 +45,11 @@ where
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -77,63 +77,16 @@ enum CommandError {
|
|
|
|
|
|
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| + | |
|
|
|
|
|
|
|
|
|
|
|
|
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| + | |
| + | |
|
|
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
|
|
|
|
|
@@ -319,7 +272,7 @@ mod tests {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -330,7 +283,7 @@ mod tests {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -369,7 +322,7 @@ mod tests {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
modified crates/radicle-node/src/runtime.rs
@@ -6,9 +6,9 @@ use std::path::PathBuf;
|
|
|
|
|
|
| - | |
| + | |
|
|
| - | |
| + | |
|
|
|
|
|
@@ -99,12 +99,12 @@ impl From<service::Error> for Error {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
| - | |
| + | |
|
|
| - | |
| + | |
|
|
|
|
|
@@ -323,7 +323,7 @@ impl Runtime {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -348,7 +348,7 @@ impl Runtime {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -361,7 +361,7 @@ impl Runtime {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
modified crates/radicle-node/src/runtime/handle.rs
@@ -5,9 +5,9 @@ use std::sync::Arc;
|
|
|
|
|
|
| - | |
| + | |
|
|
| - | |
| + | |
|
|
|
|
|
@@ -338,7 +338,7 @@ impl radicle::node::Handle for Handle {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
modified crates/radicle/Cargo.toml
@@ -63,7 +63,7 @@ unicode-normalization = { version = "0.1" }
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
modified crates/radicle/src/node.rs
@@ -25,9 +25,9 @@ use std::str::FromStr;
|
|
|
|
|
|
| - | |
| + | |
|
|
| - | |
| + | |
|
|
|
|
|
@@ -948,7 +948,7 @@ pub trait Handle: Clone + Sync + Send {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -1009,7 +1009,7 @@ impl Node {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
modified crates/radicle/src/profile.rs
@@ -603,32 +603,11 @@ impl Home {
|
|
|
|
|
|
| - | |
| - | |
| - | |
|
|
|
|
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| + | |
| + | |
| + | |
|
|
|
|
|