pkgsrc-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[pkgsrc/trunk]: pkgsrc/pkgtools/pkglint Don't complain about exact (non-range...



details:   https://anonhg.NetBSD.org/pkgsrc/rev/a0f3c0ff8536
branches:  trunk
changeset: 642890:a0f3c0ff8536
user:      schmonz <schmonz%pkgsrc.org@localhost>
date:      Sat Dec 06 22:21:30 2014 +0000

description:
Don't complain about exact (non-range, non-wildcard) dependencies.
Now that mail/qmail-run "looks fine", replace it with another of
my errant packages as a test case. Bump version.

diffstat:

 pkgtools/pkglint/Makefile         |   4 ++--
 pkgtools/pkglint/files/pkglint.pl |   4 ++--
 pkgtools/pkglint/files/pkglint.t  |  13 +++++--------
 3 files changed, 9 insertions(+), 12 deletions(-)

diffs (63 lines):

diff -r 469c5ccabb1a -r a0f3c0ff8536 pkgtools/pkglint/Makefile
--- a/pkgtools/pkglint/Makefile Sat Dec 06 22:14:34 2014 +0000
+++ b/pkgtools/pkglint/Makefile Sat Dec 06 22:21:30 2014 +0000
@@ -1,9 +1,9 @@
-# $NetBSD: Makefile,v 1.454 2014/11/23 15:30:07 mef Exp $
+# $NetBSD: Makefile,v 1.455 2014/12/06 22:21:30 schmonz Exp $
 
 # Note: if you update the version number, please have a look at the
 # changes between the CVS tag "pkglint_current" and HEAD.
 # After updating, please re-set the CVS tag to HEAD.
-PKGNAME=       pkglint-4.148
+PKGNAME=       pkglint-4.149
 CATEGORIES=    pkgtools
 
 OWNER=         wiz%NetBSD.org@localhost
diff -r 469c5ccabb1a -r a0f3c0ff8536 pkgtools/pkglint/files/pkglint.pl
--- a/pkgtools/pkglint/files/pkglint.pl Sat Dec 06 22:14:34 2014 +0000
+++ b/pkgtools/pkglint/files/pkglint.pl Sat Dec 06 22:21:30 2014 +0000
@@ -1,5 +1,5 @@
 #! @PERL@
-# $NetBSD: pkglint.pl,v 1.873 2014/10/09 13:54:47 wiz Exp $
+# $NetBSD: pkglint.pl,v 1.874 2014/12/06 22:21:30 schmonz Exp $
 #
 
 # pkglint - static analyzer and checker for pkgsrc packages
@@ -3930,7 +3930,7 @@
                },
 
                Dependency => sub {
-                       if ($value =~ m"^(${regex_pkgbase})(<|=|>|<=|>=|!=)(${regex_pkgversion})$") {
+                       if ($value =~ m"^(${regex_pkgbase})(<|=|>|<=|>=|!=|-)(${regex_pkgversion})$") {
                                my ($depbase, $depop, $depversion) = ($1, $2, $3);
 
                        } elsif ($value =~ m"^(${regex_pkgbase})-(?:\[(.*)\]\*|(\d+(?:\.\d+)*(?:\.\*)?)(\{,nb\*\}|\*|)|(.*))?$") {
diff -r 469c5ccabb1a -r a0f3c0ff8536 pkgtools/pkglint/files/pkglint.t
--- a/pkgtools/pkglint/files/pkglint.t  Sat Dec 06 22:14:34 2014 +0000
+++ b/pkgtools/pkglint/files/pkglint.t  Sat Dec 06 22:21:30 2014 +0000
@@ -1,5 +1,5 @@
 #! @PERL@
-# $NetBSD: pkglint.t,v 1.12 2014/11/24 16:31:34 schmonz Exp $
+# $NetBSD: pkglint.t,v 1.13 2014/12/06 22:21:30 schmonz Exp $
 #
 
 require 'pkglint.pl';                  # so we can test its internals
@@ -148,14 +148,11 @@
 
 sub test_lint_some_reference_packages {
        my %reference_packages = (
-               'mail/qmail-run' => {
+               'devel/syncdir' => {
                        stdout_re => <<EOT,
-^WARN: .*distinfo: File not found\. Please run '.*make makesum'\.
-ERROR: .*Makefile: Each package must define its LICENSE\.
-ERROR: .*Makefile:[0-9]+: Unknown dependency pattern \"qmail-qfilter-1\.5nb1\"\.
-WARN: .*Makefile:[0-9]+: The LOCALBASE variable should not be used by packages\.
-WARN: .*Makefile:[0-9]+: The LOCALBASE variable should not be used by packages\.
-2 errors and 3 warnings found\..*\$
+^ERROR: .*Makefile: Each package must define its LICENSE\.
+ERROR: .*patches/patch-aa:[0-9]+: Comment expected\.
+2 errors and 0 warnings found\..*\$
 EOT
                        stderr_re => undef,
                        exitcode => 1,



Home | Main Index | Thread Index | Old Index