summaryrefslogtreecommitdiff
path: root/pkgtools
diff options
context:
space:
mode:
Diffstat (limited to 'pkgtools')
-rw-r--r--pkgtools/pkglint/Makefile4
-rw-r--r--pkgtools/pkglint/files/buildlink3_test.go39
-rw-r--r--pkgtools/pkglint/files/distinfo.go27
-rw-r--r--pkgtools/pkglint/files/mkline.go9
-rw-r--r--pkgtools/pkglint/files/mkline_test.go23
-rw-r--r--pkgtools/pkglint/files/mklinechecker.go3
-rw-r--r--pkgtools/pkglint/files/mklines.go2
-rw-r--r--pkgtools/pkglint/files/mkparser.go2
-rw-r--r--pkgtools/pkglint/files/mkparser_test.go2
-rw-r--r--pkgtools/pkglint/files/patches.go8
-rw-r--r--pkgtools/pkglint/files/patches_test.go28
-rw-r--r--pkgtools/pkglint/files/plist.go31
-rw-r--r--pkgtools/pkglint/files/plist_test.go2
-rw-r--r--pkgtools/pkglint/files/vardefs.go18
14 files changed, 133 insertions, 65 deletions
diff --git a/pkgtools/pkglint/Makefile b/pkgtools/pkglint/Makefile
index a9589d003a6..7fb845dee02 100644
--- a/pkgtools/pkglint/Makefile
+++ b/pkgtools/pkglint/Makefile
@@ -1,6 +1,6 @@
-# $NetBSD: Makefile,v 1.522 2018/01/07 01:13:21 rillig Exp $
+# $NetBSD: Makefile,v 1.523 2018/01/07 17:08:15 rillig Exp $
-PKGNAME= pkglint-5.4.25
+PKGNAME= pkglint-5.4.26
DISTFILES= # none
CATEGORIES= pkgtools
diff --git a/pkgtools/pkglint/files/buildlink3_test.go b/pkgtools/pkglint/files/buildlink3_test.go
index 08cf400fd7a..3ec8bf1fb0f 100644
--- a/pkgtools/pkglint/files/buildlink3_test.go
+++ b/pkgtools/pkglint/files/buildlink3_test.go
@@ -295,3 +295,42 @@ func (s *Suite) Test_ChecklinesBuildlink3Mk_PKGBASE_with_unknown_variable(c *che
"WARN: buildlink3.mk:3: Please replace \"${LICENSE}\" with a simple string (also in other variables in this file).",
"WARN: buildlink3.mk:13: This line should contain the following text: BUILDLINK_TREE+=\t-${LICENSE}-wxWidgets")
}
+
+// Those .include lines that are not indented at all may stay as-is.
+// This special exception might have been for backwards-compatibility,
+// but ideally should be handled like everywhere else.
+// See MkLineChecker.checkInclude.
+func (s *Suite) Test_ChecklinesBuildlink3Mk_indentation(c *check.C) {
+ s.Init(c)
+ s.UseCommandLine("-Wall")
+ G.globalData.InitVartypes()
+ mklines := s.NewMkLines("buildlink3.mk",
+ mkrcsid,
+ "",
+ ".if ${VAAPI_AVAILABLE} == \"yes\"",
+ "",
+ "BUILDLINK_TREE+= libva",
+ "",
+ ". if !defined(LIBVA_BUILDLINK3_MK)",
+ "LIBVA_BUILDLINK3_MK:=",
+ "",
+ "BUILDLINK_API_DEPENDS.libva+= libva>=1.0.6",
+ "BUILDLINK_PKGSRCDIR.libva?= ../../multimedia/libva",
+ "",
+ ".include \"../../x11/libX11/buildlink3.mk\"",
+ "",
+ ". endif # LIBVA_BUILDLINK3_MK",
+ "",
+ "BUILDLINK_TREE+= -libva",
+ "",
+ ".endif # VAAPI_AVAILABLE")
+
+ ChecklinesBuildlink3Mk(mklines)
+
+ // No warning about the indentation of the .include lines.
+ s.CheckOutputLines(
+ "ERROR: buildlink3.mk:11: \"/multimedia/libva\" does not exist.",
+ "ERROR: buildlink3.mk:11: There is no package in \"multimedia/libva\".",
+ "ERROR: buildlink3.mk:13: \"/x11/libX11/buildlink3.mk\" does not exist.",
+ "WARN: buildlink3.mk:3: Expected a BUILDLINK_TREE line.")
+}
diff --git a/pkgtools/pkglint/files/distinfo.go b/pkgtools/pkglint/files/distinfo.go
index f9713bdd613..e2e3895af63 100644
--- a/pkgtools/pkglint/files/distinfo.go
+++ b/pkgtools/pkglint/files/distinfo.go
@@ -106,11 +106,10 @@ func (ck *distinfoLinesChecker) onFilenameChange(line Line, nextFname string) {
ck.algorithms = nil
}
-func (ck *distinfoLinesChecker) checkPatchSha1(line Line, patchFname, distinfoSha1Hex string) {
- patchBytes, err := ioutil.ReadFile(G.CurrentDir + "/" + patchFname)
+func computePatchSha1Hex(patchFilename string) (string, error) {
+ patchBytes, err := ioutil.ReadFile(patchFilename)
if err != nil {
- line.Errorf("%s does not exist.", patchFname)
- return
+ return "", err
}
h := sha1.New()
@@ -120,7 +119,15 @@ func (ck *distinfoLinesChecker) checkPatchSha1(line Line, patchFname, distinfoSh
h.Write(patchLine)
}
}
- fileSha1Hex := fmt.Sprintf("%x", h.Sum(nil))
+ return fmt.Sprintf("%x", h.Sum(nil)), nil
+}
+
+func (ck *distinfoLinesChecker) checkPatchSha1(line Line, patchFname, distinfoSha1Hex string) {
+ fileSha1Hex, err := computePatchSha1Hex(G.CurrentDir + "/" + patchFname)
+ if err != nil {
+ line.Errorf("%s does not exist.", patchFname)
+ return
+ }
if distinfoSha1Hex != fileSha1Hex {
if !line.AutofixReplace(distinfoSha1Hex, fileSha1Hex) {
line.Errorf("%s hash of %s differs (distinfo has %s, patch file has %s). Run \"%s makepatchsum\".", "SHA1", patchFname, distinfoSha1Hex, fileSha1Hex, confMake)
@@ -171,3 +178,13 @@ func (ck *distinfoLinesChecker) checkUncommittedPatch(line Line, patchName, sha1
ck.patches[patchName] = true
}
}
+
+func AutofixDistinfo(oldSha1, newSha1 string) {
+ distinfoFilename := G.CurrentDir + "/" + G.Pkg.DistinfoFile
+ if lines, err := readLines(distinfoFilename, false); err == nil {
+ for _, line := range lines {
+ line.AutofixReplace(oldSha1, newSha1)
+ }
+ SaveAutofixChanges(lines)
+ }
+}
diff --git a/pkgtools/pkglint/files/mkline.go b/pkgtools/pkglint/files/mkline.go
index c4b79aa14cf..f0772aced2a 100644
--- a/pkgtools/pkglint/files/mkline.go
+++ b/pkgtools/pkglint/files/mkline.go
@@ -102,12 +102,13 @@ func NewMkLine(line Line) (mkline *MkLineImpl) {
return
}
- if index := strings.IndexByte(text, '#'); index != -1 && strings.TrimSpace(text[:index]) == "" {
+ trimmedText := strings.TrimSpace(text)
+ if strings.HasPrefix(trimmedText, "#") {
mkline.data = mkLineComment{}
return
}
- if strings.TrimSpace(text) == "" {
+ if trimmedText == "" {
mkline.data = mkLineEmpty{}
return
}
@@ -312,7 +313,7 @@ func matchMkCond(text string) (m bool, indent, directive, args string) {
}
argsStart := i
- for i < n && text[i] != '#' {
+ for i < n && (text[i] != '#' || text[i-1] == '\\') {
i++
}
for i > argsStart && (text[i-1] == ' ' || text[i-1] == '\t') {
@@ -322,7 +323,7 @@ func matchMkCond(text string) (m bool, indent, directive, args string) {
m = true
indent = text[indentStart:indentEnd]
- args = text[argsStart:argsEnd]
+ args = strings.Replace(text[argsStart:argsEnd], "\\#", "#", -1)
return
}
diff --git a/pkgtools/pkglint/files/mkline_test.go b/pkgtools/pkglint/files/mkline_test.go
index 5327d4b170e..c2c07ebd4c9 100644
--- a/pkgtools/pkglint/files/mkline_test.go
+++ b/pkgtools/pkglint/files/mkline_test.go
@@ -133,7 +133,7 @@ func (s *Suite) Test_NewMkLine(c *check.C) {
"\tshell command # shell comment",
"# whole line comment",
"",
- ". if !empty(PKGNAME:M*-*) # cond comment",
+ ". if !empty(PKGNAME:M*-*) && ${RUBY_RAILS_SUPPORTED:[\\#]} == 1 # cond comment",
". include \"../../mk/bsd.prefs.mk\" # include comment",
". include <subdir.mk> # sysinclude comment",
"target1 target2: source1 source2",
@@ -159,7 +159,7 @@ func (s *Suite) Test_NewMkLine(c *check.C) {
c.Check(ln[4].IsCond(), equals, true)
c.Check(ln[4].Indent(), equals, " ")
c.Check(ln[4].Directive(), equals, "if")
- c.Check(ln[4].Args(), equals, "!empty(PKGNAME:M*-*)")
+ c.Check(ln[4].Args(), equals, "!empty(PKGNAME:M*-*) && ${RUBY_RAILS_SUPPORTED:[#]} == 1")
c.Check(ln[5].IsInclude(), equals, true)
c.Check(ln[5].Indent(), equals, " ")
@@ -480,6 +480,25 @@ func (s *Suite) Test_MkLine_variableNeedsQuoting__LDFLAGS_in_single_quotes(c *ch
"WARN: x11/mlterm/Makefile:2: Please move ${LDFLAGS:M*:Q} outside of any quoting characters.")
}
+// No quoting is necessary here.
+// PKG_OPTIONS are declared as "lkShell" although they are processed
+// using make's .for loop, which splits them at whitespace and usually
+// requires the variable to be declared as "lkSpace".
+// In this case it doesn't matter though since each option is an identifier,
+// and these do not pose any quoting problems.
+func (s *Suite) Test_MkLine_variableNeedsQuoting__package_options(c *check.C) {
+ s.Init(c)
+ s.UseCommandLine("-Wall")
+ G.globalData.InitVartypes()
+ G.Mk = s.NewMkLines("Makefile",
+ mkrcsid,
+ "PKG_SUGGESTED_OPTIONS+=\t${PKG_DEFAULT_OPTIONS:Mcdecimal} ${PKG_OPTIONS.py-trytond:Mcdecimal}")
+
+ MkLineChecker{G.Mk.mklines[1]}.Check()
+
+ s.CheckOutputEmpty()
+}
+
func (s *Suite) Test_MkLines_Check__MASTER_SITE_in_HOMEPAGE(c *check.C) {
s.Init(c)
s.UseCommandLine("-Wall")
diff --git a/pkgtools/pkglint/files/mklinechecker.go b/pkgtools/pkglint/files/mklinechecker.go
index f268d8f0aa5..4e25f7546a6 100644
--- a/pkgtools/pkglint/files/mklinechecker.go
+++ b/pkgtools/pkglint/files/mklinechecker.go
@@ -92,12 +92,11 @@ func (ck MkLineChecker) checkInclude() {
}
}
-func (ck MkLineChecker) checkCond(forVars map[string]bool) {
+func (ck MkLineChecker) checkCond(forVars map[string]bool, indentation *Indentation) {
mkline := ck.MkLine
directive := mkline.Directive()
args := mkline.Args()
- indentation := &G.Mk.indentation
switch directive {
case "endif", "endfor":
diff --git a/pkgtools/pkglint/files/mklines.go b/pkgtools/pkglint/files/mklines.go
index 08135dca444..fb301ef712c 100644
--- a/pkgtools/pkglint/files/mklines.go
+++ b/pkgtools/pkglint/files/mklines.go
@@ -130,7 +130,7 @@ func (mklines *MkLines) Check() {
}
case mkline.IsCond():
- ck.checkCond(mklines.forVars)
+ ck.checkCond(mklines.forVars, indentation)
case mkline.IsDependency():
ck.checkDependencyRule(allowedTargets)
diff --git a/pkgtools/pkglint/files/mkparser.go b/pkgtools/pkglint/files/mkparser.go
index 7647ccd63de..62af835324e 100644
--- a/pkgtools/pkglint/files/mkparser.go
+++ b/pkgtools/pkglint/files/mkparser.go
@@ -171,7 +171,7 @@ func (p *MkParser) VarUseModifiers(varname, closing string) []string {
}
case '[':
- if repl.AdvanceRegexp(`^\[[-.\d]+\]`) {
+ if repl.AdvanceRegexp(`^\[(?:[-.\d]+|#)\]`) {
modifiers = append(modifiers, repl.Since(modifierMark))
continue
}
diff --git a/pkgtools/pkglint/files/mkparser_test.go b/pkgtools/pkglint/files/mkparser_test.go
index 6d74793ffd6..a9bc039aa42 100644
--- a/pkgtools/pkglint/files/mkparser_test.go
+++ b/pkgtools/pkglint/files/mkparser_test.go
@@ -13,6 +13,7 @@ func (s *Suite) Test_MkParser_MkTokens(c *check.C) {
for i, expectedToken := range expectedTokens {
if i < len(actualTokens) {
c.Check(*actualTokens[i], deepEquals, *expectedToken)
+ c.Check(actualTokens[i].Varuse, deepEquals, expectedToken.Varuse)
}
}
c.Check(p.Rest(), equals, expectedRest)
@@ -80,6 +81,7 @@ func (s *Suite) Test_MkParser_MkTokens(c *check.C) {
check("${ALT_GCC_RTS:S%${LOCALBASE}%%:S%/%%}", varuse("ALT_GCC_RTS", "S%${LOCALBASE}%%", "S%/%%"))
check("${PREFIX:C;///*;/;g:C;/$;;}", varuse("PREFIX", "C;///*;/;g", "C;/$;;"))
check("${GZIP_CMD:[1]:Q}", varuse("GZIP_CMD", "[1]", "Q"))
+ check("${RUBY_RAILS_SUPPORTED:[#]}", varuse("RUBY_RAILS_SUPPORTED", "[#]"))
check("${DISTNAME:C/-[0-9]+$$//:C/_/-/}", varuse("DISTNAME", "C/-[0-9]+$$//", "C/_/-/"))
check("${DISTNAME:slang%=slang2%}", varuse("DISTNAME", "slang%=slang2%"))
check("${OSMAP_SUBSTVARS:@v@-e 's,\\@${v}\\@,${${v}},g' @}", varuse("OSMAP_SUBSTVARS", "@v@-e 's,\\@${v}\\@,${${v}},g' @"))
diff --git a/pkgtools/pkglint/files/patches.go b/pkgtools/pkglint/files/patches.go
index 9d2c3cb3a4e..633a6d5bbb9 100644
--- a/pkgtools/pkglint/files/patches.go
+++ b/pkgtools/pkglint/files/patches.go
@@ -90,7 +90,13 @@ func (ck *PatchChecker) Check() {
}
ChecklinesTrailingEmptyLines(ck.lines)
- SaveAutofixChanges(ck.lines)
+ sha1Before, err := computePatchSha1Hex(ck.lines[0].Filename)
+ if SaveAutofixChanges(ck.lines) && G.Pkg != nil && err == nil {
+ sha1After, err := computePatchSha1Hex(ck.lines[0].Filename)
+ if err == nil {
+ AutofixDistinfo(sha1Before, sha1After)
+ }
+ }
}
// See http://www.gnu.org/software/diffutils/manual/html_node/Detailed-Unified.html
diff --git a/pkgtools/pkglint/files/patches_test.go b/pkgtools/pkglint/files/patches_test.go
index a86e05d6d75..28259d4bb1d 100644
--- a/pkgtools/pkglint/files/patches_test.go
+++ b/pkgtools/pkglint/files/patches_test.go
@@ -27,11 +27,9 @@ func (s *Suite) Test_ChecklinesPatch__with_comment(c *check.C) {
s.CheckOutputEmpty()
}
-func (s *Suite) Test_ChecklinesPatch__without_empty_line(c *check.C) {
+func (s *Suite) Test_ChecklinesPatch__without_empty_line__autofix(c *check.C) {
s.Init(c)
- fname := s.CreateTmpFile("patch-WithoutEmptyLines", "dummy")
- s.UseCommandLine("-Wall", "--autofix")
- lines := s.NewLines(fname,
+ patchFilename := s.CreateTmpFileLines("patch-WithoutEmptyLines",
"$"+"NetBSD$",
"Text",
"--- file.orig",
@@ -41,15 +39,27 @@ func (s *Suite) Test_ChecklinesPatch__without_empty_line(c *check.C) {
"-old line",
"+old line",
" context after")
+ distinfoFilename := s.CreateTmpFileLines("distinfo",
+ "$"+"NetBSD$",
+ "",
+ "SHA1 (some patch) = 87ffcaaa0b0677ec679fff612b44df1af05f04df") // Taken from breakpoint at AutofixDistinfo
+
+ s.UseCommandLine("-Wall", "--autofix")
+ lines := LoadExistingLines(patchFilename, false)
+ G.CurrentDir = s.TmpDir()
+ G.Pkg = &Package{DistinfoFile: "distinfo"}
ChecklinesPatch(lines)
s.CheckOutputLines(
"AUTOFIX: ~/patch-WithoutEmptyLines:2: Inserting a line \"\" before this line.",
"AUTOFIX: ~/patch-WithoutEmptyLines:3: Inserting a line \"\" before this line.",
- "AUTOFIX: ~/patch-WithoutEmptyLines: Has been auto-fixed. Please re-run pkglint.")
+ "AUTOFIX: ~/patch-WithoutEmptyLines: Has been auto-fixed. Please re-run pkglint.",
+ "AUTOFIX: ~/distinfo:3: Replacing \"87ffcaaa0b0677ec679fff612b44df1af05f04df\" "+
+ "with \"a7c35294b3853da0acedf8a972cb266baa9582a3\".",
+ "AUTOFIX: ~/distinfo: Has been auto-fixed. Please re-run pkglint.")
- fixed, err := ioutil.ReadFile(fname)
+ fixed, err := ioutil.ReadFile(patchFilename)
c.Assert(err, check.IsNil)
c.Check(string(fixed), equals, ""+
"$"+"NetBSD$\n"+
@@ -63,6 +73,12 @@ func (s *Suite) Test_ChecklinesPatch__without_empty_line(c *check.C) {
"-old line\n"+
"+old line\n"+
" context after\n")
+ fixedDistinfo, err := ioutil.ReadFile(distinfoFilename)
+ c.Assert(err, check.IsNil)
+ c.Check(string(fixedDistinfo), equals, ""+
+ "$"+"NetBSD$\n"+
+ "\n"+
+ "SHA1 (some patch) = a7c35294b3853da0acedf8a972cb266baa9582a3\n")
}
func (s *Suite) Test_ChecklinesPatch__without_comment(c *check.C) {
diff --git a/pkgtools/pkglint/files/plist.go b/pkgtools/pkglint/files/plist.go
index cd18537e397..67c58d17745 100644
--- a/pkgtools/pkglint/files/plist.go
+++ b/pkgtools/pkglint/files/plist.go
@@ -175,8 +175,6 @@ func (ck *PlistChecker) checkpath(pline *PlistLine) {
ck.checkpathLib(pline, dirname, basename)
case "man":
ck.checkpathMan(pline)
- case "sbin":
- ck.checkpathSbin(pline)
case "share":
ck.checkpathShare(pline)
}
@@ -226,19 +224,6 @@ func (ck *PlistChecker) checkpathBin(pline *PlistLine, dirname, basename string)
pline.line.Warnf("The bin/ directory should not have subdirectories.")
return
}
-
- if G.opts.WarnExtra &&
- ck.allFiles["man/man1/"+basename+".1"] == nil &&
- ck.allFiles["man/man6/"+basename+".6"] == nil &&
- ck.allFiles["${IMAKE_MAN_DIR}/"+basename+".${IMAKE_MANNEWSUFFIX}"] == nil {
- pline.line.Warnf("Manual page missing for bin/%s.", basename)
- Explain(
- "All programs that can be run directly by the user should have a",
- "manual page for quick reference. The programs in the bin/ directory",
- "should have corresponding manual pages in section 1 (filename",
- "program.1), while the programs in the sbin/ directory have their",
- "manual pages in section 8.")
- }
}
func (ck *PlistChecker) checkpathEtc(pline *PlistLine, dirname, basename string) {
@@ -324,7 +309,7 @@ func (ck *PlistChecker) checkpathMan(pline *PlistLine) {
}
}
- if gz != "" {
+ if gz != "" && !line.AutofixReplaceRegexp(`\.gz$`, "") {
line.Notef("The .gz extension is unnecessary for manual pages.")
Explain(
"Whether the manual pages are installed in compressed form or not is",
@@ -334,20 +319,6 @@ func (ck *PlistChecker) checkpathMan(pline *PlistLine) {
}
}
-func (ck *PlistChecker) checkpathSbin(pline *PlistLine) {
- binname := pline.text[5:]
-
- if ck.allFiles["man/man8/"+binname+".8"] == nil && G.opts.WarnExtra {
- pline.line.Warnf("Manual page missing for sbin/%s.", binname)
- Explain(
- "All programs that can be run directly by the user should have a",
- "manual page for quick reference. The programs in the sbin/",
- "directory should have corresponding manual pages in section 8",
- "(filename program.8), while the programs in the bin/ directory",
- "have their manual pages in section 1.")
- }
-}
-
func (ck *PlistChecker) checkpathShare(pline *PlistLine) {
line, text := pline.line, pline.text
switch {
diff --git a/pkgtools/pkglint/files/plist_test.go b/pkgtools/pkglint/files/plist_test.go
index 09dcd634c4e..398b338bcd2 100644
--- a/pkgtools/pkglint/files/plist_test.go
+++ b/pkgtools/pkglint/files/plist_test.go
@@ -33,7 +33,6 @@ func (s *Suite) Test_ChecklinesPlist(c *check.C) {
s.CheckOutputLines(
"ERROR: PLIST:1: Expected \"@comment $"+"NetBSD$\".",
"WARN: PLIST:1: The bin/ directory should not have subdirectories.",
- "WARN: PLIST:2: Manual page missing for bin/program.",
"ERROR: PLIST:3: Configuration files must not be registered in the PLIST. Please use the CONF_FILES framework, which is described in mk/pkginstall/bsd.pkginstall.mk.",
"ERROR: PLIST:4: RCD_SCRIPTS must not be registered in the PLIST. Please use the RCD_SCRIPTS framework.",
"ERROR: PLIST:6: \"info/dir\" must not be listed. Use install-info to add/remove an entry.",
@@ -44,7 +43,6 @@ func (s *Suite) Test_ChecklinesPlist(c *check.C) {
"WARN: PLIST:10: Preformatted manual pages should end in \".0\".",
"WARN: PLIST:11: IMAKE_MANNEWSUFFIX is not meant to appear in PLISTs.",
"WARN: PLIST:12: Please remove this line. It is no longer necessary.",
- "WARN: PLIST:13: Manual page missing for sbin/clockctl.",
"ERROR: PLIST:14: The package Makefile must include \"../../graphics/gnome-icon-theme/buildlink3.mk\".",
"WARN: PLIST:14: Packages that install icon theme files should set ICON_THEMES.",
"ERROR: PLIST:15: Packages that install hicolor icons must include \"../../graphics/hicolor-icon-theme/buildlink3.mk\" in the Makefile.",
diff --git a/pkgtools/pkglint/files/vardefs.go b/pkgtools/pkglint/files/vardefs.go
index 02025e71494..b5ffbbccd78 100644
--- a/pkgtools/pkglint/files/vardefs.go
+++ b/pkgtools/pkglint/files/vardefs.go
@@ -807,16 +807,16 @@ func (gd *GlobalData) InitVartypes() {
usr("PKG_JVM_DEFAULT", lkNone, jvms)
acl("PKG_LEGACY_OPTIONS", lkShell, BtOption, "")
acl("PKG_LIBTOOL", lkNone, BtPathname, "Makefile: set")
- acl("PKG_OPTIONS", lkSpace, BtOption, "bsd.options.mk: set; *: use-loadtime, use")
- usr("PKG_OPTIONS.*", lkSpace, BtOption)
+ acl("PKG_OPTIONS", lkShell, BtOption, "bsd.options.mk: set; *: use-loadtime, use")
+ usr("PKG_OPTIONS.*", lkShell, BtOption)
acl("PKG_OPTIONS_DEPRECATED_WARNINGS", lkShell, BtShellWord, "")
- acl("PKG_OPTIONS_GROUP.*", lkSpace, BtOption, "Makefile, options.mk: set, append")
- acl("PKG_OPTIONS_LEGACY_OPTS", lkSpace, BtUnknown, "Makefile, Makefile.common, options.mk: append")
- acl("PKG_OPTIONS_LEGACY_VARS", lkSpace, BtUnknown, "Makefile, Makefile.common, options.mk: append")
- acl("PKG_OPTIONS_NONEMPTY_SETS", lkSpace, BtIdentifier, "")
- acl("PKG_OPTIONS_OPTIONAL_GROUPS", lkSpace, BtIdentifier, "options.mk: set, append")
- acl("PKG_OPTIONS_REQUIRED_GROUPS", lkSpace, BtIdentifier, "Makefile, options.mk: set")
- acl("PKG_OPTIONS_SET.*", lkSpace, BtOption, "")
+ acl("PKG_OPTIONS_GROUP.*", lkShell, BtOption, "Makefile, options.mk: set, append")
+ acl("PKG_OPTIONS_LEGACY_OPTS", lkShell, BtUnknown, "Makefile, Makefile.common, options.mk: append")
+ acl("PKG_OPTIONS_LEGACY_VARS", lkShell, BtUnknown, "Makefile, Makefile.common, options.mk: append")
+ acl("PKG_OPTIONS_NONEMPTY_SETS", lkShell, BtIdentifier, "")
+ acl("PKG_OPTIONS_OPTIONAL_GROUPS", lkShell, BtIdentifier, "options.mk: set, append")
+ acl("PKG_OPTIONS_REQUIRED_GROUPS", lkShell, BtIdentifier, "Makefile, options.mk: set")
+ acl("PKG_OPTIONS_SET.*", lkShell, BtOption, "")
acl("PKG_OPTIONS_VAR", lkNone, BtPkgOptionsVar, "Makefile, Makefile.common, options.mk: set; bsd.options.mk: use-loadtime")
acl("PKG_PRESERVE", lkNone, BtYes, "Makefile: set")
acl("PKG_SHELL", lkNone, BtPathname, "Makefile, Makefile.common: set")