[PATCH] clip: copy password into all displays if possible

password-store at storiepvtride.it password-store at storiepvtride.it
Thu Feb 13 23:22:15 CET 2020


Hello Daniel,

thanks for the Wayland/Sway patch, I have the same setup of you, so I am
very interested :-)

Just two small comments from my humble perspective:

> One thing I was considering is adding a -w flag that the user could use
> to specify clipping to wayland clipboard, and that -wc would do both.

I vote against changing the meaning of "-c" from "copy the password" to
"copy the password under X". How the password is copied into the
clipboard is an implementation detail the user should not worry about (I
would just ensure the required tool is available).

> - else
> -     die "Error: No X11 or Wayland display detected"

Shouldn't the script return an error if neither tools are found?

Cheers,


More information about the Password-Store mailing list