Bug 231122

Summary: ports-mgmt/portlint: Please only complain about INSTALLS_ICONS for gtk-based ports
Product: Ports & Packages Reporter: Yuri Victorovich <yuri>
Component: Individual Port(s)Assignee: Joe Marcus Clarke <marcus>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (marcus)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   

Description Yuri Victorovich freebsd_committer freebsd_triage 2018-09-03 17:44:36 UTC 
Testcase: multimedia/obs-studio

INSTALL_ICONS is only used for GTK applications:
> $ grep -r INSTALLS_ICONS Mk
> Mk/bsd.port.mk:.if defined(USE_GNOME) || defined(INSTALLS_ICONS)
> Mk/Uses/gnome.mk:# INSTALLS_ICONS	- If a GTK+ port installs Freedesktop-style icons to
. Mk/Uses/gnome.mk:.if defined(INSTALLS_ICONS)
> Mk/Uses/gnome.mk:.if defined(INSTALLS_ICONS)
Comment 1 Joe Marcus Clarke freebsd_committer freebsd_triage 2018-09-05 16:49:17 UTC 
Thanks for the example port.  This has been fixed in my CVS repo pending the next release.
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-09-16 17:51:46 UTC 
A commit references this bug:

Author: marcus
Date: Sun Sep 16 17:51:44 UTC 2018
New revision: 479898
URL: https://svnweb.freebsd.org/changeset/ports/479898

Log:
  Update to 2.18.4.

  * Do not complain about icon installation if USE_QT is defined [1]
  * Add support for checking LICENSE_PERMS [2]
  * Add a warning when NO_PACKAGE is used [2]
  * Per the Porter's handbook, check for sections for USE/USES and FLAVORS [3]
  * Check for -on/-off in OPTION names [4]
  * Check to make sure the correct LICENSE_FILE_foo is specified [5]

  PR:		231122 [1]
  		231155 [2]
  		230594 [3]
  		229280 [4]
  		231303 [5]
  Submitted by:	loader [3]
  		adamw [4]

Changes:
  head/ports-mgmt/portlint/Makefile
  head/ports-mgmt/portlint/src/portlint.pl
Comment 3 Joe Marcus Clarke freebsd_committer freebsd_triage 2018-09-16 17:53:01 UTC 
Fixed in 2.18.4.