Author: djpig
Date: 2008-06-17 00:47:32 +0200 (Tue, 17 Jun 2008)
New Revision: 1384

Modified:
   trunk/checks/scripts
   trunk/debian/changelog
Log:
* checks/scripts:
  + Do not check zsh scripts, since zsh -n gives false
    positives.  Downgrades #485885 to wishlist.  Also see
    #175467.  Patch by Raphael Geissert.


Modified: trunk/checks/scripts
===================================================================
--- trunk/checks/scripts        2008-06-16 13:50:06 UTC (rev 1383)
+++ trunk/checks/scripts        2008-06-16 22:47:32 UTC (rev 1384)
@@ -289,7 +289,8 @@
     # Syntax-check most shell scripts, but don't syntax-check scripts that end
     # in .dpatch.  bash -n doesn't stop checking at exit 0 and goes on to blow
     # up on the patch itself.
-    if ($base =~ /^(?:(?:b|d)?a|k|z)?sh$/) {
+    # zsh -n is broken, see #485885
+    if ($base =~ /^(?:(?:b|d)?a|k)?sh$/) {
        if (-x "$interpreter" && ! 
script_is_evil_and_wrong("unpacked/$filename")) {
            if ($filename !~ m,\.dpatch$,) {
                if (check_script_syntax($interpreter, "unpacked/$filename")) {

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-06-16 13:50:06 UTC (rev 1383)
+++ trunk/debian/changelog      2008-06-16 22:47:32 UTC (rev 1384)
@@ -3,6 +3,9 @@
   * checks/scripts:
     + [FL] Apply File::Spec->canonpath() to rpaths to improve
       matching with directories.
+    + [FL] Do not check zsh scripts, since zsh -n gives false
+      positives.  Downgrades #485885 to wishlist.  Also see
+      #175467.  Patch by Raphael Geissert.
 
  --
 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to