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

[SCM] Debian package checker branch, master, updated. 2.5.3-29-g57a624c



The following commit has been merged in the master branch:
commit 57a624c0e2ddad07a374a11031712ea25d491841
Author: Russ Allbery <rra@debian.org>
Date:   Sun Sep 25 15:43:59 2011 -0700

    Exclude *.erb files from several shell script tests
    
    * checks/scripts:
      + [RA] Exclude files ending in *.erb from several shell script
        checks.  ERB is a Ruby templating language, so these will probably
        have mixed shell and Ruby code and may not be executable.  Thanks,
        Laurent Arnoud.

diff --git a/checks/scripts b/checks/scripts
index a58656b..6e36b5a 100644
--- a/checks/scripts
+++ b/checks/scripts
@@ -389,6 +389,7 @@ for my $filename (sort keys %{$info->scripts}) {
 		or $filename =~ m,^usr/(lib|share)/.*\.py,
 		or $filename =~ m,^usr/(lib|share)/ruby/.*\.rb,
 		or $filename =~ m,\.in$,
+		or $filename =~ m,\.erb$,
 		or $filename =~ m,\.ex$,
 		or $filename eq 'etc/init.d/skeleton'
 		or $filename =~ m,^etc/menu-methods,
@@ -410,6 +411,7 @@ for my $filename (sort keys %{$info->scripts}) {
 	if (-x $interpreter
 	    and ! script_is_evil_and_wrong($path)
 	    and $filename !~ m,\.dpatch$,
+	    and $filename !~ m,\.erb$,
 	    # exclude some shells. zsh -n is broken, see #485885
 	    and $base !~ m/^(z|t?c)sh$/) {
 
diff --git a/debian/changelog b/debian/changelog
index 90e8f59..c356e8b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -11,6 +11,11 @@ lintian (2.5.4) UNRELEASED; urgency=low
   * checks/files.desc:
     + [JW] Suggest the usage of "-delete" rather than "| xargs rm -f".
       (Closes: #641983)
+  * checks/scripts:
+    + [RA] Exclude files ending in *.erb from several shell script
+      checks.  ERB is a Ruby templating language, so these will probably
+      have mixed shell and Ruby code and may not be executable.  Thanks,
+      Laurent Arnoud.
   * checks/source-copyright:
     + [JW] Added some more DEP-5 URLs.  (Closes: #642027)
 

-- 
Debian package checker


Reply to: