diff options
| author | rillig <rillig@pkgsrc.org> | 2020-06-11 19:38:40 +0000 |
|---|---|---|
| committer | rillig <rillig@pkgsrc.org> | 2020-06-11 19:38:40 +0000 |
| commit | 3e6bd419e004156bf52423d748820cc500af291a (patch) | |
| tree | b89721d390eb94adbf6bc2df333704999da71c31 /mk/subst.mk | |
| parent | 2cfa0b992fc1178d6f9d3202e0dedc4be52a6810 (diff) | |
| download | pkgsrc-3e6bd419e004156bf52423d748820cc500af291a.tar.gz | |
mk/subst.mk: document the unspecified application order
To avoid bmake warnings because of duplicate class names, the :O:u
modifier had been added in r1.66 on 2020-03-21. This had the side effect
that the subst classes are now applied in alphabetical order instead of
declaration order.
For this to actually matter, there must be a file that is affected by two
different subst classes and in which the substitutions depend on each
other or prevent each other. Chances for that are pretty low.
The order is intentionally documented as being unspecified, to allow for
future modifications, just in case that a bmake variable modifier is
invented that filters for duplicates without requiring the duplicates to
be adjacent to each other. In that situation, it would be nicer to
switch back to declaration order instead of alphabetical.
Diffstat (limited to 'mk/subst.mk')
| -rw-r--r-- | mk/subst.mk | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mk/subst.mk b/mk/subst.mk index 04930255c0d..d8ed0297bff 100644 --- a/mk/subst.mk +++ b/mk/subst.mk @@ -1,4 +1,4 @@ -# $NetBSD: subst.mk,v 1.97 2020/06/11 19:27:56 rillig Exp $ +# $NetBSD: subst.mk,v 1.98 2020/06/11 19:38:40 rillig Exp $ # # The subst framework replaces text in one or more files in the WRKSRC # directory. Packages can define several ``classes'' of replacements. @@ -40,6 +40,7 @@ # SUBST_CLASSES # A list of class names. When adding new classes to this list, be # sure to append them (+=) instead of overriding them (=). +# The order in which the classes are applied is unspecified. # # SUBST_STAGE.<class> # "stage" at which we do the text replacement. Should be one of |