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

lintian: r1089 - in trunk: checks debian testset testset/binary/debian



Author: rra
Date: 2008-01-03 04:08:31 +0100 (Thu, 03 Jan 2008)
New Revision: 1089

Modified:
   trunk/checks/menu-format.desc
   trunk/debian/changelog
   trunk/testset/binary/debian/menu
   trunk/testset/tags.binary
Log:
* checks/menu-format.desc:
  + [RA] Fix non-wm-module-in-wm-modules-menu-section pluralization to
    match the check and reword the long description to be hopefully
    clearer.  Thanks, Mart?\195?\173n Ferrari.  (Closes: #457527)

Modified: trunk/checks/menu-format.desc
===================================================================
--- trunk/checks/menu-format.desc	2008-01-03 02:56:27 UTC (rev 1088)
+++ trunk/checks/menu-format.desc	2008-01-03 03:08:31 UTC (rev 1089)
@@ -167,12 +167,12 @@
  Either it is a window manager and it should needs=wm, either it isn't and
  then it must be moved in another section.
 
-Tag: non-wm-module-in-wm-module-menu-section
+Tag: non-wm-module-in-wm-modules-menu-section
 Type: error
 Info: The menu item is in the FVWM Modules or Window Maker section but
- does not use a window-manager specific needs, like needs="fvwmmodule", or
- needs="wmmaker".  You should make it needs the window manager it is a
- module for and add support in the window-manager menu-method.
+ does not declare that it needs a specific window manager (using the needs
+ key in the menu file).  Modules for Fvwm should have needs="fvwmmodule".
+ Modules for WindowMaker should have needs="wmmaker".
 
 Tag: unquoted-string-in-menu-item
 Type: warning

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2008-01-03 02:56:27 UTC (rev 1088)
+++ trunk/debian/changelog	2008-01-03 03:08:31 UTC (rev 1089)
@@ -9,6 +9,10 @@
       Thanks, Joerg Jaspert.  (Closes: #458074)
   * checks/files.desc:
     + [RA] Remove a stray variable from the config-file-reserved tag.
+  * checks/menu-format.desc:
+    + [RA] Fix non-wm-module-in-wm-modules-menu-section pluralization to
+      match the check and reword the long description to be hopefully
+      clearer.  Thanks, Martín Ferrari.  (Closes: #457527)
   * checks/shared-libs{.desc,}:
     + [RA] New check for version numbers in symbol files.  Based on a
       patch from Raphael Hertzog.  (Closes: #457067)

Modified: trunk/testset/binary/debian/menu
===================================================================
--- trunk/testset/binary/debian/menu	2008-01-03 02:56:27 UTC (rev 1088)
+++ trunk/testset/binary/debian/menu	2008-01-03 03:08:31 UTC (rev 1089)
@@ -19,3 +19,6 @@
 ?package(binary):needs="text" section="Applications/System/Administration"\
  title="Run cfdisk (0)" command="cfdisk"
 ?package(binary):needs="text" section="Applications/System/Administration" title="Hello World" command="/usr/bin/hello"
+?package(binary):needs="wm" section="FVWM Modules" title="Fake Module" command="hello"
+?package(binary):needs="fvwmmodule" section="Window Maker" title="Fake Module"\
+ command="hello"

Modified: trunk/testset/tags.binary
===================================================================
--- trunk/testset/tags.binary	2008-01-03 02:56:27 UTC (rev 1088)
+++ trunk/testset/tags.binary	2008-01-03 03:08:31 UTC (rev 1089)
@@ -30,6 +30,8 @@
 E: binary: menu-item-missing-required-tag section /usr/share/menu/binary:1
 E: binary: non-wm-in-windowmanager-menu-section x11 /usr/lib/menu/binary:4
 E: binary: non-wm-in-windowmanager-menu-section x11 /usr/share/menu/binary:4
+E: binary: non-wm-module-in-wm-modules-menu-section wm /usr/lib/menu/binary:22
+E: binary: non-wm-module-in-wm-modules-menu-section wm /usr/share/menu/binary:22
 E: binary: postinst-does-not-call-installdocs usr/share/doc-base/space 
 E: binary: postinst-does-not-call-updatemenus usr/share/menu/binary
 E: binary: postrm-does-not-call-updatemenus usr/share/menu/binary


Reply to: