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

[lintian] 01/01: Fix a source-is-missing false positive



This is an automated email from the git hooks/post-receive script.

broucaries-guest pushed a commit to branch master
in repository lintian.

commit 7911e7d21b759e08ae1e5bc0d66a25fb4c058c96
Author: Bastien ROUCARIÈS <roucaries.bastien+debian@gmail.com>
Date:   Mon Jul 14 10:59:37 2014 +0200

    Fix a source-is-missing false positive
    
    Add debug source suffix for detecting javascript
    source-is-missing and improve test suite. Fix a
    false postive in mono.
    
    Signed-off-by: Bastien ROUCARIÈS <roucaries.bastien+debian@gmail.com>
---
 data/cruft/warn-file-type                                             | 2 +-
 debian/changelog                                                      | 4 ++++
 .../cruft-minified-js/debian/oldfalsepositives/debug.js/test.debug.js | 1 +
 t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.js   | 2 ++
 4 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/data/cruft/warn-file-type b/data/cruft/warn-file-type
index 0dd5794..ea2f5ae 100644
--- a/data/cruft/warn-file-type
+++ b/data/cruft/warn-file-type
@@ -11,7 +11,7 @@ source-contains-prebuilt-flash-object       ~~ ^Macromedia \s Flash
 source-contains-prebuilt-flash-project      ~~ ^Composite \s Document \s File                                            ~~ (?i)\.fla$                        ~~ s/(?i)\.fla$/.as/
 source-contains-prebuilt-java-object        ~~ (?: Java [ ] (?:Jar [ ] file|archive [ ] data) | Zip [ ] archive | JAR)   ~~ (?i)\.jar$
 # do not forget to change also $JS_EXT in file.pm
-source-contains-prebuilt-javascript-object  ~~ .*                                                                        ~~ (?i)[-._](?:compiled|lite|min|pack(?:ed)?|yc)\.js$ ~~ map{'s/(?i)[-._](?:compiled|lite|min|pack(?:ed)?|yc)\.js$/'.$_.'/'} qw(.js _orig.js .js.orig .src.js)
+source-contains-prebuilt-javascript-object  ~~ .*                                                                        ~~ (?i)[-._](?:compiled|lite|min|pack(?:ed)?|yc)\.js$ ~~ map{'s/(?i)[-._](?:compiled|lite|min|pack(?:ed)?|yc)\.js$/'.$_.'/'} qw(.js _orig.js .js.orig .src.js .debug.js)
 # ok chm could be created from docbook or texi
 source-contains-prebuilt-ms-help-file       ~~ ^MS \s Windows \s Html \s? Help \s Data ~~  (?i)\.chm$
 source-contains-prebuilt-python-object      ~~ ^python \s \d(\.\d+)? \s byte-compiled                                    ~~ .*                                ~~ s/(?i)(?:\.cpython-\d{2}|\.pypy)?\.py[co]$/.py/
diff --git a/debian/changelog b/debian/changelog
index 0d22052..106afe4 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -13,6 +13,10 @@ lintian (2.5.26) UNRELEASED; urgency=medium
       missing-pkg-php-tools-addon phpcomposer, thanks to
       David Prévot (Closes: #754738).
 
+  * data/cruft/warn-file-type:
+    + [BR] add debug source suffix for detecting javascript
+      source-is-missing and improve test suite. Fix a
+      false postive in mono.
   * data/debhelper/dh_commands:
     + [BR] Add patch from  Piotr Ożarowski, dh-python2
       moved to dh-python package (Closes: #740161).
diff --git a/t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.debug.js b/t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.debug.js
new file mode 100644
index 0000000..a1c3620
--- /dev/null
+++ b/t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.debug.js
@@ -0,0 +1 @@
+//the source found for instance in mono
diff --git a/t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.js b/t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.js
new file mode 100644
index 0000000..d9fd3cb
--- /dev/null
+++ b/t/tests/cruft-minified-js/debian/oldfalsepositives/debug.js/test.js
@@ -0,0 +1,2 @@
+// a very long javascript yuic compressed
+

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: