Use helper binary for clipboard access (#262)

Use `clip` on Windows, `pbcopy` on MacOX, `xclip` on Linux.

Remove dependency on `clipboard`.

Author: cruessler

Cargo.lock

@@ -40,7 +40,6 @@ serde = "1.0"
 anyhow = "1.0.33"
 unicode-width = "0.1"
 textwrap = "0.12"
-clipboard = { version = "0.5", optional = true }
 
 [target.'cfg(not(windows))'.dependencies]
 pprof = { version = "0.3", features = ["flamegraph"], optional = true }
@@ -50,6 +49,7 @@ maintenance = { status = "actively-developed" }
 
 [features]
 default=["clipboard"]
+clipboard=[]
 timing=["scopetime/enabled"]
 
 [workspace]

Cargo.toml

@@ -1,24 +1,53 @@
 use anyhow::Result;
-#[cfg(feature = "clipboard")]
-use clipboard::{ClipboardContext, ClipboardProvider};
+use std::io::Write;
+use std::process::{Command, Stdio};
 
-#[cfg(feature = "clipboard")]
-pub fn copy_string(string: String) -> Result<()> {
+fn execute_copy_command(
+    command: &mut Command,
+    string: &str,
+) -> Result<()> {
     use anyhow::anyhow;
 
-    let mut ctx: ClipboardContext = ClipboardProvider::new()
-        .map_err(|e| {
-            anyhow!("failed to get access to clipboard: {}", e)
-        })?;
-    ctx.set_contents(string).map_err(|e| {
-        anyhow!("failed to set clipboard contents: {}", e)
-    })?;
+    let mut process = command
+        .stdin(Stdio::piped())
+        .stdout(Stdio::null())
+        .spawn()
+        .map_err(|e| anyhow!("`{:?}`: {}", command, e))?;
+
+    process
+        .stdin
+        .as_mut()
+        .ok_or_else(|| anyhow!("`{:?}`", command))?
+        .write_all(string.as_bytes())
+        .map_err(|e| anyhow!("`{:?}`: {}", command, e))?;
+
+    process
+        .wait()
+        .map_err(|e| anyhow!("`{:?}`: {}", command, e))?;
 
     Ok(())
 }
 
+#[cfg(all(feature = "clipboard", target_os = "linux"))]
+pub fn copy_string(string: &str) -> Result<()> {
+    execute_copy_command(
+        Command::new("xclip").arg("-selection").arg("clipboard"),
+        string,
+    )
+}
+
+#[cfg(all(feature = "clipboard", target_os = "macos"))]
+pub fn copy_string(string: &str) -> Result<()> {
+    execute_copy_command(&mut Command::new("pbcopy"), string)
+}
+
+#[cfg(all(feature = "clipboard", windows))]
+pub fn copy_string(string: &str) -> Result<()> {
+    execute_copy_command(&mut Command::new("clip"), string)
+}
+
 #[cfg(not(feature = "clipboard"))]
-pub fn copy_string(_string: String) -> Result<()> {
+pub fn copy_string(_string: &str) -> Result<()> {
     Ok(())
 }
 

src/clipboard.rs

@@ -269,7 +269,7 @@ impl DiffComponent {
                 self,
                 "copy to clipboard error:",
                 crate::clipboard::copy_string(
-                    lines_to_copy.join("\n")
+                    &lines_to_copy.join("\n")
                 )
             );
         }