Bug 202547

Summary: [patch] net-im/licq-qt-gui: qt4-linguist -> qt4-linguisttools, use OPTIONS helpers
Product: Ports & Packages Reporter: Andriy Voskoboinyk <avos>
Component: Individual Port(s)Assignee: Dirk Meyer <dinoex>
Status: Closed FIXED    
Severity: Affects Only Me CC: avos
Priority: --- Keywords: patch
Version: LatestFlags: avos: maintainer-feedback?
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch.diff
none
portlint.log
none
poudriere-9.3-i386-hunspell.log
none
poudriere-9.3-i386-kde.log
none
poudriere-10.2-amd64-default.log
none
Makefile.diff
none
portlint.log
none
poudriere-10.2-amd64-default.log
none
poudriere-9.3-i386-kde.log
none
poudriere-9.3-i386-hunspell.log none

Description Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-21 14:48:33 UTC
Created attachment 160173 [details]
Patch.diff

- Add HUNSPELL option (disabled by default, not used when KDE is selected).
- Use OPTIONS helpers where possible.
- Use qt4-linguisttools instead of qt4-linguist for *.ts files.
- Depend on qt4-xml and qt4-gui even when KDE is selected (they are used anyway).
- Sort USE_XORG.
- Regenerate patches.
- Bump PORTREVISION.
Comment 1 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-21 14:49:16 UTC
Created attachment 160174 [details]
portlint.log
Comment 2 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-21 14:52:40 UTC
Created attachment 160175 [details]
poudriere-9.3-i386-hunspell.log
Comment 3 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-21 14:52:58 UTC
Created attachment 160176 [details]
poudriere-9.3-i386-kde.log
Comment 4 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-21 14:58:57 UTC
Created attachment 160177 [details]
poudriere-10.2-amd64-default.log
Comment 5 Dirk Meyer freebsd_committer freebsd_triage 2015-08-24 18:12:47 UTC
Do NOT rename the patch-files!

The order of XORG should match the order of the libs used in linking.
Sorting the list alphabetic is confusing,
it changes the order of dependencies and make verifying it more complex.
Comment 6 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-27 15:07:38 UTC
Created attachment 160408 [details]
Makefile.diff

Don't touch patches & USE_XORG (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=202547#c5)
Comment 7 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-27 15:08:28 UTC
Created attachment 160409 [details]
portlint.log
Comment 8 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-27 15:12:31 UTC
Created attachment 160410 [details]
poudriere-10.2-amd64-default.log
Comment 9 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-27 15:13:31 UTC
Created attachment 160411 [details]
poudriere-9.3-i386-kde.log
Comment 10 Andriy Voskoboinyk freebsd_committer freebsd_triage 2015-08-27 15:26:45 UTC
Created attachment 160412 [details]
poudriere-9.3-i386-hunspell.log
Comment 11 commit-hook freebsd_committer freebsd_triage 2015-09-05 15:15:56 UTC
A commit references this bug:

Author: dinoex
Date: Sat Sep  5 15:15:43 UTC 2015
New revision: 396181
URL: https://svnweb.freebsd.org/changeset/ports/396181

Log:
  - Add HUNSPELL option (enabled by default)
  - Use qt4-linguisttools instead of qt4-linguist for *.ts files.
  - Depend on qt4-xml and qt4-gui even when KDE is selected (they are used anyway).
  - Use OPTIONS helpers where possible.
  PR:		202547
  Submitted by:	Andriy Voskoboinyk

Changes:
  head/net-im/licq-qt-gui/Makefile