pkgsrc-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[pkgsrc/trunk]: pkgsrc/pkgtools/pkglint pkgtools/pkglint: update to 5.5.12
details: https://anonhg.NetBSD.org/pkgsrc/rev/12724d43e307
branches: trunk
changeset: 380681:12724d43e307
user: rillig <rillig%pkgsrc.org@localhost>
date: Sat May 19 12:58:24 2018 +0000
description:
pkgtools/pkglint: update to 5.5.12
Changes since 5.5.11:
* Improve support for TEST_DEPENDS and USE_TOOLS+= *:test (thanks @leot)
* Add checks for ALTERNATIVES files
diffstat:
pkgtools/pkglint/Makefile | 4 +-
pkgtools/pkglint/files/alternatives.go | 42 +++++++++++++++++++++++++++++
pkgtools/pkglint/files/alternatives_test.go | 27 ++++++++++++++++++
pkgtools/pkglint/files/distinfo.go | 7 ++--
pkgtools/pkglint/files/distinfo_test.go | 2 +-
pkgtools/pkglint/files/licenses.go | 9 +++--
pkgtools/pkglint/files/line.go | 4 +-
pkgtools/pkglint/files/package.go | 11 +++++++
pkgtools/pkglint/files/pkglint.go | 3 --
pkgtools/pkglint/files/pkgsrc.go | 7 ++++-
pkgtools/pkglint/files/plist.go | 4 +-
pkgtools/pkglint/files/toplevel.go | 4 +-
pkgtools/pkglint/files/vartypecheck.go | 4 +-
pkgtools/pkglint/files/vartypecheck_test.go | 4 +-
14 files changed, 108 insertions(+), 24 deletions(-)
diffs (truncated from 322 to 300 lines):
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/Makefile
--- a/pkgtools/pkglint/Makefile Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/Makefile Sat May 19 12:58:24 2018 +0000
@@ -1,6 +1,6 @@
-# $NetBSD: Makefile,v 1.538 2018/05/14 20:25:48 rillig Exp $
+# $NetBSD: Makefile,v 1.539 2018/05/19 12:58:24 rillig Exp $
-PKGNAME= pkglint-5.5.11
+PKGNAME= pkglint-5.5.12
DISTFILES= # none
CATEGORIES= pkgtools
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/alternatives.go
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/pkgtools/pkglint/files/alternatives.go Sat May 19 12:58:24 2018 +0000
@@ -0,0 +1,42 @@
+package main
+
+import (
+ "netbsd.org/pkglint/regex"
+ "strings"
+)
+
+func CheckfileAlternatives(filename string, plistFiles map[string]bool) {
+ lines, err := readLines(filename, false)
+ if err != nil {
+ return
+ }
+
+ for _, line := range lines {
+ if m, wrapper, space, implementation := match3(line.Text, `^(\S+)([ \t]+)(\S+)`); m {
+ if plistFiles[wrapper] {
+ line.Errorf("Alternative wrapper %q must not appear in the PLIST.", wrapper)
+ }
+
+ relImplementation := strings.Replace(implementation, "@PREFIX@/", "", 1)
+ plistName := regex.Compile(`@(\w+)@`).ReplaceAllString(relImplementation, "${$1}")
+ if !plistFiles[plistName] && !G.Pkg.vars.Defined("ALTERNATIVES_SRC") {
+ if plistName != implementation {
+ line.Errorf("Alternative implementation %q must appear in the PLIST as %q.", implementation, plistName)
+ } else {
+ line.Errorf("Alternative implementation %q must appear in the PLIST.", implementation)
+ }
+ }
+
+ fix := line.Autofix()
+ fix.Notef("@PREFIX@/ can be omitted from the file name.")
+ fix.Explain(
+ "The alternative implementation is always interpreted relative to ${PREFIX}.")
+ fix.ReplaceAfter(space, "@PREFIX@/", "")
+ fix.Apply()
+ } else {
+ line.Errorf("Invalid ALTERNATIVES line %q.", line.Text)
+ Explain(
+ "Run \"" + confMake + " help topic=alternatives\" for more information.")
+ }
+ }
+}
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/alternatives_test.go
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/pkgtools/pkglint/files/alternatives_test.go Sat May 19 12:58:24 2018 +0000
@@ -0,0 +1,27 @@
+package main
+
+import "gopkg.in/check.v1"
+
+func (s *Suite) Test_Alternatives_PLIST(c *check.C) {
+ t := s.Init(c)
+
+ t.SetupFileLines("ALTERNATIVES",
+ "sbin/sendmail @PREFIX@/sbin/sendmail.postfix@POSTFIXVER@",
+ "sbin/sendmail @PREFIX@/sbin/sendmail.exim@EXIMVER@",
+ "bin/echo bin/gnu-echo",
+ "bin/editor bin/vim -e")
+
+ G.Pkg = NewPackage("")
+ G.Pkg.PlistFiles["bin/echo"] = true
+ G.Pkg.PlistFiles["bin/vim"] = true
+ G.Pkg.PlistFiles["sbin/sendmail.exim${EXIMVER}"] = true
+
+ CheckfileAlternatives(t.TempFilename("ALTERNATIVES"), G.Pkg.PlistFiles)
+
+ t.CheckOutputLines(
+ "ERROR: ~/ALTERNATIVES:1: Alternative implementation \"@PREFIX@/sbin/sendmail.postfix@POSTFIXVER@\" must appear in the PLIST as \"sbin/sendmail.postfix${POSTFIXVER}\".",
+ "NOTE: ~/ALTERNATIVES:1: @PREFIX@/ can be omitted from the file name.",
+ "NOTE: ~/ALTERNATIVES:2: @PREFIX@/ can be omitted from the file name.",
+ "ERROR: ~/ALTERNATIVES:3: Alternative wrapper \"bin/echo\" must not appear in the PLIST.",
+ "ERROR: ~/ALTERNATIVES:3: Alternative implementation \"bin/gnu-echo\" must appear in the PLIST.")
+}
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/distinfo.go
--- a/pkgtools/pkglint/files/distinfo.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/distinfo.go Sat May 19 12:58:24 2018 +0000
@@ -157,16 +157,17 @@
// Inter-package check for differing distfile checksums.
func (ck *distinfoLinesChecker) checkGlobalMismatch(line Line, fname, alg, hash string) {
- if G.Hash != nil && !hasPrefix(fname, "patch-") { // Intentionally checking the filename instead of ck.isPatch
+ hashes := G.Pkgsrc.Hashes
+ if hashes != nil && !hasPrefix(fname, "patch-") { // Intentionally checking the filename instead of ck.isPatch
key := alg + ":" + fname
- otherHash := G.Hash[key]
+ otherHash := hashes[key]
if otherHash != nil {
if otherHash.hash != hash {
line.Errorf("The hash %s for %s is %s, which differs from %s in %s.",
alg, fname, hash, otherHash.hash, otherHash.line.ReferenceFrom(line))
}
} else {
- G.Hash[key] = &Hash{hash, line}
+ hashes[key] = &Hash{hash, line}
}
}
}
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/distinfo_test.go
--- a/pkgtools/pkglint/files/distinfo_test.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/distinfo_test.go Sat May 19 12:58:24 2018 +0000
@@ -32,7 +32,7 @@
t := s.Init(c)
otherLine := t.NewLine("other/distinfo", 7, "dummy")
- G.Hash = map[string]*Hash{"SHA512:pkgname-1.0.tar.gz": {"Some-512-bit-hash", otherLine}}
+ G.Pkgsrc.Hashes = map[string]*Hash{"SHA512:pkgname-1.0.tar.gz": {"Some-512-bit-hash", otherLine}}
lines := t.NewLines("distinfo",
RcsID,
"",
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/licenses.go
--- a/pkgtools/pkglint/files/licenses.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/licenses.go Sat May 19 12:58:24 2018 +0000
@@ -6,7 +6,8 @@
)
func checkToplevelUnusedLicenses() {
- if G.UsedLicenses == nil {
+ usedLicenses := G.Pkgsrc.UsedLicenses
+ if usedLicenses == nil {
return
}
@@ -16,7 +17,7 @@
licensename := licensefile.Name()
licensepath := licensedir + "/" + licensename
if fileExists(licensepath) {
- if !G.UsedLicenses[licensename] {
+ if !usedLicenses[licensename] {
NewLineWhole(licensepath).Warnf("This license seems to be unused.")
}
}
@@ -52,8 +53,8 @@
}
if licenseFile == "" {
licenseFile = G.Pkgsrc.File("licenses/" + license)
- if G.UsedLicenses != nil {
- G.UsedLicenses[license] = true
+ if G.Pkgsrc.UsedLicenses != nil {
+ G.Pkgsrc.UsedLicenses[license] = true
}
}
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/line.go
--- a/pkgtools/pkglint/files/line.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/line.go Sat May 19 12:58:24 2018 +0000
@@ -162,7 +162,7 @@
// Usage:
//
-// fix := mkline.Line.Autofix()
+// fix := line.Autofix()
//
// fix.Errorf("Must not be ...")
// fix.Warnf("Should not be ...")
@@ -174,7 +174,7 @@
//
// fix.Replace("from", "to")
// fix.ReplaceAfter("prefix", "from", "to")
-// fix.ReplaceRegex(`\s+`, "space", "from", "to")
+// fix.ReplaceRegex(`\s+`, "space", -1)
// fix.InsertBefore("new line")
// fix.InsertAfter("new line")
// fix.Delete()
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/package.go
--- a/pkgtools/pkglint/files/package.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/package.go Sat May 19 12:58:24 2018 +0000
@@ -26,6 +26,7 @@
EffectivePkgnameLine MkLine // The origin of the three effective_* values
SeenBsdPrefsMk bool // Has bsd.prefs.mk already been included?
PlistDirs map[string]bool // Directories mentioned in the PLIST files
+ PlistFiles map[string]bool // Regular files mentioned in the PLIST files
vars Scope
bl3 map[string]Line // buildlink3.mk name => line; contains only buildlink3.mk files that are directly included.
@@ -42,6 +43,7 @@
pkg := &Package{
Pkgpath: pkgpath,
PlistDirs: make(map[string]bool),
+ PlistFiles: make(map[string]bool),
vars: NewScope(),
bl3: make(map[string]Line),
plistSubstCond: make(map[string]bool),
@@ -216,6 +218,14 @@
}
}
+ if G.opts.CheckAlternatives {
+ for _, fname := range files {
+ if path.Base(fname) == "ALTERNATIVES" {
+ CheckfileAlternatives(fname, pkg.PlistFiles)
+ }
+ }
+ }
+
if !isEmptyDir(G.CurrentDir + "/scripts") {
NewLineWhole(G.CurrentDir + "/scripts").Warnf("This directory and its contents are deprecated! Please call the script(s) explicitly from the corresponding target(s) in the pkg's
Makefile.")
}
@@ -934,6 +944,7 @@
for _, line := range lines {
text := line.Text
+ pkg.PlistFiles[text] = true // XXX: ignores PLIST conditionals for now
// Keep in sync with PlistChecker.collectFilesAndDirs
if !contains(text, "$") && !contains(text, "@") {
for dir := path.Dir(text); dir != "."; dir = path.Dir(dir) {
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/pkglint.go
--- a/pkgtools/pkglint/files/pkglint.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/pkglint.go Sat May 19 12:58:24 2018 +0000
@@ -41,9 +41,6 @@
CvsEntriesDir string // Cached to avoid I/O
CvsEntriesLines []Line
- Hash map[string]*Hash // Maps "alg:fname" => hash (inter-package check).
- UsedLicenses map[string]bool // Maps "license name" => true (inter-package check).
-
errors int
warnings int
explainNext bool
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/pkgsrc.go
--- a/pkgtools/pkglint/files/pkgsrc.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/pkgsrc.go Sat May 19 12:58:24 2018 +0000
@@ -35,6 +35,9 @@
UserDefinedVars map[string]MkLine // varname => line; used for checking BUILD_DEFS
Deprecated map[string]string //
vartypes map[string]*Vartype // varcanon => type
+
+ Hashes map[string]*Hash // Maps "alg:fname" => hash (inter-package check).
+ UsedLicenses map[string]bool // Maps "license name" => true (inter-package check).
}
func NewPkgsrc(dir string) *Pkgsrc {
@@ -51,7 +54,9 @@
make(map[string]string),
make(map[string]MkLine),
make(map[string]string),
- make(map[string]*Vartype)}
+ make(map[string]*Vartype),
+ nil, // Only initialized when pkglint is run for a whole pkgsrc installation
+ nil}
// Some user-defined variables do not influence the binary
// package at all and therefore do not have to be added to
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/plist.go
--- a/pkgtools/pkglint/files/plist.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/plist.go Sat May 19 12:58:24 2018 +0000
@@ -402,8 +402,8 @@
line := pline.line
if cmd == "unexec" {
- if m, arg := match1(arg, `^(?:rmdir|\$\{RMDIR\} \%D/)(.*)`); m {
- if !contains(arg, "true") && !contains(arg, "${TRUE}") {
+ if m, dir := match1(arg, `^(?:rmdir|\$\{RMDIR\} \%D/)(.*)`); m {
+ if !contains(dir, "true") && !contains(dir, "${TRUE}") {
pline.line.Warnf("Please remove this line. It is no longer necessary.")
}
}
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/toplevel.go
--- a/pkgtools/pkglint/files/toplevel.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/toplevel.go Sat May 19 12:58:24 2018 +0000
@@ -32,8 +32,8 @@
if G.opts.Recursive {
if G.opts.CheckGlobal {
- G.UsedLicenses = make(map[string]bool)
- G.Hash = make(map[string]*Hash)
+ G.Pkgsrc.UsedLicenses = make(map[string]bool)
+ G.Pkgsrc.Hashes = make(map[string]*Hash)
}
G.Todo = append(append([]string(nil), ctx.subdirs...), G.Todo...)
}
diff -r 1dfa2c7b718a -r 12724d43e307 pkgtools/pkglint/files/vartypecheck.go
--- a/pkgtools/pkglint/files/vartypecheck.go Sat May 19 12:38:28 2018 +0000
+++ b/pkgtools/pkglint/files/vartypecheck.go Sat May 19 12:58:24 2018 +0000
@@ -950,9 +950,9 @@
cv.Line.Errorf("Unknown tool %q.", toolname)
}
switch tooldep {
- case "", "bootstrap", "build", "pkgsrc", "run":
+ case "", "bootstrap", "build", "pkgsrc", "run", "test":
default:
- cv.Line.Errorf("Unknown tool dependency %q. Use one of \"bootstrap\", \"build\", \"pkgsrc\" or \"run\".", tooldep)
+ cv.Line.Errorf("Unknown tool dependency %q. Use one of \"bootstrap\", \"build\", \"pkgsrc\" or \"run\" or \"test\".", tooldep)
}
} else if cv.Op != opUseMatch {
Home |
Main Index |
Thread Index |
Old Index