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

lintian: r196 - in branches/1.23.0: checks debian



Author: jeroen
Date: 2004-04-19 23:11:42 +0200 (Mon, 19 Apr 2004)
New Revision: 196

Modified:
   branches/1.23.0/checks/menus
   branches/1.23.0/debian/changelog
Log:
checks/menus:
+ [JvW] Stricter checking for whether update-menus is actually called
  (Closes: #244628)


Modified: branches/1.23.0/checks/menus
===================================================================
--- branches/1.23.0/checks/menus	2004-04-19 21:10:19 UTC (rev 195)
+++ branches/1.23.0/checks/menus	2004-04-19 21:11:42 UTC (rev 196)
@@ -279,7 +279,9 @@
 	}
 
 	# does the script call update-menus?
-	if (/update-menus/ && !/--no-update-menus/) {
+	# TODO this regex-magic should be moved to some lib for checking
+	# whether a certain word is likely called as command... --Jeroen
+	if (/(?:^\s*|[;&|]|(?:then|do)\s+)update-menus(?:\s|[;&|<>]|$)/) {
 	    # yes, it does.
 	    $pres->{'calls-updatemenus'} = 1;
 

Modified: branches/1.23.0/debian/changelog
===================================================================
--- branches/1.23.0/debian/changelog	2004-04-19 21:10:19 UTC (rev 195)
+++ branches/1.23.0/debian/changelog	2004-04-19 21:11:42 UTC (rev 196)
@@ -166,7 +166,7 @@
       based on patch by Robert Lemmen <robertle@semistable.com>, thanks!
       (Closes: #198617)
 
- -- Debian Lintian Maintainers <lintian-maint@debian.org>  Sun, 18 Apr 2004 21:23:21 +0200
+ -- Debian Lintian Maintainers <lintian-maint@debian.org>  Mon, 19 Apr 2004 23:01:36 +0200
 
 lintian (1.22.12) unstable; urgency=low
 
@@ -243,6 +243,8 @@
   * checks/menus{,.desc}:
     + [FL] Remove non-exectuable-in-etc-menu-methods as it doesn't
       reflects the current menu manual (Closes: #229460)
+    + [JvW] Stricter checking for whether update-menus is actually called
+      (Closes: #244628)
   * checks/po-debconf:
     + [JvW] Check whether files listed in debian/po/POTFILES.in actually
       exist, and only positively detect po-debconf usage if dependency is



Reply to: