Author: jamessan
Date: 2009-08-06 22:23:54 +0000 (Thu, 06 Aug 2009)
New Revision: 1956

Modified:
   trunk/debian/NEWS
   trunk/debian/changelog
   trunk/scripts/manpage-alert.sh
Log:
manpage-alert: Fix handling of commands which start with non-word characters.

Modified: trunk/debian/NEWS
===================================================================
--- trunk/debian/NEWS   2009-08-05 09:41:03 UTC (rev 1955)
+++ trunk/debian/NEWS   2009-08-06 22:23:54 UTC (rev 1956)
@@ -1,3 +1,16 @@
+devscripts (2.10.54) UNRELEASED; urgency=low
+
+    manpage-alert now, as originally intended, prints out the full path to the
+    binary which is lacking a manpage instead of simply the binary name.
+
+    Before:
+      No manual entry for somebinary
+
+    After:
+      No manual entry for /usr/bin/somebinary
+
+ -- James Vega <[email protected]>  Thu, 06 Aug 2009 18:03:41 -0400
+
 devscripts (2.10.24) unstable; urgency=low
 
     Starting with devscripts 2.10.21, debchange attempts to

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2009-08-05 09:41:03 UTC (rev 1955)
+++ trunk/debian/changelog      2009-08-06 22:23:54 UTC (rev 1956)
@@ -4,6 +4,8 @@
   * debchange: Clarify the wording for the explanation of
     --force-save-on-release.  (Closes: #539155)
   * grep-excuses: Document that -w only works with a package name.
+  * manpage-alert: Fix handling of commands which start with non-word
+    characters.  Based on a patch from Ville Skyttä.
 
   [ Adam D. Barratt ]
   * bts:

Modified: trunk/scripts/manpage-alert.sh
===================================================================
--- trunk/scripts/manpage-alert.sh      2009-08-05 09:41:03 UTC (rev 1955)
+++ trunk/scripts/manpage-alert.sh      2009-08-06 22:23:54 UTC (rev 1956)
@@ -67,8 +67,8 @@
         if [ $RET = "0" ]; then
             NUM_MANPAGES_FOUND=$(( $NUM_MANPAGES_FOUND + 1 ))
         else
-            echo "$OUT" | sed -e "/^.*'man 7 undocumented'.*$/ d" \
-              -e "s,\(.\)\b${F##*/}(\b|$),\1$F," -e 's,//,/,'
+            echo "$OUT" | perl -ne "next if /^.*'man 7 undocumented'.*$/;" \
+              -e "s,(\W)\Q${F##*/}\E(?:\b|$),\1$F,; s,//,/,; print;"
             NUM_MANPAGES_MISSING=$(( $NUM_MANPAGES_MISSING + 1 ))
         fi
     done



--
To unsubscribe, send mail to [email protected].

Reply via email to