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

[SCM] Debian package checker branch, master, updated. 2.2.5-41-ge817bea



The following commit has been merged in the master branch:
commit d240f466c2333ed010e17e633e8712762362f679
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date:   Sun Feb 22 00:20:43 2009 +0000

    Disable the check for embedded copies of yahoo.js to avoid false positives.
    
    Modify the check for embedded Javascript libraries to ignore yahoo.js as
    there are similarly named, but unrelated,  scripts in the archive.

diff --git a/checks/files b/checks/files
index e0c3490..690fbb3 100644
--- a/checks/files
+++ b/checks/files
@@ -37,7 +37,10 @@ our @jslibraries = (
     [ qr,(?i)fckeditor\.js(\.gz)?$, => 'fckeditor' ],
     [ qr,(?i)cropper(\.uncompressed)?\.js(\.gz)?$, => 'libjs-cropper' ],
     [ qr,(?i)swfobject\.js(\.gz)?$, => 'libjs-yui' ],
-    [ qr,(?i)yahoo(-(dom-event|min))?\.js(\.gz)?$, => 'libjs-yui' ],
+    [ qr,(?i)yahoo-(dom-event|min)\.js(\.gz)?$, => 'libjs-yui' ],
+# Disabled due to false positives.  Needs a content check adding to verify
+# that the file being checked is /the/ yahoo.js
+#    [ qr,(?i)yahoo\.js(\.gz)?$, => 'libjs-yui' ],
     [ qr,(?i)jsjac(\.packed)?\.js(\.gz)?$, => 'libjs-jac' ],
     [ qr,(?i)jsMath(-fallback-\w+)?\.js(\.gz)?$, => 'jsmath' ],
     [ qr,(?i)tiny_mce(_(popup|src))?\.js(\.gz)?$, => qr'tinymce2?' ],
diff --git a/debian/changelog b/debian/changelog
index 9fcbf38..dbc88b8 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -33,6 +33,9 @@ lintian (2.2.6) UNRELEASED; urgency=low
       Geissert.  (Closes: #509624)
     + [RA] Make static regexes global to avoid recreating them for each
       file we scan.  Patch from Raphael Geissert.
+    + [ADB] Modify the check for embedded Javascript libraries to ignore
+      yahoo.js as there are similarly named, but unrelated,  scripts in
+      the archive.      
   * checks/scripts:
     + [RA] Make static regexes global to avoid recreating them for each
       file we scan.  Patch from Raphael Geissert.

-- 
Debian package checker


Reply to: