Bug 1996323 - Pass missing dep on wl-clipboard
Summary: Pass missing dep on wl-clipboard
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: pass
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Christophe Fergeau
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 2002593 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-08-21 20:27 UTC by Jerry James
Modified: 2021-11-12 02:06 UTC (History)
8 users (show)

Fixed In Version: pass-1.7.4-3.fc36 pass-1.7.4-3.fc35 pass-1.7.4-3.fc33 pass-1.7.4-2.fc34
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-11-12 00:38:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Jerry James 2021-08-21 20:27:25 UTC
Description of problem:
I tried using pass for the first time since the update to version 1.7.4 came through:

$ pass -c <key name>
/usr/bin/pass: line 180: wl-copy: command not found
Error: Could not copy data to the clipboard

Since wl-copy is provided by the wl-clipboard package, this indicates that the pass packages should have "Requires: wl-clipboard".

Version-Release number of selected component (if applicable):
pass-1.7.4-1.fc34.noarch

How reproducible:
Always.

Steps to Reproduce:
1. pass -c <key name>

Actual results:
The error shown above.

Expected results:
The password is copied to the clipboard.

Additional info:

Comment 1 Christophe Fergeau 2021-08-23 10:03:47 UTC
The Requires is there as a conditional one:

$ dnf repoquery --requires  pass-1.7.4-1.fc34.noarch

(xclip or wl-clipboard)
[snip]

However, this does not correctly express pass needs, as it requires the use of wl-clipboard if $WAYLAND_DISPLAY is set, and of xclip if $DISPLAY is set.
We can probably add a `Requires: wl-clipboard` as its only wayland-specific dependency is libwayland-client which has no further deps.
Not sure what to do about xclip though. Maybe just a weak dep?

Comment 2 Jiri Konecny 2021-08-23 12:06:47 UTC
Hello,

After a consultation with expert. I think the best solution could be something like:

Requires: (wl-clipboard if libwayland-client else xclip)
Requires: (xclip if xorg-x11-server-Xorg else wl-clipboard)

Comment 3 Christophe Fergeau 2021-08-23 12:35:54 UTC
Ah, I did not know this was possible, thanks a lot! Do you know why the `else xclip`/`else wl-clipboard` fallbacks are needed? It seems to me that
```
Requires: (wl-clipboard if libwayland-client)
Requires: (xclip if xorg-x11-server-Xorg)
```

should work well enough, especially for headless setups? (though I'm not sure how well pass would work there)

Comment 4 Christophe Fergeau 2021-08-23 12:37:44 UTC
'libx11' might be better than 'xorg-x11-server-Xorg', this is what xclip depends on, and similar to libwayland-client I believe?

Comment 5 Christophe Fergeau 2021-09-09 11:19:00 UTC
*** Bug 2002593 has been marked as a duplicate of this bug. ***

Comment 6 Fedora Update System 2021-09-09 11:33:53 UTC
FEDORA-2021-7822811209 has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2021-7822811209

Comment 7 Fedora Update System 2021-09-09 11:35:06 UTC
FEDORA-2021-7822811209 has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 8 Christophe Fergeau 2021-09-09 11:36:37 UTC
I've only pushed the build to rawhide for now, this will need to be merged back to all the stable release once it's seen a bit of testing.

Comment 9 Raphael Kubo da Costa 2021-10-29 12:17:15 UTC
I see that this has been fixed for F33, F35 and F36, but the F34 update is still pending in https://koji.fedoraproject.org/koji/buildinfo?buildID=1844157

Is there something blocking it? I also couldn't find 1.7.4-2 in Bodhi.

Comment 10 Raphael Kubo da Costa 2021-10-29 12:18:00 UTC
> I also couldn't find 1.7.4-2 in Bodhi.

Sorry, I meant 1.7.4-2.fc34

Comment 11 Fedora Update System 2021-11-03 14:38:23 UTC
FEDORA-2021-01fcc023c1 has been submitted as an update to Fedora 33. https://bodhi.fedoraproject.org/updates/FEDORA-2021-01fcc023c1

Comment 12 Fedora Update System 2021-11-03 14:39:07 UTC
FEDORA-2021-4f95aded14 has been submitted as an update to Fedora 34. https://bodhi.fedoraproject.org/updates/FEDORA-2021-4f95aded14

Comment 13 Fedora Update System 2021-11-03 14:39:20 UTC
FEDORA-2021-126b5679df has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2021-126b5679df

Comment 14 Christophe Fergeau 2021-11-03 14:40:18 UTC
> Is there something blocking it?

Nothing special, just that I made the builds, switched to something different while they were being completed, and never filed the bodhi updates :)

Comment 15 Fedora Update System 2021-11-04 13:56:47 UTC
FEDORA-2021-01fcc023c1 has been pushed to the Fedora 33 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-01fcc023c1`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-01fcc023c1

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 16 Fedora Update System 2021-11-04 14:08:40 UTC
FEDORA-2021-126b5679df has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-126b5679df`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-126b5679df

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 17 Fedora Update System 2021-11-04 14:13:48 UTC
FEDORA-2021-4f95aded14 has been pushed to the Fedora 34 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-4f95aded14`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-4f95aded14

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 18 Dominik 'Rathann' Mierzejewski 2021-11-05 23:25:04 UTC
(In reply to Jiri Konecny from comment #2)
> Hello,
> 
> After a consultation with expert. I think the best solution could be
> something like:
> 
> Requires: (wl-clipboard if libwayland-client else xclip)
> Requires: (xclip if xorg-x11-server-Xorg else wl-clipboard)

This is wrong. Having libwayland-client installed doesn't imply you're actually running a Wayland session.

libwayland-client is pulled as dependency on many essential packages even in a Xorg-only desktops such as MATE.

Please just add Recommends: on both xclip and wl-clipboard and be done with it.

Comment 19 Fedora Update System 2021-11-12 00:38:05 UTC
FEDORA-2021-126b5679df has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 20 Fedora Update System 2021-11-12 00:58:48 UTC
FEDORA-2021-01fcc023c1 has been pushed to the Fedora 33 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 21 Fedora Update System 2021-11-12 02:06:55 UTC
FEDORA-2021-4f95aded14 has been pushed to the Fedora 34 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.