Bug 231420 - ports-mgmt/portlint: Wrong warning: extra item placed in the *_DEPENDS section, for example, "py34_"
Summary: ports-mgmt/portlint: Wrong warning: extra item placed in the *_DEPENDS sectio...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Joe Marcus Clarke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-17 08:22 UTC by Yuri Victorovich
Modified: 2018-10-13 15:39 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (marcus)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yuri Victorovich freebsd_committer 2018-09-17 08:22:32 UTC
Testcase: textproc/py-orange3-text

py34_RUN_DEPENDS= is a valid syntax for the xx_DEPENDS section, and portlint shouldn't complain about it.

Version 2.18.4.
Comment 1 Joe Marcus Clarke freebsd_committer 2018-10-13 12:50:21 UTC
Fixed in my CVS repo pending the next release.
Comment 2 commit-hook freebsd_committer 2018-10-13 15:38:53 UTC
A commit references this bug:

Author: marcus
Date: Sat Oct 13 15:38:08 UTC 2018
New revision: 481991
URL: https://svnweb.freebsd.org/changeset/ports/481991

Log:
  Update to 2.18.5.

  * Do not complain about extra FLAVORS in *_DEPENDS. [1]
  * Do not fully expand variable dependencies to allow for more granular checks. [2]
  * Allow GH_* in USES section. [3]
  * Check for duplication between *_USES and USES. [4]
  * Check for incorrect use of *_CMAKE_ARGS [5]
  * Check that CMAKE_BOOL arguments do not contain a -D flag [5]
  * Check for use of autoplist and pkg-plkist or PLIST_FILE. [6]

  PR:		231420 [1]
  		231421 [2]
  		231422 [3]
  		231566 [4]
  		231894 [5]
  		231865 [6]
  Submitted by:	db [3] [6]

Changes:
  head/ports-mgmt/portlint/Makefile
  head/ports-mgmt/portlint/src/portlint.pl
Comment 3 Joe Marcus Clarke freebsd_committer 2018-10-13 15:39:01 UTC
Fixed in 2.18.5.