11:45 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.22.1

Remove the WWW check for pkg-descr.

PR:		278069

    2157107

17:30 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.22

This fixes some spurious errors by concatenating continued lines in the
Makefile.  The downside of this is that line numbers will get more
askew.  I thought I'd get some time to imrove this, but didn't.  And
this fix is important to get out now.

PR:		275075

    7c58302

19:02 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.21.0

* Soften the desktop-file-utils message [1]
* Avoid a bogus error on ports that append to DISTFILES. [2]
* Colorize output for warnings and errors. [3]

PR:		271623 [1]
		268681 [2]
		274981 [3]
Inspired by:	yuri [3]

    045f999

13:50 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.20.0

* Check for duplicate distinfo items [1]
* Remove vestiges of SVN [2]

[2] Switch to using `git status` to determine if a file is under git version
control.  Also, PL_SVN_IGNORE has been renamed to PL_GIT_IGNORE.

PR:		267168 [1]
		267145 [2]
Submitted by:	rodrigo [1]
Thanks to:	pauamma for the `git status` command [2]

    ce9d055

14:03 Joe Marcus Clarke (marcus)  Author: Stefan Eßer

ports-mgmt/portlint: Update to 2.19.14

Remove some stale WWW warnings for pkg-descr.

    2197afc

17:24 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.13

* Allow for a plus ('+') in other places in a PORTVERSION [1]
* Check for .dkestop file installation [2]
* Support the new Makefile WWW macro [3]

PR:		265364 [1]
		265551 [2]
		266068 [3]
Submitted by:	Robert Clausecker [1]
		se [3]

    9800743

15:37 Joe Marcus Clarke (marcus)  Author: Stefan Eßer

ports-mgmt/portlint: Update to 2.19.12

In a discussion with members of the port managers team (tcberner, rene) the
issue of conflicting package base names came up.

I have offered to create portlint checks for 2 possible Makefile issues:

1) Conflicting use of ${PKGBASE} of an existing port.

2) Multiple flavors resulting in the same ${PKGBASE}.

The attached patch adds checks for both possible issues:

1) Check for the existence of the new PKGBASE in the INDEX file (if present) and
warn if it is used for a different ORIGIN.

2) Generate PKGBASE for all FLAVORS and check for duplicates.

PR:		263807

    415d07a

20:40 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.11

* Do not strip modifiers from variables when dereferencing.

PR:		262941

    3ac754d

20:06 Stefan Eßer (se) 

ports-mgmt/portlint: remove "conflicts with itself"

The warning about a CONFLICTS entry matching the package being built
is bogus - the ports system excludes the port being built from the
list of conflicts that is reported by "make list-install-conflicts"
and the other targets that check for conflicts.

It is in fact quite useful to specify CONFLICTS with wildcards that
include the port being built instead of explicitly listing all other
versions (e.g. in case of the mysql or mariadb ports - it is possible
to use mysql[0-9][0-9]-client for example in the mysql57-client port
instead of complex patterns like "mysql5[56]-client mysql80-client"
in an attempt to not match the "57").

Since I'm fixing CONFLICTS entries in a large number of ports and
want to use wildcards as explained above to simplify the entries and
to reduce the maintenance effort (often are CONFLICTS entries stale
when new versions of e.g. mysql are imported).

The bogus portlint messages let maintainers think that the wildcards
need to be adjusted to prevent this "self conflict", leading them to
put back the unnecessarily complex patterns.

The author of portlint has not provided any kind of response to my
request to remove the misleading output sent more than 6 weeks ago.

Due to a maintainer time-out I'm removing the bogus message. The test
and print command are commented out in order to only apply the minimal
change required. The version number is unchanged since I do not want
to introduce a version that does not exist in the upstream sources,
but a portrevision is added to enforce the creation of an updated
package in the official repository.

    52ddb47

19:45 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.10

* Don't check for direct use of commands in PORTNAME [1]
* Split on first '@' to avoid breaking directory detection [2]
* Remove checks for deprecated INSTALLS_ICONS [3]

PR:		229627 [1]
		260246 [2]
		261556 [3]
Submitted by:	gerald [1]
		Derek Schrock <dereks@lifeofadishwasher.com> [2]

    c33aae9

18:19 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.9

* Update CONFLICTS check to cover all variables and checks performed by
  bsd.port.mk [1]
* Add some other allowed fully-qualified paths [2]
* Don't consider variable substitution when checking DEPENDS [3]

PR:		259813 [2]
		259244 [3]
Sybmitted by:	se [1]
		gerald [2]
Differential Revision:	https://reviews.freebsd.org/D32733 [1]

    c8fa510

17:12 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.8

* Warn about deprecated USE_AUTOTOOLS [1]
* Add support for BROKEN_*

PR:		248487 [1]
		257951 [2]

    772ba6c

17:56 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.7

* Account for LEGAL_TEXT and per-ARCH DISTFILES in the Makefile sections
  [1]
* Relax USE_LDCONFIG checks now that we don't need a specific lib
  pattern [2]

PR:		255764 [1]
		255651 [2]

    c810710

13:45 Rene Ladan (rene) 

cleanup: remove LEGAL and (support in) related tools.

