ports-mgmt/portmaster: Avoid make -D flag

Both FreeBSD make and GNU make pass command line flags to sub-makes
through the MAKEFLAGS environment variable, but the flags aren't
compatible and GNU make 4.4 exits with an error when it encounters
flags it doesn't understand, e.g. -D.

Avoid using the flag for now until ports run gmake with SETENVI.

PR:		272216, 277492

MAN?PREFIX: eleminate its usage  and move man to share/man

ports-mgmt/portmaster: fix accidentially left in debug tracing

Remove "set -x" thathad been added to verify the previous fix.

ports-mgmt/portmaster: upgrade to version 3.28

This version supports TRYBROKEN passed in the process environment or
in a Makefile (or, e.g., in /etc/make.conf as a per port setting) and
treats it as equivalent to the --try-broken option.

PR:		275307

ports-mgmt/portmaster: Fix issue when updating from packages only

When using the -P or --package option to portmaster and all updates
can be performed without building at least one port, the $DI_FILES
variable may be unset.

In this situation portmaster stops after the installation of each
port. Entering a CR makes it continue.

This was reported by Vlad Markov with a suggested fix that has been
applied (with a white-space change).

PR:		273660
Reported by:	dvoich@aim.com (Vlad Markov)

ports-mgmt/portmaster: Make --clean-packages consider port flavors

This commit fixes GutHub issue #55 which affected the cleaning of
stale package files for  flavored ports.

Some ports have package names that are prefixed by e.g. an interpreter
version (or where the interpreter version follows the name), but do
not support flavors. Such packages are not dealt with by this update,
since there is no generic way to do so.

The fix does not apply to --clean-packages used with --index or
--index-only, since the	INDEX file does	not record port flavors.

ports-mgmt/portmaster: update to version 3.25

The previous update attempted to fix the deletion of stale distfiles
that have names with blank characters or shell glob characters.
It introduced a regression that affected the deletion of some files,
and it has been found to be incomplete.

Testing revealed that the stale distfile check could start before the
full list of relevant distfiles had been read in the background. This
could have lead to distfiles being removed that are still relevant for
other ports.

This update should contain fixes for all these issues.

Reported by:	Dan Allen <danallen46@gmail.com>

ports-mgmt/portmaster: fix 2 issues in --clean-distfiles

There was no need to deal with distfile names that contained embedded
blanks or characters with special meaning in file glob patterns, but
this has changed with file names derived from repository URIs or other
naming systems.

This update fixes 2 issues reported by Daniel Rossberg on GitHub:
- issue #75: --clean-distfiles cannot handle paths with spaces
- issue #76: --clean-distfiles with paths containing square brackets

Reported by:	Daniel Rossberg

ports-mgmt/portmaster: fix GitHub issue #38 (clean stale ports)

Version 1.19.1 of the pkg command removed a previously working
feature (combinations of "-e" with other options of "pkg query"),
causing the use of "pkg query -ae" to fail in "portmaster -Ds".

Reported by:	JulienDlq at GitHub

Remove WWW entries moved into port Makefiles

Commit b7f05445c00f has added WWW entries to port Makefiles based on
WWW: lines in pkg-descr files.

This commit removes the WWW: lines of moved-over URLs from these
pkg-descr files.

Approved by:		portmgr (tcberner)

Add WWW entries to port Makefiles

It has been common practice to have one or more URLs at the end of the
ports' pkg-descr files, one per line and prefixed with "WWW:". These
URLs should point at a project website or other relevant resources.

Access to these URLs required processing of the pkg-descr files, and
they have often become stale over time. If more than one such URL was
present in a pkg-descr file, only the first one was tarnsfered into
the port INDEX, but for many ports only the last line did contain the
port specific URL to further information.

There have been several proposals to make a project URL available as
a macro in the ports' Makefiles, over time.

ports-mgmt/portmaster: Fix an issue with flavored dependencies

Flavored run dependencies of an unflavored port could cause the
installation of the port to fail after it had been successfully
built.

This issue should be fixed by this update, which executes a previously
commented out function call to update the FLAVOR in the environment of
the make command used to install the port.

I do not remember why I had commented out that function call, a long
time ago - but I do not expect it to cause any regressions, since the
same FLAVOR is now set enforced during "make install" as had been used
to build the port.

PR:		265084
Reported by:	alt2600 at icloud.com

ports-mgmt/portmaster: update to version 3.21

Merge pull request #47 by Oleg Pudeyev:

