Bug 198785
| Summary: | [PATCH] Update security/qtkeychain and deskutils/mirall. | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Gleb Popov <arrowd> | ||||||||
| Component: | Individual Port(s) | Assignee: | Guido Falsi <madpilot> | ||||||||
| Status: | Closed FIXED | ||||||||||
| Severity: | Affects Only Me | CC: | madpilot, miguelmclara | ||||||||
| Priority: | --- | Keywords: | patch | ||||||||
| Version: | Latest | ||||||||||
| Hardware: | Any | ||||||||||
| OS: | Any | ||||||||||
| Attachments: |
|
||||||||||
Created attachment 154657 [details]
Updated patch
After some discussion with mail/trojita maintainer, I've decided to support both Qt versions in qtkeychain port, so i made a slave port.
Updated patch attached.
Bump. Take this report for testing and committing. I'll wait for an updated patch which can be applied to the present ports tree if possible, thanks. Created attachment 157044 [details] Updated patch Done. Please, don't forget to close this PR too: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=198668 Thanks. Will work on it in the next few days. *** Bug 198668 has been marked as a duplicate of this bug. *** A commit references this bug: Author: madpilot Date: Sun May 24 21:03:22 UTC 2015 New revision: 387341 URL: https://svnweb.freebsd.org/changeset/ports/387341 Log: - Rename security/qtkeychain to qtkeychain-qt5, update to 0.5.0 and make it use Qt5 - Add slave port qtkeychain-qt4 - Make deskutils/owncloudclient link against qtkeychain-qt5 - Update owncloudclient comment and pkg-descr PR: 198785 Submitted by: 6yearold at gmail.com (security/qtkeychain maintainer) Changes: head/MOVED head/deskutils/owncloudclient/Makefile head/deskutils/owncloudclient/pkg-descr head/deskutils/owncloudclient/pkg-plist head/security/Makefile head/security/qtkeychain/ head/security/qtkeychain-qt4/ head/security/qtkeychain-qt4/Makefile head/security/qtkeychain-qt5/ head/security/qtkeychain-qt5/Makefile head/security/qtkeychain-qt5/distinfo head/security/qtkeychain-qt5/files/ head/security/qtkeychain-qt5/pkg-plist head/security/qtkeychain-qt5/pkg-plist.qt4 head/security/qtkeychain-qt5/pkg-plist.qt5 Hi, Thanks for your submission. I took the freedom to update qtkeychain ports to version 0.5.0, which is now available. I also modified your submission, moving the port to qtkeychain-qt5, it goes along better with the qtkeychain-qt4 port. The owncloudcloud client port needs a portrevision bump after this change. A few more things I had to change: - I did not enable the owncloud own SSP support. SSP is already handled by the ports system which adds the correct flags to enable it whenever possible. At present there are some problems with it with gcc built ports for i386 on 9.x and 8.x, where the ports system does not enable it, you can find more information in Mk/bsd.ssp.mk - The plists had a few issues, first of all pkg is now able to handle directories by itself, except rare cases, so @dirrm directives are deprecated (check CHANGES entry dated 20140922), you also missed a pair of files. - You also missed some dependencies. Please always run portlint on the ports before submitting, this will warn you about most formal errors and deprecated features. To spot missing dependencies the best solution is to use poudriere. |
Created attachment 154648 [details] Patch This patch updates mirall and qtkeychain ports to their most recent version and also switches them to Qt 5. I already have maintainership on qtkeychain and this with patch assume it on mirall too. Additionally, this patch renames deskutils/mirall to deskutils/owncloud-client since it has been renamed upstream. This PR supersedes PR 198668, so it can be closed too when this is commited. Based on Tobias Berner (tcberner) work, original patches are by him.