[pkgsrc/trunk]: pkgsrc/pkgtools/pkglint/files Added explicit checks for share...

[rillig <rillig%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/0b03470cc60f
branches:  trunk
changeset: 551029:0b03470cc60f
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Thu Dec 04 18:07:52 2008 +0000

description:
Added explicit checks for share/icons/hicolor directories in PLISTs, as
suggested by wiz.

diffstat:

 pkgtools/pkglint/files/pkglint.pl |  8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diffs (30 lines):

diff -r 59e697a32ca4 -r 0b03470cc60f pkgtools/pkglint/files/pkglint.pl
--- a/pkgtools/pkglint/files/pkglint.pl Thu Dec 04 17:35:18 2008 +0000
+++ b/pkgtools/pkglint/files/pkglint.pl Thu Dec 04 18:07:52 2008 +0000
@@ -1,5 +1,5 @@
 #! @PERL@
-# $NetBSD: pkglint.pl,v 1.792 2008/12/02 09:00:28 rillig Exp $
+# $NetBSD: pkglint.pl,v 1.793 2008/12/04 18:07:52 rillig Exp $
 #
 
 # pkglint - static analyzer and checker for pkgsrc packages
@@ -7420,6 +7420,9 @@
 "created the directory.");
                                }
 
+                               if ($pkgpath ne "graphics/hicolor-icon-theme" && $arg =~ m"^share/icons/hicolor(?:$|/)") {
+                                       $line->log_warning("Please .include \"../../graphics/hicolor-icon-theme/buildlink3.mk\" and remove this line.");
+                               }
                        } elsif ($cmd eq "imake-man") {
                                my (@args) = split(/\s+/, $arg);
                                if (@args != 3) {
@@ -7567,6 +7570,9 @@
                        } elsif (defined($effective_pkgbase) && $text =~ m"^share/\Q${effective_pkgbase}\E/") {
                                # Fine.
 
+                       } elsif ($pkgpath ne "graphics/hicolor-icon-theme" && $text =~ m"^share/icons/hicolor/icon-theme\.cache") {
+                               $line->log_error("Please .include \"../../graphics/hicolor-icon-theme/buildlink3.mk\" and remove this line.");
+
                        } elsif ($text =~ m"^share/info/") {
                                $line->log_warning("Info pages should be installed into info/, not share/info/.");
                                $line->explain_warning(