[BUG] Doesn't work on Linux

[murlakatamenka]

Hi, thanks for the program.

---

It doesn't work on Linux because of the reasons explained in copypasta_ext (link):

A clipboard library providing useful extensions for the copypasta library.

I had a growing annoyance with copypasta, because the clipboard is cleared on the Linux/X11 platform when your application exits as per X11 design. The crate maintainer didn't want to implement workarounds (for valid reasons). This copypasta-ext crate provides additional clipboard contexts that solve this, along with a few other additions.

If you're willing to fix it, then switching to copypasta_ext and using its copypasta_ext::x11_fork::ClipboardContext is sufficient (doc):

use copypasta_ext::prelude::*;
use copypasta_ext::x11_fork::ClipboardContext;

let mut ctx = ClipboardContext::new()?;

Alternative is to use

// no imports needed
let mut ctx = copypasta_ext::try_context().unwrap("failed to get clipboard context);

but it may use other binaries like xsel or wl-copy under the hood.

[printfn]

Right, in that case using copypasta_ext probably makes sense. I might go and implement that!

[printfn]

I've pushed the change to the main branch, would you be able to double-check that it works for you? It is a bit unfortunate that copypasta_ext pulls in an older version of copypasta.

[murlakatamenka]

I've pushed the change to the main branch, would you be able to double-check that it works for you?

Yes, it works now, at least

echo -e ' foo\n' | bp
bp

produces an expected outcome foo

I've checked both approaches before submitting the issue though :)

Some extra details to consider. If runtime detection is used, then it integrates better with current de-facto standard wl-{copy,paste}
utilities, i.e. you can copy with pb, then paste with wl-paste > /tmp/screenshot.png. Kinda defeats the purpose of having a single pb instead, right?

---

It is a bit unfortunate that copypasta_ext pulls in an older version of copypasta.

That's true, somehow copypasta is pinned at specific =0.8.2 for some reason.
I've submitted a PR, let's see how it goes:

https://gitlab.com/timvisee/copypasta-ext/-/merge_requests/2

[murlakatamenka]

@printfn Tim merged that PR. It's not on crates.io yet (still 0.4.4), but you can use git as a crate source for now.

[jankatins]

I can confirm that installing via cargo install --git https://github.com/printfn/bp makes it working on linux (fedora 40, wayland)