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

lintian: r1318 - in trunk: checks debian



Author: djpig
Date: 2008-05-20 04:08:16 +0200 (Tue, 20 May 2008)
New Revision: 1318

Modified:
   trunk/checks/scripts
   trunk/debian/changelog
Log:
* checks/scripts:
  + Always use /bin/bash for shell script syntax check to give
    consistent errors on systems with changed /bin/sh.  Suggested
    by Raphael Geissert.  (Closes: #478192)


Modified: trunk/checks/scripts
===================================================================
--- trunk/checks/scripts	2008-05-20 01:45:09 UTC (rev 1317)
+++ trunk/checks/scripts	2008-05-20 02:08:16 UTC (rev 1318)
@@ -469,7 +469,7 @@
     if ($shellscript) {
 	$checkbashisms = $base eq "sh" ? 1 : 0;
 	if ($base eq 'sh' or $base eq 'bash') {
-	    if (check_script_syntax("/bin/$base", $filename)) {
+	    if (check_script_syntax("/bin/bash", $filename)) {
 		tag("maintainer-shell-script-fails-syntax-check", $file);
 	    }
 	}

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2008-05-20 01:45:09 UTC (rev 1317)
+++ trunk/debian/changelog	2008-05-20 02:08:16 UTC (rev 1318)
@@ -21,6 +21,10 @@
   * checks/patch-systems:
     + [FL] Add basic support for dpatch list files which use cpp for
       preprocessing.  Patch by Jörg Sommer.  (Closes: #477057)
+  * checks/scripts:
+    + [FL] Always use /bin/bash for shell script syntax check to give
+      consistent errors on systems with changed /bin/sh.  Suggested
+      by Raphael Geissert.  (Closes: #478192)
   
   * frontend/lintian:
     + [FL] Don't issue a warning about Distribution: UNRELEASED in


Reply to: