[pkgsrc/trunk]: pkgsrc/pkgtools/pkglint/files Warn if GNU_CONFIGURE is set bu...

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

details:   https://anonhg.NetBSD.org/pkgsrc/rev/a513ed081e6f
branches:  trunk
changeset: 516440:a513ed081e6f
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Fri Jul 21 05:02:52 2006 +0000

description:
Warn if GNU_CONFIGURE is set but USE_LANGUAGES does not contain "c", since
that one is needed in almost all cases.

diffstat:

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

diffs (30 lines):

diff -r 220534bcb27d -r a513ed081e6f pkgtools/pkglint/files/pkglint.pl
--- a/pkgtools/pkglint/files/pkglint.pl Fri Jul 21 04:51:11 2006 +0000
+++ b/pkgtools/pkglint/files/pkglint.pl Fri Jul 21 05:02:52 2006 +0000
@@ -1,5 +1,5 @@
 #! @PERL@
-# $NetBSD: pkglint.pl,v 1.657 2006/07/18 21:13:22 rillig Exp $
+# $NetBSD: pkglint.pl,v 1.658 2006/07/21 05:02:52 rillig Exp $
 #
 
 # pkglint - static analyzer and checker for pkgsrc packages
@@ -6275,6 +6275,19 @@
                $pkgctx_vardef->{"RESTRICTED"}->log_error("Restricted packages must have a LICENSE.");
        }
 
+       if (exists($pkgctx_vardef->{"GNU_CONFIGURE"}) && exists($pkgctx_vardef->{"USE_LANGUAGES"})) {
+               my $languages_line = $pkgctx_vardef->{"USE_LANGUAGES"};
+
+               if ($languages_line->text =~ regex_varassign) {
+                       my (undef, $op, $value, undef) = ($1, $2, $3, $4);
+
+                       if ($value !~ qr"\bc\b") {
+                               $pkgctx_vardef->{"GNU_CONFIGURE"}->log_warning("GNU_CONFIGURE almost always needs a C compiler, ...");
+                               $languages_line->log_warning("... but \"c\" is not added to USE_LANGUAGES.");
+                       }
+               }
+       }
+
        my $distname_line = $pkgctx_vardef->{"DISTNAME"};
        my $pkgname_line = $pkgctx_vardef->{"PKGNAME"};