Should not just use xclip

[zefr0x]

Regarding this function:

silicon/src/bin/silicon/main.rs

Lines 25 to 40 in cf3668c

	#[cfg(target_os = "linux")]
	pub fn dump_image_to_clipboard(image: &DynamicImage) -> Result<(), Error> {
	let mut temp = tempfile::NamedTempFile::new()?;
	image.write_to(&mut temp, ImageOutputFormat::Png)?;
	Command::new("xclip")
	.args([
	"-sel",
	"clip",
	"-t",
	"image/png",
	temp.path().to_str().unwrap(),
	])
	.status()
	.map_err(|e| format_err!("Failed to copy image to clipboard: {}", e))?;
	Ok(())
	}

In Linux the xclip command just work on X11, not Wayland.

Options:

1. xclip could be an optional dependency alongside with wl-clipboard, and warn the user if he don't have any one of them.
2. Depend on Rust libraries to handle clipboard for both X11 and Wayland without using an external tool.
3. Just support Wayland and deprecating X11 support using 1 or 2.

[chris-av]

Also a linux user, and use wl-clipboard. The only reason I have xclip on my system is because of silicon. Would definitely appreciate if the code took Wayland into account.

[pixlmint]

I am on Ubuntu 24.04 Wayland, wl-clipboard was not installed by default and silicon just failed, it should check if wl-clipboard is installed just like it does with xclip