LEGAL is badly maintained, LEGALlint notices quite a number of ports
which mismatch their line in LEGAL. The port itself is always leading.

* math/giacxcas: restrict RESTRICTED_FILES to giac_* only

Differential Revision: https://reviews.freebsd.org/D30061

Reviewed by:	portmgr (mat), dch, eadler, marcus

    0b673d6

17:15 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.6

* Allow ports without any leading comment section [1]
  loader@ submitted a patch on which this was based
* Add a check to make sure the license is actually valid
PR:		255520 [1]

    122c3ec

20:49 Joe Marcus Clarke (marcus) 

ports-mgmt/portlint: Update to 2.19.5

* Reverse $FreeBSD$ check logic to make the use of them FATAL [1]
* Remove embedded $FreeBSD$
PR:		255153 [1]
Obtained from:	lwhsu [1]
		With issue report from Tatsuki Makino

    f36c30b

14:31 Mathieu Arnold (mat) 

all: Remove all other $FreeBSD keywords.

    135fdee

19:22 marcus 

Updqte to 2.19.4.

* Correct path to apache.mk [1]
* Warn if one declares USE_LDCONFIG without a properly named shared object [2]
* Allow make use-before-declare lazy evaluation in dependencies [3]

PR:		252003 [1]
		252181 [2]
		244239 [3]
Submitted by:	brnrd [1]

 

17:24 marcus 

Update to 2.19.3.

* Do not complain about USE_LDCONFIG32 [1]
* Allow a master port to have its last line be ".endif" [2]
* Attempt to find OPTIONS use in a smarter way [3]

PR:		251431 [1]
		251430 [2]
		249970 [3]
Submitted by:	gerald [1]

 

15:17 marcus 

Update to 2.19.2.

The makevar padding method was missing leading and trailing variable values
when those values were undefined.  Try to be better about this.

 

23:20 marcus 

Properly fix typo to USE_GITLAB not USE_GITHUB.

 

23:15 marcus 

Update to 2.19.1.

Fix typos introduced in previous commits.

PR:		246881
		246882

 

13:46 marcus 

Update to 2.19.0.

* Do not be overly critical of macros which are used before declaration in
DEPENDS
* Do not treat comments to ONLY_FOR_ARCHS and NOT_FOR_ARCHS as sortable data [1]
* Check for PORTSDIR defined in /etc/make.conf [2]
* Do not warn about ${DISTNAME}${EXTRACT_SFX} in DISTFILES if
USE_GITHUB=nodefault [3]
* Thoroughly check for an OPTION description [4]
* Specify flag equivalency for -A in portlint(1) [5]

PR:		246346 [1]
		244807 [2]
		244741 [3]
		244480 [4]
		245338 [5]
Submitted by:	0mp [3]
		kevans [5]

 

22:23 marcus 

Update to 2.18.11

* No longer warn for manpages installed under share/man [1]
* Remove code to combine comments and blank lines in a section [2]
* Remove code to combine comments and blank lines [3]
* USE_PYQT sets the necessary QT USES [4]
* Only warn on lack of INSTALLS_ICONS when USES=gnome is set [5]
* Check for files that have been patched multiple times [6]

PR:		240359 [2]
		244480 [3]
		240445 [4]
		244039 [5]
		243600 [6]
Submitted by:	0mp [1] [5]
		vladimir.chukharev@gmail.com [4]
		swills [6]
Differential Revision:	D23896 [1]

 

15:07 marcus 

Update to 2.18.10.

* Relax wording around restrictive licensing [1]
* Remove the check for extra items in the USE/USES section [2]
  One is allowed to put any _related_ variables here, and this will always
change.
  It's easier not to check than to provide wrong recommendations.
* Fix the check for unspecified license files [3]

PR:		240233 [1]
		239045 [2]
		231303 [3]

 

13:25 marcus 

Update to 2.18.9.

* Add check for sensible CMAKE settings to ports-mgmt/portlint [1]
* Do not warn about absolute paths in *_OLD_CMD (when USES=shebangfix is set)
[2]
* If an option only has QMAKE_ON or QMAKE_OFF settings, it is flagged as no
effect [3]
* Check for options defined per ARCH to make sure we don't have a lot
  of false positives. [4]

(Note: [1] was already in PORTREVISION 2).

PR:		232948 [2]
		237045 [3]
		234888 [4]
Submitted by:	adridg [3]

 

12:08 adridg 

Add check for sensible CMAKE settings to ports-mgmt/portlint

If a port sets CMAKE variables (CMAKE_ARGS, or the option helpers
like foo_CMAKE_BOOL) then it should have USES=cmake; otherwise
those variables don't make any sense.

This is the slightly simplified version of the patch: there
really shouldn't be options that switch (meta-)build systems.

PORTREVISION bumped because I'm not sure what the versioning
scheme is for portlint.

PR:		235650
Reported by:	yuri
Approved by:	marcus (maintainer timeout 2 weeks)
Differential Revision:	https://reviews.freebsd.org/D19525

 

16:21 marcus 

Correct a typo in the previous commit.

 

16:19 marcus 

Update to 2.18.8

* Handle multiple licenses [1]
* Warn about leading and trailing whitespace in pkg-descr [2]

PR:		233774 [1]
		235070 [2]
Submitted by:	amdmi3 [1]
		swills [2]

 

20:05 marcus 

Remove the warnings around USE_PYTHON and autoplist.

You can use autoplist with a pkg-plist or PLIST_*.  Some ports require this
apparently.  Remove these warnings for now until a more robust
duplicate-checking solution can be implemented.

Reported by:	koobs

 

17:50 marcus 

Update to 2.18.6.

Fix an uninitialized variable warning in non-Python ports.

Reported by:	leres

 

15:38 marcus 

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]

 

17:51 marcus 

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]

 

12:49 marcus 

Fix breakage introduced in previous commits.

PR:		230427

 

18:13 adamw 

Reevert change accidentally included in r476184

 

16:33 adamw 

Fix a common grammar error: "can not" means the opposite of "cannot."

"Can not" means "it is possible not to," and "cannot" means "it is impossible
to."

 

16:13 marcus 

Update to 2.18.3.

* Be less strict in the check for MASTER_SITE_SUBDIR use.
  This change checks that the subdirectory used is actually different
  than what is defined in MASTER_SITES_SUBDIRS.  To make this easier to
  do, support has been added to single-quote variables passed to the
  get_makevar* functions. [1]

* Now only PY_FLAVOR is recommended as opposed to FLAVOR. [2]

* Remove an errant ':' when suggesting PY_FLAVOR. [3]

PR:		228227 [1]
Requested by:	mat [2]
Reported by:	koobs [3]

 

22:14 marcus 

Bump version to 2.18.2.

Fix a regression in the mae makepatch check.

Make sure only the patch header is checked, but make sure
to read in the entire patch file.  This regression was
introduced in 2.18.0.

PR:		228203

 

18:59 marcus 

Bump version to 2.18.1.

Soften the py-* dependency error around flavors.

Now the message is a warning and a suggestion. This is pending more discussion,
but this seems like a good compromise for now.

PR:		225654
Requested by:	antoine

 

21:33 marcus 

Update to 2.18.0.  Enough has happened that it's time for a minor bump.

* Fix the check for duplicate SHEBANG_LANG values [1]
* Check to make sure patch files end with newlines [2]
* Allow flavored *_DEPENDS in the DEPENDS section [3]
* Check that kde:4 OR kde:5 is defined with USE_KDE [4]
* Add a check to make sure pkg-descr is at least a certain number of lines [5]
* Fix a couple of typos [6] [7]
* Allow for any *_DEPENDS item in the DEPENDS section and fix a bug in the
  dependency syntax checking [8]
* Add a check to make sure Python dependencies include a FLAVOR [9]

PR:		226709 [1]
		226109 [2]
		226581 [3]
		227255 [5]
		227729 [6]
		227878 [7]
		227556 [8]
		225654 [9]
Submitted by:	tcberner [4]
		0mp [5][6][7]
Differential Revision:	D13723 [4]

 

21:05 marcus 

Update to 2.17.16

* Check for USE_PYQT when USES=pyqt:5 is defined [1]
* Avoid more bogus direct command use warnings.  Also try and fix some line
numbers [2]
* Check for over-verbose use of magic MASTER_SITES_SUBDIRS [3]
* Do not complain about missing Created by: [4]
* Avoid checking direct command usage in embedded comments [5]

PR:		225615 [1]
		225612 [2]
		225503 [3]
		225781 [5]
Reported by:	Jason Bacon <bacon4000@gmail.com> [4]

 

23:46 marcus 

Update to 2.17.15.

* Don't check for direct command use for ONLY_FOR_ARCHS_REASON
  and NOT_FOR_ARCHS_REASON when a specific arch has been specified. [1]
* Preserve an original variable value to make sure the check for copying
  dependecies is actually performed.
* Don't complain about incon installation if the port is a Qt5 port. [2]
* Check that PLIST_FILES entries do not contain %%FOO% notation. [3]
* Check that DOCS and EXAMPLES are defined as OPTIONS when using %%PORTDOCS%%
  and %%PORTEXAMPLES%%. [4]
* Check that if USE_LDCONFIG is defined, then the port installs shared
  objects. [5]
* Check that when USE_KDE is defined, then USES=kde:5 is also defined. [6]
* Ignore flavors when trying to find port directories. [7]
* Revert the fix for 221971 (USE_GITHUB).  More debate is needed here.

PR:		223285 [1]
		223498 [2]
		223539 [3]
		223541 [4]
		223762 [5]
		224140 [6]
		224245 [7]
Submitted by:	adridg [2]
		mat [4]
		224245 [7]

 

11:26 marcus 

Emergency update to 2.17.14.

Escape a variable in an RE to prevent corruption.

Submitted by:	gahr

 

17:58 marcus 

Update to 2.17.13.

* Do not complain about USE_GITHUB before USES [1]
* Make sure each variable in a dependency is fully expanded [2]
* Clean up the earlier checks for BROKEN, etc. to prevent bogus warnings [3]
* 'USE=execinfo' is removed with ports r450351. So stop suggesting to use it [4]

PR:		221971 [1]
		221605 [2]
		222409 [3]
		222653 [4]
Submitted by:	amdmi3 [1]
		jrm [2]
		Yasuhiro KIMURA <yasu@utahime.org> [4]

 

12:43 marcus 

Update to 2.17.12.

* Don't check for absolute paths inside make variables [1]
* Check pkg-plist for use of OPTIONS [1]
* Fix an issue with line number offsets [2]
* Don't alert on double-dollar (VARIABLE) [3]
* Allow BROKEN, IGNORE, DEPRECATED to appear before the *_DEPENDS block [4]
* Check that manpages are not installed into share/man [5]
* Check for a space anywhere between a variable and its value [6]

PR:		221507 [1]
		221291 [2]
		221397 [3]
		221377 [4]
		221455 [5]
		221219 [6]
Submitted by:	Dmitri Goutnik <dg@syrec.org> [4]

 

12:55 marcus 

Update to 2.17.11.

* Trim comments after USE_GCC [1]
* Add a check for redundant SHEBANG_LANG values [2]
* Make sure both USES and USE_* are actually seen before checking
  to see if one is included before the other [3]

PR:		221085 [1]
		220500 [2]
Submitted by:	gerald [1]
Reported by:	dbaio [3]

 

01:51 marcus 

Update to 2.17.10.

* Remove the other check for conflicts that are too broad [1]
* Remove the check for portlint that is more than 30 days old [2]
* Check to see uf USE_* comes before USES [3]

PR:		220087 [2]
		220340 [3]
Reminded by:	dbaio [1]

 

22:24 marcus 

Update to 2.17.9

* Remove a check for narrow conflicts now that we are in the post-pkg world [1]
* Genericize the USES_ sorting code and check to see if NOT_FOR_ARCHS
  and ONLY_FOR_ARCHS are sorted. [2]
* Fix the check for JAVALIBDIR so that it falls before the variable expansion
  code. [3]

PR:		219684 [1]
		218823 [2]
		219360 [3]
Submitted by:	eugen [3]

 

19:50 marcus 

Update to 2.17.8.

* Check for use of OPT_XXX_OFF syntax to prevent false incorrect option
  usage assertions. [1]
* Remove now-obsolete warning about using :LU operators with make(1). [2]
* Fix the regex to prevent direct command use from hittin on lines
  like FOO_USES=... [3]

PR:		218260 [1]
		218325 [2]
		218672 [3]
Submitted by:	jrm [2]
Reported by:	rezny [2]

 

15:28 marcus 

Update to 2.17.7.

* Fold in the same removal of the obsolete HTTP-only warning
* Add support for checking for direct dependency on bison, fmake, and
libexecinfo. [1]

PR:		216268 [1]

 

13:35 amdmi3 

- Remove useless warning

HTTPS-only is normal novadays, also may@ says that the warning was
there because before 8.4 libfetch did not support going through a
proxy for HTTPS

PR:		217995
Approved by:	maintainer timeout (marcus, 2 weeks)
Differential Revision:	D10015

 

23:19 marcus 

Update to 2.17.6.

Add support for CMAKE_BOOL

PR:		215203
Submitted by:	Dmitri Goutnik <dg@syrec.org>

 

02:13 marcus 

Correct a typo with TIMESTAMP handling.  While here, don't use a pure magic
number
for the 30 days worth of seconds.

PR:		212091
Submitted by:	jrm

 

00:39 marcus 

Update to 2.17.5.

* Allow https URLs for the WWW line in pkg-descr.  Why are we only NOW doing
  this? [1]
* Check TIMESTAMP to make sure it's in the present/past, but not more than 30
  days in the past. [2]

PR:		212091 [2]
Submitted by:	amdmi3 [1]
Differential Revision:	D8462 [1]

 

14:24 marcus 

Update to 2.17.4.

* Check PLIST_FILES for icons [1]
* Add VARS and VARS_OFF to the option helper list [2]
* Add NO_ARCH [2]

PR:		211175 [1]
Submitted by:	koobs [2]

 

17:42 marcus 

Update to 2.17.3.

This release fixes a bug in newer versions of Perl where '{' and '}' caused
regexp compilation to fail.

PR:		210809
Submitted by:	mat

 

18:43 marcus 

Update to 2.17.2.

* Properly check for TIMESTAMP in distinfo

 

18:22 marcus 

Bump version to 2.17.1 for previous commit and sync with my local
repo.

 

16:09 bapt 

Make portlint not yelling at TIMESTAMP in distinfo

PR:		209522
Reported by:	John W. O'Brien <john@saltant.com>

 

18:30 marcus 

Update to 2.17.0.

* Drop the need for PORTSDIR in dependencies [1]
* Allow a trailing ':' in URLs [2]
* Check to make sure the value(s) of SUB_FILES is healthy [3]
* Expand variables on all sides of the dependency tuple [4]
  (this last change is rather big, and thus bumping minor version)

PR:		208272 [1]
		207843 [2]
		207477 [3]
		206061 [4]
Submitted by:	amdmi3 [1]

 

21:16 marcus 

Update to 2.16.8.

* Attempt to properly detect gettext [1]
* Add support for @<pre|post>[un]exec plist directives [2]
* Add support for the new @sample plist directive changes [2]
* Modify the text when checking for used OPTIONS [3]

PR:		204370 [1]
		204900 [2]
		204575 [3]

 

17:27 marcus 

Update to 2.16.7.

* Add support for @(...) notation [1]
* Remove the check for USES being sorted.  Order is important. [2]
* Add support for making sure @owner and @group are properly reset [3]

PR:		202570 [1]
		203908 [2]
		202711 [3]

 

22:23 marcus 

Update to 2.16.6.

* Only report "make makepatch" once per patch file [1]
* Attempt to re-word the make makepatch warning to avoid unnecessary repo
  churn [2]
* Attempt to test some other macros for sorting [3]
* Do not check empty dependencies for validity [4]

PR:		202050 [1]
		201647 [2]
		201409 [3]
		202052 [4]
Submitted by:	gerald [1]

 

14:51 marcus 

Update to 2.16.5.

* Add a check for BROKEN similar to IGNORE [1]
* Quiet the noise when used with Perl 5.22 [2]
* Fix a bug checking for USES sortedness [3]
* Correct a logic bug checking for USE_ANT [4]

PR:		200634 [2]
		200792 [3]
		200421 [4]
Submitted by:	pi [2]
		John Hein <z7dr6ut7gs@snkmail.com> [4]
Reported by:	koobs [1]

 

22:44 marcus 

Update to 2.16.4

* Correct a typo.  This should have been SITE_ARCH_REL. [1]
* Pre-populate USES with an explicit grep of the Makefile. [2]
* Correct a typo to prevent direct command usage checking in pkg-* files. [3]
* Adjust the order of first-paragraph items so that proper checking is done. [4]
* Adjust the warning about installing docs by wrapping with
  .if ${PORT_OPTIONS:MDOCS}. [5]
* Add a check for ${SETENV} instead of ${ENV}. [5]

PR:		199974 [2]
		199693 [3]
		199428 [4]
		199417 [5]
Submitted by:	adamw [4]
		sunpoet [1]

 

04:53 marcus 

Update to 2.16.3.

* Check the patches to make sure they were generated using ``make patch'' [1]
* Do not throw a bunch of Perl errors if the makefile is bad [2]

PR:		197747
		198887

 

17:10 marcus 

Update to 2.16.2.

* Exclude .git directories from error reporting [1]
* Add some additional Perl checks for STAGEDIR [2]
* Try not to flag on direct use of "install" if we're running a make command [3]
* Fix the checks for Perl given the new hierarchy [4]

PR:		195780 [1]
		194759 [3]
		196222 [4]
Submitted by:	gerald [3]
		Yasuhiro KIMURA <yasu@utahime.org> [1]
Requested by:	mat [2]

 

16:02 marcus 

Update to 2.16.1.

Add an additional check for EXTRACT_SUFX.

Submitted by:	ak

 

18:34 marcus 

Update to 2.16.0

* Remove @MASTERSITES_WHITELIST
* Remove some remnants of multi-OS support: always false conditions and
  unused variables ($automan, $autoinfo, $manstrict, $mancompress,
  $multiplist, $newxdef, $rcsidinplist, $ldconfigwithtrue)
* Remove check for USE_SIZE
* Remove check for 'Version Required' in header
* Remove pkgtools remnants: fix pkg_version invocations,
  remove check for PKGINSTALLVER, remove check for old packages in $portsdir
* Update check for IGNOREFILES macro
* Remove comment about EXEC_DEPENDS
* Remove check for NO_CONFIGURE/NO_PATCH
* Remove '@dirrm for %%DOCSDIR%%' check
* Remove pkg-comment check
* Remove check for /usr/X11R6
* Remove check for gcc_val 3.x and 4.1 conditions
* STAGE is now required
* Remove MANLANG check
* Add NOPORTEXAMPLES/NOPORTDOCS deprecation warning
* Add @stopdaemon, @cwd deprecation warning
* Add PLIST_DIRSTRY deprecation warning
* Add check for make :L :U modifiers
* Add check for for MAN[1-9] macro
* Allow use of absolute path in plist
* Fix checks for EXTRACT_SUFX
* Fix USE_ANT and USES=gmake warning
* Convert WITH(OUT)_$1 to PORT_OPTIONS:M$1 in warning
* Convert USE_DOS2UNIX to USES=dos2unix in warning
* Convert USE_ZIP to USES=zip in EXTRACT_SUFX check
* Convert USE_PERL5 to USES=perl5 in warning
* Convert USE_ICONV to USES=iconv in warning
* Modify checkplist() to accept @fc/@shell/other new keywords

PR:		194285
Submitted by:	ak

 

05:53 marcus 

Update to 2.15.6.

* Check to make sure each instance of USES is properly sorted.

PR:		193792

 

23:40 marcus 

Update to 2.15.5.

* Do not report on DEBUG as an OPTION. [1]
* Check for use of muted INSTALL_foo commands.
* Warn on use of @dirrm[try].
* Check to make sure rc.d scripts contain a '# PROVIDE:' line. [2]
* Check to see that the options provided to USES are sorted. [3]
* Fix an errant "direct use of command" warning. [4]
* Remove the check to see if StartupNotify is the empty string in
  DESKTOP_ENTRIES.  This is allowed per the spec and per the porter's
  handbook. [5]
* Check for empty lines in plist. [6]
* Check for direct command use in SHEBANG_FILES. [7]

PR:		192525 [1]
		193464 [2]
		193792 [3]
		193793 [4]
		193456 [5]
		193794 [6]
		193795 [7]

 

13:21 marcus 

Update to 2.15.4.

Use File::Basename so that only the patch filename is checked for the 100
character limit.

PR:		192161

 

14:52 marcus 

Update to 2.15.3.

* Make sure a port is placed in its proper category directory. [1]
* Don't check INSTALL_TARGET for direct command use. [2]
* Remove the single MASTER_SITE warning. [3]
* Check to make sure patch file names contain no more than 100 characters. [4]

PR:		191631 [2]
		191079 [3]
Submitted by:	wen heping <wenheping@gmail.com> [1]
		gerald [2]
Requested by:	demon [4]
Discussed on:	ports@ [3]

 

05:35 bapt 

Convert GMAKE to MAKE_CMD
Please note that lots of invocation of MAKE_CMD here are wrong as they do not
properly respect MAKE_ENV and friends

With hat:	portmgr

 

19:24 marcus 

Update to 2.15.2.

* Add s3.amazonaws.com to the list of whitelist master sites. [1]
* Correct a typo. [2]
* Add support for the @sample directive. [3]
* Check for USE_AUTOTOOLS=libtool and suggest USES=libtool instead.

PR:		183864 [3]
Submitted by:	skreuzer [1]
		culot [2]

 

18:39 marcus 

Update to 2.15.1.

* Remove www.freebsd.org/~$user check from urlcheck()
* Remove 'in the module alias name' warning
* Remove 'binary -ko'
* Silence warnings in case if svn binary does not exist (try 2)
* Use more strict regexp for share/doc detection
* Add sdl-config to direct commands check
* Fix phase 2 of direct use of pathnames check
* Ignore comments in direct use of pathnames check
* Support nested knobs in plist for OPTIONS_SUB check
* Add check for @dirrm(try) libdata/pkgconfig in plist
* Remove USE_MESA, INSTALLS_SHLIB, APACHE_COMPAT, USE_XPM from deprecated
  macro check
* Fix 'Consider adding support for a NLS knob to conditionally disable gettext'
  warning in the case of use 'NLS_USES= gettext'

PR:		188389
Submitted by:	ak

 

20:08 marcus 

Update to 2.15.0.

- Remove NetBSD support, its completely bitrotted
- Suggest to set DEVELOPER=yes in /etc/make.conf if committer flag specified in
command line
- Fix WARN: Makefile: [11]: possible direct use of command "libtool" found...
- Small fixes:
  * 'WWW: UR:' -> 'WWW: URL'
  * @option_helpers -> remove leading '_' use _$m instead
  * fix a few 'undefined variable' warnings
- Make MAN{X} checks NO_STAGE only (consolidate/simplify check)
- Make some checks in checkplist NO_STAGE only
- Convert PERL_CONFIGURE check to USE_PERL5=configure
- Make "use .if ${PORT_OPTIONS:MDOCS} to wrap installation of files into
  $localbase/share/doc" warning NO_STAGE only
- Add special case for global options(DOCS/EXAMPLES/NLS/IPV6) in
  OPTIONS_DEFINE, but not in PORT_OPTIONS:M$i
- Check for relative path (/../) in *DEPENDS
- Remove MASTER_SITES that don't resolve or returns 5xx
- Remove LINTSKIP check that was commented since 1997

PR:		186639
Submitted by:	ak

 

19:59 marcus 

Update to 2.14.10.

* Check for old-style LIB_DEPENDS
* Don't emit a warning if there is only one MASTER_SITE, but that MASTER_SITE
  is known in our list of site macros. [1]
* Attempt to find corrupt DESKTOP_ENTRIES that do not specify a proper
  startupnotify value. [1]

Reported by:	Hardy Schumacher <hardy.schumacher@gmx.de>

 

05:52 marcus 

Update to 2.14.9.

* Correct some unitialized problems that pop up with certain ports
* Replace USE_DOS2UNIX with USES=dos2unix
* Do not check for direct use of commands in OPTIONS descriptions

Reported by:	Hardy Schumacher <hardy.schumacher@gmx.de>

 

05:29 marcus 

Update to 2.14.8.

* Do not warn on direct use of "false" if it is found in DESKTOP_ENTRIES. [1]
* Make sure the DESKTOP_ENTRIES true/false parameter does not include quotes.
* Remove an unreferenced variable dereference. [2]
* Add some additional variables to avoid explicit command use checks. [3]

PR:		185086 [1]
		185225 [2]
		185110 [3]

 

14:49 marcus 

Update to 2.14.7.

* Fix a typo resulting in an undefined function [1]
* Add X11 to the list of common OPTIONS [2]
* Remove an errant space [2]
* Do not warn on empty .svn directories. [2]

PR:		183330 [1]
Submitted by:	Kubilay Kocak <koobs.freebsd@gmail.com> [1]
Reported by:	eadler [2]

 

00:53 marcus 

Update to 2.14.6.

* Mute svn warnings if .svn doesn't exist. [1]
* Add check for ${OPTION}_* (option helpers) [1]
* Suggest LICENSE as warning [1]
* Fix get_makevar and use instead of direct calls to make(1) [1]
* Remove MAKE_JOBS_SAFE [1]
* Remove vestiages of old OPTIONS [1]
* Suggest converting to stage if NO_STAGE is defined. [1]
* Check for use of MANCOMPRESSED when STAGE support is enabled.
* Do not warn about man pages in the plist if the port is built with stage
  support.
* Do not flag .svn in the search for dotfiles.

Submitted by:	ak [1]

 

16:44 marcus 

Properly bump the version of portlint.

 

10:14 bdrewery 

- Update to 2.14.5

Changes:
  * Fix recommending to replace USES=gmake with ${GMAKE} [1]
  * Ignore USES when recommending to replace direct commands [1]
  * Fix COMMENT check so it checks for length even if it
    is not formatted properly [2]

PR:		ports/181359 [1]
Submitted by:	Alan Hicks <ahicks@p-o.co.uk> [1]
Reported by:	many
PR:		ports/181730 [2]
Submitted by:	gerald
Approved by:	maintainer timeout (> 1 month)

 

16:15 marcus 

Update to 2.14.4.

* Fix some minor issues in the portlint man page [1]
* Warn about using ${SITE_PERL} in depends [2]
* Add a check to see that OPTIONS_{RADIO/SINGLE/MULTI/GROUP}_SOMETHING exists
  if OPTIONS_{RADIO/SINGLE/MULTI/GROUP} is defined [3]
* Join all lines with a backslash before doing anything to fix parsing
  of comments [4]
* Stop looping through MASTER_SITES as soon as a comment is seen [4]
* Detect plain ftp/http sites and print a warning if none have been
  found [4]
* Opt for USES=pkgconfig instead of USE_PKGCONFIG [5]
* Add check for ${TRUE}/${FALSE} in DESKTOP_ENTRIES [5]
* Remove CDRTOOLS check [5]
* Remove USE_REINPLACE check [5]
* Remove USE_GETOPT_LONG check [5]
* Modify USE_GETTEXT -> USES gettext [5]
* Remove WITHOUT_NLS check [5]
* Relax headers check for $FreeBSD$ + $MCom$ case [5]
* Fix URL suggestion for CPAN so that it ends with a '/' [6]

PR:		178952 [2]
		179262 [3]
		179338 [5]
		180260 [6]
Submitted by:	eadler [1]
		"Anton Yuzhaninov" <ayuzhaninov@openstat.ru> [2]
		"Ilya A. Arkhipov" <rum1cro@yandex.ru> [3]
		tijl [4]
		Alex Kozlov <spam@rm-rf.kiev.ua> [5]
		Anes Mukhametov <anes@anes.su> [6]

 

23:18 marcus 

Update to 2.14.3.

* Add support for the PL_SVN_IGNORE environment variable. [1]
* Promote PORT_OPTIONS:MDOCS to NOPORTDOCS. [2]
* Add a check for pure FTP or HTTP mirrors in MASTER_SITES. [3]

PR:		177043 [1]
		177048 [2]
		177805 [3]
Submitted by:	tota [1]
		pclin [2]
		Tijl Coosemans <tijl@coosemans.org> [3]

 

06:16 marcus 

Bump to version 2.14.2.

* Add a missing comma to fix a really weird error on checking certain ports. [1]
* Remove unused PL_CVS_IGNORE [2]
* Add check for a single space after WWW: in pkg-descr [2]
* Clarify 'Created by:' warning message [2]
* Add new OPTIONS_* knobs to @options_early [2]
* Add supported for USE_GCC=yes. [3]

PR:		175186 [1]
		175317 [2]
		174758 [3]
Submitted by:	gerald [1] [3]
		ak [2]

 

00:40 tota 

- Switch the VCS tool for the ports tree from CVS to SVN
- Pet portlint in porttools
  * Use PORT_OPTIONS:MDOCS instead of NOPORTDOCS
  * Replace tab with space WWW: line in pkg-descr

PR:		ports/174860
Submitted by:	tota (myself)
Approved by:	maintainers (timeout > 2 months)

 

23:19 marcus 

Update to 2.14.1.

Remove the check for empty header comments.  I think this is too
restrictive.  There are a number of ports that do this for various reasons.

 

23:28 marcus 

Update to 2.14.0.

* Add a test for empty(${VARIABLE}) (instead of empty(VARIABLE)) [1]
* Add a test for NOPORTDOCS and point to PORT_OPTIONS:MDOCS [1]
* Add GitHub to the MASTERSITES whitelist (as this is a cloud service). [2]
* Add a warning about .DS_Store files appearing in the port directory. [3]
* Add a check for the new style Makefile headers. [4]
* Remove the CVS checks now that the ports tree has been converted to SVN.
  Note: we need SVN checks. [5]
* Add support for optionsNG [6]
* Deprecate USE_GNOME=pkgconfig [6]
* Add missing period at the end of some sentences [6]

PR:		172621 [1]
		172834 [2]
		173649 [3]
		174688 [4]
		174693 [5]
		174697 [6]
