Bug 223498

Summary: ports-mgmt/portlint: Don't warn about INSTALLS_ICONS when the project isn't GTK+
Product: Ports & Packages Reporter: Yuri Victorovich <yuri>
Component: Individual Port(s)Assignee: Joe Marcus Clarke <marcus>
Status: Closed FIXED    
Severity: Affects Only Me CC: adridg, rakuco, tcberner
Priority: --- Flags: bugzilla: maintainer-feedback? (marcus)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch to suppress INSTALLS_ICONS warning for Qt5 ports none

Description Yuri Victorovich freebsd_committer freebsd_triage 2017-11-07 18:51:11 UTC 
Currently, it always warns:
> WARN: /usr/ports/multimedia/simplescreenrecorder/pkg-plist: [62]: installing icons, please define INSTALLS_ICONS as appropriate

Handbook says that it is only needed for GTK+ ports: https://www.freebsd.org/doc/en/books/porters-handbook/using-gnome.html

You should print the above warning only for GTK+ ports, but warn when INSTALLS_ICONS is set for non-GTK+ ports.
Comment 1 Yuri Victorovich freebsd_committer freebsd_triage 2017-11-07 19:26:28 UTC 
Another thing is that the warning is printed a  lot of times. One time is enough.
Comment 2 Adriaan de Groot freebsd_committer freebsd_triage 2017-11-17 12:26:21 UTC 
Created attachment 188072 [details]
Patch to suppress INSTALLS_ICONS warning for Qt5 ports

This patch adds a `needs_installs_icons()` which states whether a port needs INSTALLS_ICONS when there are icons present. It's pessimistic, to preserve existing warnings and only suppress the warning for Qt5-based applications.

Additionally, checks for INSTALLS_ICONS when it isn't needed (i.e. Qt5).

The `needs_installs_icons()` function could be massaged to really check only for GNOME ports, but this seems simpler.
Comment 3 Adriaan de Groot freebsd_committer freebsd_triage 2017-11-17 12:27:25 UTC 
Adding tcberner@ (mentor for me and yuri) and rakuco@ (mentor for me).
Comment 4 Tobias C. Berner freebsd_committer freebsd_triage 2017-11-17 19:23:27 UTC 
This sounds like a good idea.
Comment 5 Joe Marcus Clarke freebsd_committer freebsd_triage 2017-12-28 22:39:36 UTC 
Fix committed to my repo pending the next release.
Comment 6 Joe Marcus Clarke freebsd_committer freebsd_triage 2017-12-28 23:47:30 UTC 
Committed, thanks!