| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
For the Python 3.8 default switch.
|
|
|
|
Seen in mk/fetch/fetch.mk for FETCH_USE_IPV4_ONLY.
|
|
|
|
The width only applies to a single group, not to several or even all
groups.
|
|
|
|
The test framework over there makes it easier to run several independent
tests. The previous way of squeezing all tests into a single package
Makefile did not scale well and made it invonvenient to add new tests.
In addition, there was no need to create a full-fledged package just to
test this tiny piece of the pkgsrc infrastructure, since that can be
used in categories and the top-level as well.
|
|
The buildlink3 variable names are quite long. So long that using the
default column width of 24 characters, most of the variable values are
not aligned. In this case, it makes sense to shift them all to the right
a bit.
|
|
Now that haskell.mk distinguishes between plain and outdated PLIST files,
this is possible again. When haskell.mk knew only missing and outdated,
this was still ambiguous and therefore skipped.
|
|
|
|
As seen in devel/hs-hashable/PLIST r1.1, which listed only the
package-description but not the package-id.
|
|
The PLIST_SUBST and PLIST_PRINT_AWK definitions for Haskell library
packages are only useful if the package-description file exists. If
that file is absent though, these are skipped.
The test whether the file exists is made as late as possible since that
file does not yet exist at the point where the package Makefile is
parsed.
This also affects the show-all-haskell target, which only shows these
values after the install phase. This is not perfect but good enough for
practical cases.
|
|
These tests demonstrate the current behavior, to document further changes
to haskell.mk.
|
|
|
|
|
|
|
|
Fixes PR pkg/55364.
|
|
Discovered by John Klos in PR pkg/55364.
|
|
|
|
|
|
The general rule is that a SUBST_SED that contains _any_ identity
substitution may leave files unmodified, no matter if there are other
substitutions as well.
|
|
Discovered by Juraj in mail/policyd-weight.
https://mail-index.netbsd.org/pkgsrc-changes/2020/06/06/msg215480.html
|
|
|
|
Seen in cross/arm-none-eabi-gdb.
|
|
|
|
|
|
|
|
Before, relative paths had been stored as-is. This affected those
packages that defined PATCHDIR or FILESDIR as relative directory instead
of prefixing it with ${.CURDIR}.
Since there are already several other paths that are interpreted relative
to the package directory (CONFLICTS, DEPENDS), allow PATCHDIR and
FILESDIR to be specified as relative paths, too. This makes the package
Makefiles a bit shorter.
|
|
The previous implementation could not reliably detect outdated configure
options. This was apparent in devel/gettext-tools, where the option
--with-included-libcroco had become unknown between May 2019 and May
2020, but the check was not run.
The behavior is the same in the pkgsrc default configuration. Only if
GNU_CONFIGURE_STRICT=yes, the new check is activated and will make
packages fail that previously succeeded to build. Since that variable is
not widely known, there won't be much sudden breakage, if any.
|
|
|
|
Pointed out by wiz@.
This occured in math/libixion/Makefile.common until 2020-05-19, and still
occurs in math/xyconvert/Makefile. In all other packages, PKGDIR is
prefixed with ${.CURDIR} and is thus an absolute path.
It should not be necessary to always specify PATCHDIR as an absolute
path, and the code in mk/pkgformat/pkg/metadata.mk seems to be the only
place where relative paths are handled wrong.
|
|
The test infrastructure from test.subr already takes care of setting up a
temporary directory.
|
|
|
|
Like directories, they probably come from shell globs.
Seen in misc/byobu.
|
|
|
|
|
|
This makes the SUBST blocks stricter than before, to detect outdated or
unnecessary definitions.
Filename patterns that are not affected by any of the SUBST_SED
expressions make the build fail. It is still ok if only some of the
files from a pattern are affected and some aren't.
The latest bulk build shows that most of the build failures are fixed.
The packages that fail in that build are mostly due to other failures,
like missing C headers, wrong PLIST files, unresolved references at link
time. There may be a few packages that still fail because of this, but
these are near the leaves of the dependency tree.
https://mail-index.netbsd.org/pkgsrc-bulk/2020/05/14/msg018919.html
|
|
Seen in games/bastet. The 1 is not necessary though since it only
repeats the default behavior of sed.
|
|
The test can now be run with bash and ksh as well.
Bash exits if a function call returns failure, the NetBSD /bin/sh
doesn't. After reading POSIX, sections "2.9.1 Simple Commands" and "set
-e", I'm in favor of bash here.
Ksh exits in "set -eu" mode if "$@" is used but no arguments are given.
|
|
There had been unnecessary variations in the code.
|
|
|
|
Strings in single quotes are simpler to understand and also need fewer
pixels on the screen.
|
|
In a basic regular expression, a dollar-sign only means end-of-string if
it appears at the end of the pattern, or (at the choice of the
implementation) at the end of a \(...\) subexpression.
This affects the package converters/help2man that uses a regular
expression containing a dollar in a non-final position. This regular
expression had not been detected as an identity substitution even though
it is one.
|
|
|
|
This isolates the tests from the test infrastructure and allows the test
infrastructure to create arbitrary files for its own purpose without
affecting any of the tests.
|
|
|
|
|
|
This test is for the old compiler wrapper.
Using it with the cwrappers from 2014 makes several test cases fail.
|
|
|
|
|