Do not delete distfiles by default.

Merge pull request #59 by Babak Farrokhi:

Add -U alias for --update-if-newer

Merge pull request #71 by Kubilay Kocak:

Update IRC Channel Link (libera chat)

Fix issue #68 opened by Helmut1972

Make --no-confirm affect display of package messages

Any package messages of updated ports are displayed at the end of the
port updates, normally through a pager requiring conformation of each
page displayed.

This update makes --cno-confirm remove the need to confirm the display
of package messages.

ports-mgmt/portmaster: update to version 3.20

Add "I" to the getopt pattern to enable use of the already present
functionality.

Reported as https://github.com/freebsd/portmaster/issues/67.

Reported by:	Michael Schmiedgen via Github

ports-mgmt/portmaster: Make compatible with pkg-1.17.0

The new pkg command uses the ".pkg" extension for all files,
independently of the archive format chosen.

This update adapts portmaster to that convention and removes all
support for other extensions. (This might be considered premature,
but supporting multiple extensions would require logic changes and
I expect all users to quickly convert their local repositories to
comply with the new naming.)

The package format can still be selected as before.

If you have package files in /usr/packages/* that you want to be
visible to portmaster, then rename them, e.g. from "*.txz" to "*.pkg".
(The "mmv" command (misc/mmv port) performs this renaming with the
following command in /usr/packages/All:

	  mmv "*.txz" =1.pkg

ports-mgmt/portmaster: Fix syntax error introduced in last commit

A last minute change from "case" to "if" caused a syntax error since
I had forgotten to remove a now stray ";;" ...

Reported by:	bsd@dino.sk

ports-mgmt/portmaster: make --no-confirm omit the PAGER

As requested in Github issue #64 the pager should not be used if
portmaster is executed within a script. In order to not introduce
a new option, --no-conform will now imply that output will not be
halted waiting for user input to proceed when displaying the package
messages.

Reported by:	Michael Osipov (michael-o on Github)

ports-mgmt/portmaster: make CONFLICTS change catch more cases

As a performance optimization the list of conflicting ports was only
fetched if the port's Makefile contained a CONFLICTS* entry.

There are ports that only set some option defaults and then include
the Makefile of some generic port. No conflicts change was performed
in that case.

Fix the behavior by always checking all CONFLICTS definitions that
apply, even for ports that do not have such a line in their direct
Makefile. This removes a performance optimization, but is required to
correctly build and install ports that have dependencies that can be
satisfied by an alternative (e.g. ImageMagick6 is installed and the
port being built has a dependency on ImageMagick6-nox11, which does
not contain a CONFLICTS definition it its Makefile).

One more small cleanup, forgotten yesterday.
Reported by:	lwhsu

Remove # $FreeBSD$ from Makefiles.

Enable updating of locked ports

It was already possible to override the locked status of a package, but
the deinstallation of the old version failed and made the update fail.

After this change the package is temporarily unlocked during the upgrade.

Reported by:	rkoberman@gmail.com (Kevin Oberman)

Correctly report package locks that prevent an upgrade

Reported by:	rkoberman@gmail.com (Kevin Oberman)

Fix regression that has been introduced in r524227

Revert part of the patch that introduced the --try-broken option, it
caused problems in combination with -m.

Submitted by:	Rudolf Cejka (cejkar at fit.vutbr.cz)
Approved by:	antoine (implicit)

Save the file with instructions how to restart portmaster after a failure
to non-world-writable directory.

Save this file in the user's home directory instead of in /tmp to prevent
a possible sym-link attack against the user.

PR:		233378
Submitted by:	Robert Schulze
Approved by:	antoine (implicit)

Make the pre-build "make clean" use SU_CMD if appropriate.

Additionally document that PORTSDIR can be passed in the environment.

PR:		228439
Approved by:	antoine (implicit)

Make portmaster fall back to other locations if invoked in, e.g., /sys/conf.

PR:		237061
Approved by:	antoine (implicit)

Add option --try-broken

This option makes portmaster try to build ports that are marked as
BROKEN or IGNORE.

Passing this option implies the "-m -DTRYBROKEN" option.

The change is inspired by (but not based on) the patch suggested by
Walter Schwarzenfeld.

PR:		235793
Reported by:	Yasuhiro KIMURA
Approved by:	antoine (implicit)

Do not re-build ports unneccessarily

The previous patch has not completely fixed the issue, according to some
reports (it did on my system and for my set of installed ports).

This version contains a further change suggested by Walter Schwarzenfeld.

Submitted by:	Walter Schwarzenfeld
Approved by:	antoine (implicit)

Attempt to fix a problem that leads to repeated building of some ports

There are build dependencies, that were compiled again and again even
though the latest version was already installed.

This fix is based on a patch suggested by Tatsuki Makino, with a small
change to better comply with the existing style used in this program.

An extended patch has been provided by Walter Schwarzenfeld, but in my
tests it did not make a difference. If further testing shows a need for
the additional commands suggested by Walter, I'll add them (or a variation)
in a follow-up commit.

PR:		241227, 242086, 243432
Submitted by:	Tatsuki Makino
Approved by:	antoine (implicit)

- Update WWW

Approved by:	portmgr blanket

Remove reference to obsolete INDEX files for FreeBSD7-9 and report build
time for successful builds, as suggested by Walter Schwarzenfeld.

Approved by:	antoine (implicit)

Fix upgrades of moved ports.

Make portmaster use "pkg-static" instead of "pkg". This prevents problems
with inconsistent shared library references that may occur when openssl in
base is upgraded to version 1.1.1 with 1.0.x installed from a port.

Approved by:	antoine (implicit)

Fix quoting of optional FLAVOR argument in some make invocations, which
were uncovered by fixed quoting in the pm_make* functions.

Reported by:	tatsuki_makino at hotmail.com (Tatsuki Makino)
Approved by:	antoine (implicit)

Make some sub-routines use "$@" instead of $* to correctly pass arguments
with blanks to invoked commands. This fixes clean-up of shared library
backups, where only one file was removed due to a parameter list being
terminated by the first blank within one parameter.

Reported by:	tatsuki_makino at hotmail.com (Tatsuki Makino)
Approved by:	antoine (implicit)

Fix building of specific qt4 ports after qt5 has become the default flavor.

The ports infrastructure does not consistently deal with FLAVOR environment
variables set to an empty string. This caused the dependency check in
portmaster to fail for multimedia/phonon.

With FLAVOR="" in the environment, "make -V FLAVOR" returns "qt4", but the
dependency list produced by "make build-depends-list" assumes "qt5", even
though no user preferences for qt5 have been set.

Reported by:	Walter Schwarzenfeld <w.schwarzenfeld at utanet.at>
Approved by:	antoine (implicit)

Document --package-format=<fmt> option. I had implemented that option some
time ago, but decided to not expose it in the man-page, since portmaster
will not necessarily install the latest version if installing from local
Jpackages is requested and there exist multiple package files that only
differ in archive format. But since a non-standard package format allows
to choose between an optimization for speed or for disk space, this is a
useful option that shall be described in the man page.

Approved by:	antoine (implicit)

Fix recursive building of ports (-r option). The match_flavor function was
called before being defined.

The issue and cause has been reported by Herbert J. Skuhra. (Thanks!)

PR:		227882
Reported by:	Herbert J. Skuhra <herbert@gojira.at>
Approved by:	antoine (implicit)

Fix previous attempt to cache some parameters in environment variables.
The quoting was messed up by use of "eval" and the cached values did not
match what the ports system expected.

Submitted by:	tijl
Reported by:	tijl
Approved by:	antoine (implicit)

Cache some build parameters in the environment that were recalculated over
and over for individual ports. This significantly reduces the time to scan
for updates if many ports with USES=compiler are present.

Based on a suggestion and code fragment provided by Tijl Coosemans.

While here commit a few changes present in my local version that should not
cause any functional change.

Submitted by:	tijl
Approved by:	antoine (implicit)

Make the -n option work better for non-root users.

Approved by:	antoine (implicit)

Fix problem with debug output from export-flavor garbling the gen_dep_list
result. The solution is different from the patch suggested in the PR, since
the debug message was only meant to be used during early development of
flavors support. In my local development version of portmaster, all output
from pm_v is redirected to STDERR, but that change seemed to disruptive to
apply to the "published" version.

Unrelated changes: make pm_isdir_pd more robust and fix variable name of
BACKUP_FORMAT option in sample file.

PR:             225843
Reported by:    John Hein
Approved by:    antoine (implicit)

Fix typo (missing dash in find argument list).

Reported by:	Martin Birgmeier
Approved by:	antoine (implicit)

Fix build for ports that have a LICENSE if DISABLE_LICENSES is defined.

The behavior of the ask-license target should be to silently succeed instead
of returning an error, IMHO, but I can test for this condition and skip the
ask-license phase in portmaster, too.

While here, I'm slightly simplifying the fix implemented in r460294.

PR:		225699
Submitted by:	dewayne@heuristicsystems.com.au
Approved by:	antoine (implicit)

Fix a problem that could result in de-installation of a port that shares
a prefix of the package name with a new port to be installed. The cause of
this bug was that a pattern was applied without anchor at the end.

Testing revealed that "pkg info -x" ignores an anchor at the end of the
pattern (which might be a bug in pkg), therefore the output of the query
is now additionally filtered with egrep to obtain the desired result.

PR:		225496
Reported by:	Martin Birgmeier
Approved by:	antoine (implicit)

Print different action list text for packages that are not going to be
installed (e.g. for make targets "extract", "patch" or "stage" in build
dependencies).
Limit depth of search for empty directories to reduce the scan time if
there are non-empty work directory sub-trees.
Add package format parameters to rc.sample file.

Approved by:	antoine (implicit)

Fix previous commit (r459202) which introduced dependencies that specify
an optional make target. The patch file contained 2 typoes ("{$" instead
of "${").

Add clean phase for dependencies that are not installed but required when
the dependent port is built (e.g. because of references to source files of
that port, or because of static linking against libraries in the stage area
of such a dependency).

Approved by:	antoine (implicit)

Add support for dependencies specified with a non-standard make target.

This is required for e.g. net/openntpd, which links against static libressl
libraries in that port's staging area.

In case of a port that is not fully installed, there is no clean phase for
the dependency's port directory. The dependency tracking does not maintain
the necessary state to perform this cleanup when the depending port is
finished.

PR:		213887
Reported by:	blackmore@pichove.org (Simeon Simeonov)
Approved by:	antoine (implicit)

Test privileges early and complain if privileges are insufficient.

The patch is based on ideas presented in the PR, but implemented
differently. Re-open the PR, if this solution has unexpected side-effects
or proves insufficient.

PR:		174729
Reported by:	ngie
Approved by:	antoine (implicit)

Update to version 3.19, which is identical to 3.18_12 with all port patches
applied.

Approved by:	antoine (implicit)

Fix display of license acceptance menu if build logs are hidden with -H.

PR:		206738
Reported by:	torsten.eichstaedt@web.de
Approved by:	antoine (implicit)

Fix problem with ports being converted from a non-flavored to a flavored
version, if PYTHON3_DEFAULT is set to anything but 3.6. Since the MOVED
file sets the new origin and flavor such that a Python-3.6 compatible
package is built, the original Python version for this port is ignored.

Reported by:	lev
Approved by:	antoine (implicit)

Small fix for directory accesses in  find_dl_distfiles, which did not take
the optional flavor part of the passed argument into consideration.

Approved by:    antoine (implicit)

Generally path FLAVOR via the environment, not the "make" command line, to
circumvent a check for an empty FLAVOR parameter being passed to certain
ports.

Reported by:	Walter Schwarzenfeld <w.schwarzenfeld@utanet.at>
Approved by:	antoine (implicit)

Fix version number ... I have to remember to manually fix-up this string
before each commit and I'll whether I can create a pre-commit script that
checks this for me.

Reported by:    Walter Schwarzenfeld <w.schwarzenfeld@utanet.at>
Approved by:    antoine (implicit)

Remove spurious trace output left over from debugging.

Reported by:	Walter Schwarzenfeld <w.schwarzenfeld@utanet.at>
Approved by:	antoine (implicit)

Fix pm_islocked function, which could lead to a spurious error message if
used for a port that is not found in the package database.

Reported by:	Tatsuki Makino <tatsuki_makino@hotmail.com>
Approved by:	antoine (implicit)

Add undocumented option --package-format=<fmt>. I'm not sure, that this is
a useful feature, and the txz-format is currently hard-wired in a lot of
places in portmaster, but there has been a request to allow creation of
packages in a non-standard archive format.

Approved by:	antoine (implicit)

Fix package name: the pattern that gets replaced with the actual version
string at build time had been replaced by a version string from a test
version.

Reported by:	herbert@gojira.at (Herbert J. Skuhra)
Approved by:	antoine (implicit)

Increment PORTREVISION, forgotten in previous commit.

Approved by:	antoine (implicit)

Add --backup-format=<fmt> option to allow faster creation of package
backups with no or less compression than the default of "txz" provides.

Support for different archive format for packages in general could be
provided, but not added at this time, since I'm not convinced that this
would be a useful feature (please let me know, if you disagree ...).

Approved by:    antoine (implicit)

----------------------------------------------------------------------------
Fix spelling of OPTIONS_FILE.

Approved by:	antoine (implicit)

Bump PORT_REVISION, forgotten in previous commit.

Approved by:	antoine (implicit)

Fix regression in kill_bad_children: add  missing $ before progcmd ...

Approved by:	antoine (implicit)

Fix installation from local or remote package including the --packages-build
case. Partially based on patches supplied by Alexander Leidinger.

Reported by:	phk, netchild
Approved by:	antoine (implicit)

Add PKG_DEPENDS, EXTRACT_DEPENDS, PATCH_DEPENDS and FETCH_DEPENDS to the
make_dep_list function when called with parameter build-depends-list or
all-depends-list.

Reported by:	phk
Approved by:	antoine (implicit)

Remove useless MASTER_SITES entries

Update to version 3.18, which already contains all patches required for the
previous version.

Approved by:    antoine (implicit)

Patch package version into the port and install man-page with patched path
for non-standard LOCALBASE. The port used to patch the man-page, but then
installed the unpatched version.

Approved by:	antoine (implicit)

Fix execution of portmaster in a port directory without any port or package
parameters.

Reported by:	lenzi.sergio@gmail.com
Approved by:	antoine (implicit)

Add flavor support to routines that convert port origins to shell variable
names. The character '@' is illegal in such names and must be converted to
a legal character same as [-+./].
remobve post processing of "pkg check -dn" output, which was a left-over
from pre-PKG_NG times.

Reported by:	koobs (invalid @), tatsuki_makino@hotmail.com (pkg check)
Approved by:	antoine (implicit)

Add back local package installation support, which was disabled for PKG_NG.
The patch attached to PR 181450 did no longer apply cleanly, therefore I
edited portmaster to follow the intent and in part the letter of the patch.
This patch should not affect port building in any way. I have no use for
the feature (and I'm still considering to completely remove support for the
installation of packages in portmaster), and therefore did not test that
it performs as expected by the submitter.

PR:		181450
Submitted by:	rozhuk.im@gmail.com
Approved by:	antoine (implicit)

Add all-depends-list case required for portmaster -t.

Approved by:	antoine (implicit)

Remove stray double-quote character left from an incomplete edit of that
line (result of directly editing the patch for a trivial change ...)

Approved by:	antoine (implicit)

Add LIB_DEPENDS to dependencies relevant for installation of a package.

Approved by:	antoine (implicit)

Add support for ports that have been upgraded with a change of both
origin and package name (sans version), as was the case for e.g. the
lang/cython3 port, which was moved to lang/cython@py36 with a package
name change from cython3-$version to py36-cython-$version.

Approved by:	antoine (implicit)

Fix query for package to be replaced: This could lead to a wrong package
being identified and replaced when the new package is installed.

Add library dependencies to the list of build dependencies to consider.

Reported by:	Robert Huff
Approved by:	antoine (implicit)

USe correct origin in multiport case (several packages specified on the
command line, which is different from passing "-a").

Reported by:	Robert Huff
Approved by:	antoine (implicit)

Add another check for a flavor passed attached to the port directory.
Without this fix, the default flavor was built in specific situations.

Approved by:    antoine (implicit)

Fix patch: The patch had been created from the version used to debug the
previous version's problem and I forgot to
Fix shbang: Remove the -x that had been added for debugging.

Reported by:    (Walter Schwarzenfeld) w.schwarzenfeld@utanet.at
Approved by:	antoine (implicit)

Fix regression:	The find_new_port function is now invoked from an earlier
place than before flavor	support	was added. Move	its definition to an
appropriate place.

Silence	one pkq	query command that may be called for a non-existing
package	name.

Approved by:    antoine	(implicit)
Reported by:	Milan Obuch, Paul Beard

Add flavor support to portmaster. This version has been lightly tested and
supports upgrades from non-flavored port versions based on the information
in the MOVED file.

For initial installations of flavored ports, the flavor must be specified
as part of the port origin, e.g. "devel/py-py@py36" for the Python-3.6
version of that port. Dependent ports will automatically be installed with
the correct flavor passed via the dependency mechanism.

It is planned to add a --flavor option to ease flavor selection for ports
that are initially installed with portmaster.

Approved by:	antoine (implicit)

ports-mgmt/portmaster: Pass maintainer-ship to se@

Stefan will continue to maintain portmaster.

- Update to 3.17.10
  Fixes --clean-distfiles trying to read in base/ [1]
  Saves port failures to TMPDIR/portmasterfail.txt [2]

PR:		212376 [1]
Reported by:	wblock [1]
Tested by:	wblock [1]
Submitted by:	Torsten Zuhlsdorff <t.zuehlsdorff@anymotion.de> [2]

Fix "portmaster --check-port-dbdir" broken by the previous commit. While
the variable $unique_list was created with identical content, its value was
only set within the implicit sub-shell (2nd command in a pipe) and lost on
exit from the while loop.

Reported by:	Jakub Laach

Bump PORTREVISION to have portmaster be re-installed with the fix commited
as SVN rev. 418619.

Reported by:	Herbert J. Skuhra (herbert at mailbox.org)

A read lock on the sqlite file used by pkg can lock out writers for
indefinite amounts of time, preventing installation or upgrade of
ports with portmaster.  Instead of using a pipe to read pkg query
input to be consumed by a shell loop, buffer the query result in a
shell variable and let the loop iterate over the contents of this
variable.

While here, use a call to all_pkgs_by_origin() instead of an inline
versin of equivalent code.

Reviewed by:	tz
Approved by:	tz

Change MAINTAINER from ports@toco-domains.de to tz@FreeBSD.org

Approved by:	junovitch (mentor)

Enable BASH and ZSH options (install their completions) by default.
Also, add NO_ARCH. PORTREVISION bump for pkg change.

PR:		207075
Submitted by:	maintainer

Remove unneeded dependencies on bash and zsh.

With hat:	portmgr
Sponsored by:	Absolight

ports-mgmt/portmaster: Assign maintainership to Torsten Z.

Torsten has accepted the challenge of whipping portmaster back into
shape.  In addition to changing MAINTAINER, he's starting with fixing
both non-default options by providing their necessary RUN_DEPENDS.

PR:	207075

Update to 3.17.9

Changes:
 - Update manpage for rebuilding ports with pkg.

PR:		191166
Submitted by:	wblock

Under strong objection and mandate by portmgr, remove DEPRECATION

I've been ordered by portmgr to remove DEPRECATION designation because
others have indicated they believe people should not be so directly
informed of its poor state.  Despite the fact that there was no expiration
date set and that functionality was not affected in any way (leaving now
informed people free to use this unmaintained port), it was considered a
disruptive change.

Let the record show that I strongly object to this decision and that I
firmly believe that portmaster is a port that *must* have a competent
maintainer that can *develop* it.  It should *not* be allowed to be
unmaintained and still maintain a presence in FreeBSD documentation.

ports-mgmt/portmaster: DEPRECATE without expiration date

The portmaster script hasn't had an official maintainer in 9 months and 2
years before that it was only patched in reaction to changes in the ports
framework.  There are many unclaimed PRs in the bugzilla database, many
known bugs, and several areas where portmaster no longer aligns with how
ports work today.  The problem isn't simply getting a maintainer; that
person has to be a ports framework expert and it appears that the people
with these qualifications don't want anything to do with this port.

Moreover, there are better options available.  All FreeBSD platforms
support ports-mgmt/poudriere (although some many struggle under the load)
and the most common amd64 and i386 platform users have the additional
option of ports-mgmt/synth which is user-friendly, lightweight, and aimed
at users of portmaster, portupgrade, and even poudriere.

Unless something drastic regarding portmaster occurs, it's nearing its
natural EOL, so it's users should evaluate alternatives and try to
migrate off of it.

Remove obsolete pkg-message

Update to 3.17.8.

Changes:
  - Fixes handling of PORTSDIR when /usr/ports is a symlink after head
    r272363 and stable/10 r287334.

Reported by:	dhw

Use OPTIONS helpers, add a newline to quiet a half dozen portlint
fatal messages, and make the installation commands more consistent.

I'm really not working on this. Please direct questions to ports@FreeBSD.org.

If someone else would like to maintain this please discuss with me and I will
get you access to the github account where the code lives.

For the new USE_GITHUB with only GH_TAGNAME set default DISTNAME to
include GH_PROJECT/GH_ACCOUNT/GH_TAGNAME. This prevents the distfile
having the same name despite changing one of these values and causing
a bad checksum.

Differential Revision:	https://reviews.freebsd.org/D2103
Reviewed by:	mat
With hat:	bdrewery

Convert my ports to the new USE_GITHUB