Submitted by:	Alan Hicks <ahicks@p-o.co.uk> [1]
		"Steven Kreuzer" <skreuzer@exit2shell.com> [2]
		ehaupt [3]
		crees [4]
		tota [5]
		Alex Kozlov <spam@rm-rf.kiev.ua> [6]

 

22:56 marcus 

Update to 2.13.12.

Replace some of the CVS verbiage with SVN.

Submitted by:	beat

01:36 marcus 

Update to 2.13.11.

* Fix typo. [1]
* Add support for DOCS/NLS using optionsNG. [2]
* Remove references to PATCHDIR. [3]

PR:             169110 [1]
                168364 [3]
Submitted by:   Hung-te Liang <lhd@cs.nctu.edu.tw> [1]
                ak [2]
                crees [3]

16:36 marcus 

Correct a typo.

Reported by:    adamw

05:10 marcus 

Upgrade to 2.13.10.

* Add a check for over-specific shared lib dependencies [1]
* Add support for TEST_DEPENDS [2]
* Make the error for NO_CHECKSUM to be scarier [3]

PR:             165851 [1]
                166173 [2]
                167923 [3]
Submitted by:   matthew [1]
                sunpoet [2]
                "Bryan Drewery" <bryan@shatow.net> [3]

18:46 marcus 

Update to 2.13.9.

* Include LICENSE_FILE and LICENSE_GROUPS in the regexp check. [1]
* Add LICENSE_DISTFILES. [1]
* Add a check for the reassignment of ECHO_MSG [2]
* Remove old X.Org checks [3]
* Add PROJECTHOST to the PORTNAME section [4]

Reminded by:    Jan Beich <jbeich@tormail.net> [1]
Requested by:   jgh [2]
Submitted by:   eadler [3]
                Alex Kozlov <spam@rm-rf.kiev.ua> [3]

01:26 marcus 

Update to 2.13.8.

* Complete the check for CPPFLAGS being set indirectly by checking MAKE_ENV
in addition to CONFIGURE_ENV. [1]

* Remove checks for port file headers as we are phasing them out in FreeBSD. [2]

* Check to make sure CPAN URLs end with a '/'. [3]

* Check for LDFLAGS being overridden just as we do for CPPFLAGS [4]

* Suggest += instead of = to override CPPFLAGS and LDFLAGS [4]

* Check for any overriding use of *FLAGS and suggest an alternative [4]

* Do not warn of direct command use for WX_COMPS. [5]

* Add support for named license macros when checking for order. [6]

* Make the check for the direct Apache dependency more strict. [7]

PR:             160448 [1]
                159272 [3]
                160959 [4]
                161971 [5]
                163617 [6]
                163523 [7]
Submitted by:   gjb [2]
                amdmi3 [4]

23:35 marcus 

Update to 2.13.7.

* Change a comment to relfect the check number [1]
* Add a check to see if a file that ends in ".core" was checked into CVS [2]
* Do a proper check for 2 or 3 tuple depends [3]
* Correct the detection of the WWW string in pkg-descr [4]

PR:             159057 [3]
                159522 [4]
Reported by:    Ilya A. Arkhipov <micro@heavennet.ru> [1]
Requested by:   netchild [2]
Sbumitted by:   sunpoet [4]

05:03 linimon 

Now that the Java 1.3 and Java 1.4 ports are deprecated and will expire soon,
remove support for them from bsd.java.mk.  As Jikes is not available in Java 1.5
or higher, remove it from bsd.java.mk too (suggested by hq@) and from the ports
which used it (only occurences were USE_JIKES=no).  Support for the Blackdown VM
is also removed, as it is not available in Java 1.5 and higher.

Also remove the mapping from Java 1.1-1.4 to Java 1.5+ in bsd.java.mk to detect
old, broken ports; therefore bump the minimal value of JAVA_VERSION to 1.5.
While here, replace static values of JAVA_VERSION in files/*.in by
%%JAVA_VERSION%% .

PR:             ports/158969
Submitted by:   rene
Tested on:      pointyhat-west -exp

21:31 marcus 

Update to 2.13.6.

* Be more careful is searching for WITH(OUT)_FOO OPTIONS checks so to avoid
false positive warnings. [1]
* Support := in the *_DEPENDS section. [2]
* Make sure we expand DISTNAME to catch potential superfluous uses of it. [3]
* Fix the error seen when one passes CPPFLAGS to CONFIGURE_ENV. [4]
* Add the license block check after the MAINTAINER block [5]
* Add a check for use of ${FIND}...${XARGS} ${RM}. [6]

PR:             156164 [1]
                155753 [2]
                154320 [3]
                158305 [5]
                157566 [6]
Submitted by:   sahil [1]
                "Ilya A. Arkhipov" <micro@heavennet.ru> [5]
Reported by:    pav [4]

01:00 marcus 

Update to 2.13.5.

* Check for the direct assignment of BUILD_DEPENDS to RUN_DEPENDS [1]
* Add a deprecated pointer for USE_XPM. [2]
* CPPFLAGS is now passed to CONFIGURE_ENV automatically. [3]

PR:             154896 [1]
                155550 [3]
Submitted by:   eadler [2]