Bug 204397
| Summary: | security/keepass: Depends on x11/xsel but expects x11/xsel-conrad's CLI api | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Alex Zhukov <baron.pampa> | ||||||
| Component: | Individual Port(s) | Assignee: | Jason Unovitch <junovitch> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Only Me | CC: | adamw, junovitch, portmaster, woodsb02 | ||||||
| Priority: | --- | Keywords: | easy, patch, patch-ready | ||||||
| Version: | Latest | Flags: | woodsb02:
maintainer-feedback+
junovitch: merge-quarterly+ |
||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| Attachments: |
|
||||||||
|
Description
Alex Zhukov
2015-11-09 11:12:34 UTC
Created attachment 163377 [details]
Patch to update security/keepass port to use x11/xsel-conrad instead of x11/xsel
Thank you for reporting this problem and proposing the fix to depend on x11/xsel-conrad instead of x11/xsel. I have tested this and can confirm it does indeed fix the copy to clipboard. The attached patch will implement this change in dependency.
Created attachment 163378 [details]
QA: Successful poudriere testport logs with new x11/xsel-conrad dependency on FreeBSD current amd64
Koobs: this is ready to be committed. Can it please also be merged to quarterly? @Ben, I'd already set the merge-quarterly flag, so it's up to the committer who takes this issue to do so A commit references this bug: Author: junovitch Date: Wed Dec 23 00:40:33 UTC 2015 New revision: 404272 URL: https://svnweb.freebsd.org/changeset/ports/404272 Log: security/keepass: fix XSEL option dependency - Switch x11/xsel -> x11/xsel-conrad. This resolves a run time issue when copying passwords to the clipboard. PR: 204397 Reported by: Alex Zhukov <baron.pampa@gmail.com> Submitted by: Ben Woods <woodsb02@gmail.com> (maintainer) MFH: 2015Q4 Changes: head/security/keepass/Makefile (In reply to Ben Woods from comment #3) Awaiting approval on the MFH now. (In reply to Jason Unovitch from comment #6) > Awaiting approval on the MFH now. If it is my permission as maintainer that you are waiting for to MFH, you have it. Bugzilla will not let me set the merge-quarterly flag to a positive status. (In reply to Jason Unovitch from comment #6) > Awaiting approval on the MFH now. If it is my permission as maintainer that you are waiting for to MFH, you have it. Bugzilla will not let me set the merge-quarterly flag to a positive status. A commit references this bug: Author: junovitch Date: Wed Dec 23 20:18:57 UTC 2015 New revision: 404317 URL: https://svnweb.freebsd.org/changeset/ports/404317 Log: MFH: r404272 security/keepass: fix XSEL option dependency - Switch x11/xsel -> x11/xsel-conrad. This resolves a run time issue when copying passwords to the clipboard. PR: 204397 Reported by: Alex Zhukov <baron.pampa@gmail.com> Submitted by: Ben Woods <woodsb02@gmail.com> (maintainer) Approved by: ports-secteam (feld) Changes: _U branches/2015Q4/ branches/2015Q4/security/keepass/Makefile All, thanks for working together on finding and testing the resolution to this! (In reply to Ben Woods from comment #8) Commits to the quarterly branch must be approved by ports-secteam or portmgr. In this case, I realized after going to bed that the following blanket approval for this as a "reliability" fix would have been appropriate and I could have immediately MFH'd the update. However, I got the approval today while I was at work anyway. "Backport of security and reliability fixes which only result in PORTREVISION bumps and no changes to enabled features." https://wiki.freebsd.org/ports-secteam#Blanket_Approval - Set PR to closed/fixed - Set merge-quarterly+ (approved by ports-secteam (feld)) |
