Build failed in Jenkins: lintian-tests_stretch #1046

2017-08-28 Thread jenkins
See 

--
Started by upstream project "lintian-tests_buster" build number 116
originally caused by:
 Started by upstream project "lintian-tests_sid" build number 1885
 originally caused by:
  Started by an SCM change
[EnvInject] - Loading node environment variables.
Building on master in workspace 

Wiping out workspace first.
Cloning the remote Git repository
Cloning repository git://anonscm.debian.org/lintian/lintian.git
 > git init  # 
 > timeout=10
Fetching upstream changes from git://anonscm.debian.org/lintian/lintian.git
 > git --version # timeout=10
 > git fetch --tags --progress git://anonscm.debian.org/lintian/lintian.git 
 > +refs/heads/*:refs/remotes/origin/*
ERROR: Timeout after 10 minutes
ERROR: Error cloning remote repo 'origin'
hudson.plugins.git.GitException: Command "git fetch --tags --progress 
git://anonscm.debian.org/lintian/lintian.git 
+refs/heads/*:refs/remotes/origin/*" returned status code 143:
stdout: 
stderr: remote: Counting objects: 1   remote: Counting objects: 175 
  remote: Counting objects: 180   remote: Counting objects: 236 
  remote: Counting objects: 293   remote: Counting objects: 309 
  remote: Counting objects: 453   remote: Counting objects: 9763
   remote: Counting objects: 9913   remote: Counting objects: 10009 
  remote: Counting objects: 10168   remote: Counting objects: 
10246   remote: Counting objects: 10337   remote: Counting 
objects: 10412   remote: Counting objects: 10436   remote: 
Counting objects: 10486   remote: Counting objects: 10566   
remote: Counting objects: 10632   remote: Counting objects: 10752   
remote: Counting objects: 10934   remote: Counting objects: 11179   
remote: Counting objects: 11219   remote: Counting objects: 
11664   remote: Counting objects: 11795   remote: Counting 
objects: 12070   remote: Counting objects: 12453   remote: 
Counting objects: 12705   remote: Counting objects: 12825   
remote: Counting objects: 12978   remote: Counting objects: 13057   
remote: Counting objects: 13204   remote: Counting objects: 13263   
remote: Counting objects: 13424   remote: Counting objects: 
13535   remote: Counting objects: 13580   remote: Counting 
objects: 13599   remote: Counting objects: 13694   remote: 
Counting objects: 13812   remote: Counting objects: 13870   
remote: Counting objects: 13929   remote: Counting objects: 13938   
remote: Counting objects: 14117   remote: Counting objects: 14346   
remote: Counting objects: 14370   remote: Counting objects: 
14387   remote: Counting objects: 14442   remote: Counting 
objects: 14524   remote: Counting objects: 14710   remote: 
Counting objects: 14728   remote: Counting objects: 14743   
remote: Counting objects: 14776   remote: Counting objects: 14804   
remote: Counting objects: 14820   remote: Counting objects: 14849   
remote: Counting objects: 14887   remote: Counting objects: 
14919   remote: Counting objects: 14950   remote: Counting 
objects: 15000   remote: Counting objects: 15029   remote: 
Counting objects: 15042   remote: Counting objects: 15057   
remote: Counting objects: 15071   remote: Counting objects: 15078   
remote: Counting objects: 15094   remote: Counting objects: 15097   
remote: Counting objects: 15111   remote: Counting objects: 
15128   remote: Counting objects: 15136   remote: Counting 
objects: 15151   remote: Counting objects: 15175   remote: 
Counting objects: 15206   remote: Counting objects: 15216   
remote: Counting objects: 15220   remote: Counting objects: 15233   
remote: Counting objects: 15241   remote: Counting objects: 15261   
remote: Counting objects: 15287   remote: Counting objects: 
15307   remote: Counting objects: 15339   remote: Counting 
objects: 15350   remote: Counting objects: 15354   remote: 
Counting objects: 15359   remote: Counting objects: 15363   
remote: Counting objects: 15394   remote: Counting objects: 15417   
remote: Counting objects: 15428   remote: Counting objects: 15434   
remote: Counting objects: 15462   remote: Counting objects: 
15479   remote: Counting objects: 15487   remote: 

Bug#873520: lintian: Check for bad-distribution in debian/changelog too

2017-08-28 Thread Jeremy Bicha
Source: lintian
Version: 2.5.52

I was surprised to see that pyjunitxml 0.6-1.2 was accepted into
Debian unstable with its debian/changelog still set to UNRELEASED. I
guess the uploader managed to generate a valid .changes file.

https://tracker.debian.org/news/859782

lintian already has a check for bad-distribution-in-changes-file

I request that a bad-distribution-in-debian-changelog check be added.

I think this would make a good candidate for dak auto-reject too.

Thanks,
Jeremy Bicha



[lintian] 01/01: Modify Lintian::Data's "all" to always return keys in insertion order, dropping dependency on libtie-ixhash-perl.

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 0ffea0eb355973d1be2fd788dd8e4aadeac81222
Author: Chris Lamb 
Date:   Mon Aug 28 16:09:24 2017 +0100

Modify Lintian::Data's "all" to always return keys in insertion order, 
dropping dependency on libtie-ixhash-perl.
---
 checks/fields.pm|  3 +--
 debian/changelog|  3 +++
 debian/control  |  2 --
 lib/Lintian/Data.pm | 50 +-
 4 files changed, 21 insertions(+), 37 deletions(-)

diff --git a/checks/fields.pm b/checks/fields.pm
index a9d145f..f625abc 100644
--- a/checks/fields.pm
+++ b/checks/fields.pm
@@ -101,8 +101,7 @@ my $NAME_SECTION_MAPPINGS = Lintian::Data->new(
 qr/\s*=>\s*/,
 sub {
 return {'regex' =>  qr/$_[0]/x, 'section' => $_[1]};
-},
-Lintian::Data->get_orderedtype());
+});
 
 my %VCS_EXTRACT = (
 browser => sub { return @_;},
diff --git a/debian/changelog b/debian/changelog
index 7f23d43..c7a4759 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -117,6 +117,9 @@ lintian (2.5.53) UNRELEASED; urgency=medium
   * lib/Lintian/Check.pm:
 + [CL] Stop emitting {maintainer,uploader}-address-causes-mail-loops
   for @packages.debian.org addresses.  (Closes: #871575)
+  * lib/Lintian/Data.pm:
++ [CL] Modify Lintian::Data's "all" to always return keys in insertion
+  order, dropping dependency on libtie-ixhash-perl.
 
   * helpers/coll/objdump-info-helper:
 + [CL] Apply patch from Steve Langasek to accomodate binutils 2.29
diff --git a/debian/control b/debian/control
index e2ee71e..31f5672 100644
--- a/debian/control
+++ b/debian/control
@@ -52,7 +52,6 @@ Build-Depends: aspell,
libtest-synopsis-perl,
libtext-levenshtein-perl,
libtext-template-perl,
-   libtie-ixhash-perl,
libtimedate-perl,
liburi-perl,
libyaml-libyaml-perl,
@@ -98,7 +97,6 @@ Depends: binutils,
  liblist-moreutils-perl,
  libparse-debianchangelog-perl,
  libtext-levenshtein-perl,
- libtie-ixhash-perl,
  libtimedate-perl,
  liburi-perl,
  libyaml-libyaml-perl,
diff --git a/lib/Lintian/Data.pm b/lib/Lintian/Data.pm
index 3275a78..a683172 100644
--- a/lib/Lintian/Data.pm
+++ b/lib/Lintian/Data.pm
@@ -27,8 +27,6 @@ use POSIX qw(ENOENT);
 
 use Lintian::Util qw(strip);
 
-use Tie::IxHash;
-
 our $LAZY_LOAD = 1;
 
 sub _checked_open {
@@ -62,11 +60,6 @@ sub new {
 return $self;
 }
 
-sub get_orderedtype {
-tie my %myhash, 'Tie::IxHash';
-return \%myhash;
-}
-
 # _get_data fetches an already loaded dataset by type.  It is
 # mostly useful for determining whether it makes sense to make
 # sense to be "lazy".
@@ -87,15 +80,14 @@ sub get_orderedtype {
 sub _load_data {
 my ($self, $data_spec) = @_;
 my $data_name = $data_spec->[0];
-my $data_type = $data_spec->[3] // {};
 unless (exists($data{$data_name})) {
 my $vendors = $self->_get_vendor_names;
-my $dataset = $data_type;
+my ($dataset, $keyorder) = ({}, []);
 my ($fd, $vno) = $self->_open_data_file($data_name, $vendors, 0);
-$self->_parse_file($data_name, $fd, $dataset, $data_spec,
-$vendors, $vno);
+$self->_parse_file($data_name, $fd, $dataset, $keyorder,
+$data_spec, $vendors, $vno);
 close($fd);
-$data{$data_name} = $dataset;
+$data{$data_name} = {dataset => $dataset, keyorder => $keyorder};
 }
 return $self->{'data'} = $data{$data_name};
 }
@@ -157,7 +149,8 @@ sub get_orderedtype {
 }
 
 sub _parse_file {
-my ($self, $data_name, $fd, $dataset, $data_spec, $vendors, $vno)= @_;
+my ($self, $data_name, $fd, $dataset, $keyorder, $data_spec, $vendors,$vno)
+  = @_;
 my (undef, $separator, $code) = @{$data_spec};
 my $filename = $data_name;
 $filename = $vendors->[$vno] . '/' . $data_name if $vno < scalar @$vendors;
@@ -170,12 +163,13 @@ sub _parse_file {
 if ($op eq 'delete') {
 croak "Missing key after \@delete in $filename at line $."
   unless defined $value && length $value;
+@{$keyorder} = grep { $_ ne $value } @{$keyorder};
 delete $dataset->{$value};
 } elsif ($op eq 'include-parent') {
 my ($pfd, $pvo)
   = $self->_open_data_file($data_name, $vendors,$vno +1);
-$self->_parse_file($data_name, $pfd, $dataset, $data_spec,
-$vendors, $pvo);
+$self->_parse_file($data_name, $pfd, $dataset, $keyorder,
+$data_spec, $vendors, $pvo);
 close($pfd);
 } elsif ($op eq 'if-vendor-is' or $op eq 'if-vendor-is-not') {
 my ($desired_

[lintian] branch master updated (eaaf401 -> 0ffea0e)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  eaaf401   Add missing entries in Test-For fields to make 
development/testing less error-prone.
   new  0ffea0e   Modify Lintian::Data's "all" to always return keys in 
insertion order, dropping dependency on libtie-ixhash-perl.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/fields.pm|  3 +--
 debian/changelog|  3 +++
 debian/control  |  2 --
 lib/Lintian/Data.pm | 50 +-
 4 files changed, 21 insertions(+), 37 deletions(-)

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



Bug#873434: lintian: Please check for @import ur(http://...) in /usr/share/doc

2017-08-28 Thread Bastien Roucaries


Le 27 août 2017 17:16:04 GMT+02:00, Ian Jackson 
 a écrit :
>Package: lintian
>Version: 2.5.50.3
>Severity: wishlist
>
>I have discovered various files in /usr/share/doc/*/html on my system
>which do things like this:
>  @import url(http://fonts.googleapis.com/css?family=Open+Sans);
>
>Eg,
>  /usr/share/doc/python-scipy-doc/html/_static/less/spc-bootstrap.less
>  /usr/share/doc/libfreetype6-dev/css/freetype2.css
>
>Please would you make lintian complain about @import url(), or about
>fonts.googleapis.com, or something.


Do you have piece of advices for replacing this kind of API ?

Thanks
>
>Thanks,
>Ian.
>
>-- System Information:
>Debian Release: 9.0
>  APT prefers stable-debug
>  APT policy: (500, 'stable-debug'), (500, 'stable')
>Architecture: amd64 (x86_64)
>Foreign Architectures: i386
>
>Kernel: Linux 4.9.0-3-amd64 (SMP w/4 CPU cores)
>Locale: LANG=C.UTF-8, LC_CTYPE=C.UTF-8 (charmap=UTF-8),
>LANGUAGE=C.UTF-8 (charmap=UTF-8)
>Shell: /bin/sh linked to /bin/dash
>Init: sysvinit (via /sbin/init)
>
>Versions of packages lintian depends on:
>ii  binutils  2.28-5
>ii  bzip2 1.0.6-8.1
>ii  diffstat  1.61-1+b1
>ii  file  1:5.30-1
>ii  gettext   0.19.8.1-2
>ii  intltool-debian   0.35.0+20060710.4
>ii  libapt-pkg-perl   0.1.32
>ii  libarchive-zip-perl   1.59-1
>ii  libclass-accessor-perl0.34-1
>ii  libclone-perl 0.38-2+b1
>ii  libdigest-sha-perl5.96-1+b1
>ii  libdpkg-perl  1.18.24
>ii  libemail-valid-perl   1.202-1
>ii  libfile-basedir-perl  0.07-1
>ii  libipc-run-perl   0.94-1
>ii  liblist-moreutils-perl0.416-1+b1
>ii  libparse-debianchangelog-perl 1.2.0-12
>ii  libperl5.24 [libdigest-sha-perl]  5.24.1-2
>ii  libtext-levenshtein-perl  0.13-1
>ii  libtimedate-perl  2.3000-2
>ii  liburi-perl   1.71-1
>ii  libyaml-libyaml-perl  0.63-2
>ii  man-db2.7.6.1-2
>ii  patchutils0.3.4-2
>ii  perl  5.24.1-2
>ii  t1utils   1.39-2
>ii  xz-utils  5.2.2-1.2+b1
>
>Versions of packages lintian recommends:
>ii  dpkg 1.18.24
>ii  libperlio-gzip-perl  0.19-1+b2
>ii  perl 5.24.1-2
>ii  perl-modules-5.22 [libautodie-perl]  5.22.2-1
>ii  perl-modules-5.24 [libautodie-perl]  5.24.1-2
>
>Versions of packages lintian suggests:
>pn  binutils-multiarch 
>ii  dpkg-dev   1.18.24
>ii  libhtml-parser-perl3.72-3
>ii  libtext-template-perl  1.46-1
>
>-- no debconf information

-- 
Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.



[lintian] 01/01: Add missing entries in Test-For fields to make development/testing less error-prone.

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit eaaf4017fc1c27c424efc55ecb2ec62c6ce55be0
Author: Chris Lamb 
Date:   Mon Aug 28 15:59:14 2017 +0100

Add missing entries in Test-For fields to make development/testing less 
error-prone.
---
 debian/changelog  |  5 +-
 t/tests/binaries-multiarch/desc   |  2 +
 t/tests/binaries-unsafe-open/desc |  2 +
 t/tests/cruft-general-test-suite/desc |  2 +
 t/tests/fields-vcs-fields/desc|  2 +
 t/tests/legacy-binary/desc| 90 +++
 t/tests/legacy-debconf/desc   | 47 
 t/tests/legacy-debug/desc | 30 +++
 t/tests/legacy-etcfiles/desc  | 16 ++
 t/tests/legacy-fields/desc| 17 ++
 t/tests/legacy-filenames/desc | 75 ++
 t/tests/legacy-libbaz/desc| 47 
 t/tests/legacy-maintainer-scripts/desc| 57 
 t/tests/legacy-relations/desc | 40 ++
 t/tests/legacy-scripts/desc   | 62 +
 t/tests/lintian-check-tags-file/desc  |  3 ++
 t/tests/lintian-no-override/desc  |  2 +
 t/tests/scripts-ocamlrun/desc |  2 +
 t/tests/unpack-slashes-in-field-name/desc |  3 ++
 t/tests/watch-file-bug-765995/desc|  2 +
 t/tests/watch-file-pgpmode-next/desc  |  2 +
 21 files changed, 506 insertions(+), 2 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 8d124bf..7f23d43 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -129,8 +129,9 @@ lintian (2.5.53) UNRELEASED; urgency=medium
   (Closes: #873434)
 + [CL] Add explicit test for packages including external fonts via
   the Typekit API via  HTML tags.
-  * t/tests/legacy-foo++/tags:
-+ [CL] Add missing entries in "Test-For" field.
+  * t/tests/*/desc:
++ [CL] Add missing entries in "Test-For" fields to make
+  development/testing workflow less error-prone.
 
  -- Niels Thykier   Sun, 23 Jul 2017 10:16:35 +
 
diff --git a/t/tests/binaries-multiarch/desc b/t/tests/binaries-multiarch/desc
index 471dbea..ba0ab1d 100644
--- a/t/tests/binaries-multiarch/desc
+++ b/t/tests/binaries-multiarch/desc
@@ -6,3 +6,5 @@ Options: --pedantic -I -E
 Test-Depends: debhelper (>= 9.20151004~)
 Test-Against:
  triplet-dir-and-architecture-mismatch
+Test-For:
+ pre-depends-directly-on-multiarch-support
diff --git a/t/tests/binaries-unsafe-open/desc 
b/t/tests/binaries-unsafe-open/desc
index 9abe0b3..dabbccd 100644
--- a/t/tests/binaries-unsafe-open/desc
+++ b/t/tests/binaries-unsafe-open/desc
@@ -2,3 +2,5 @@ Testname: binaries-unsafe-open
 Architecture: any
 Version: 1.0
 Description: Test for possible command execution when processing filenames
+Test-For:
+ binary-without-manpage
diff --git a/t/tests/cruft-general-test-suite/desc 
b/t/tests/cruft-general-test-suite/desc
index 47040e3..4d84c98 100644
--- a/t/tests/cruft-general-test-suite/desc
+++ b/t/tests/cruft-general-test-suite/desc
@@ -20,3 +20,5 @@ Test-Against:
  source-contains-svn-commit-file
  source-contains-svn-conflict-file
  source-contains-svn-control-dir
+Test-For:
+ debian-watch-may-check-gpg-signature
diff --git a/t/tests/fields-vcs-fields/desc b/t/tests/fields-vcs-fields/desc
index d9ee1e3..ce5c2ad 100644
--- a/t/tests/fields-vcs-fields/desc
+++ b/t/tests/fields-vcs-fields/desc
@@ -8,3 +8,5 @@ Test-Against:
  vcs-field-uses-not-recommended-uri-format
  vcs-field-uses-unknown-uri-format
  vcs-git-uses-invalid-user-uri
+Test-For:
+ vcs-field-uses-insecure-uri
diff --git a/t/tests/legacy-binary/desc b/t/tests/legacy-binary/desc
index 7db5686..6d57e26 100644
--- a/t/tests/legacy-binary/desc
+++ b/t/tests/legacy-binary/desc
@@ -3,3 +3,93 @@ Source: binary
 Sequence: 
 Version: 4-1.1
 Description: Legacy test "binary"
+Test-For:
+ ancient-standards-version
+ arch-dep-package-has-big-usr-share
+ binary-control-field-duplicates-source
+ binary-has-unneeded-section
+ binary-without-manpage
+ capitalization-error-in-description
+ changelog-file-not-compressed
+ changelog-news-debian-mismatch
+ command-in-menu-file-and-desktop-file
+ control-file-is-empty
+ debian-changelog-file-missing-or-wrong-name
+ debian-copyright-file-uses-obsolete-national-encoding
+ debian-news-entry-has-strange-distribution
+ debian-news-file-uses-obsolete-national-encoding
+ debian-rules-ignores-make-clean-error
+ debian-rules-missing-recommended-target
+ debian-rules-missing-required-target
+ debian-rules-should-not-use-pwd
+ depends-on-metapackage
+ description-contains-homepage
+ desktop-command-not-in-package
+ desktop-contains-deprecated-header
+ desktop-entry-contains-encoding-key
+ desktop-entry-contains-unknown-key
+ desktop-entry-file-has-crs
+ desktop-entry-invalid-category
+ desktop-entry-lacks-icon-entry
+ desktop-entry-lacks

[lintian] branch master updated (a5146df -> eaaf401)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  a5146df   Upgrade latest-debian-changelog-entry-without-new-date 
from a warning to an error. (Closes: #873490)
   new  eaaf401   Add missing entries in Test-For fields to make 
development/testing less error-prone.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 debian/changelog  |  5 +-
 t/tests/binaries-multiarch/desc   |  2 +
 t/tests/binaries-unsafe-open/desc |  2 +
 t/tests/cruft-general-test-suite/desc |  2 +
 t/tests/fields-vcs-fields/desc|  2 +
 t/tests/legacy-binary/desc| 90 +++
 t/tests/legacy-debconf/desc   | 47 
 t/tests/legacy-debug/desc | 30 +++
 t/tests/legacy-etcfiles/desc  | 16 ++
 t/tests/legacy-fields/desc| 17 ++
 t/tests/legacy-filenames/desc | 75 ++
 t/tests/legacy-libbaz/desc| 47 
 t/tests/legacy-maintainer-scripts/desc| 57 
 t/tests/legacy-relations/desc | 40 ++
 t/tests/legacy-scripts/desc   | 62 +
 t/tests/lintian-check-tags-file/desc  |  3 ++
 t/tests/lintian-no-override/desc  |  2 +
 t/tests/scripts-ocamlrun/desc |  2 +
 t/tests/unpack-slashes-in-field-name/desc |  3 ++
 t/tests/watch-file-bug-765995/desc|  2 +
 t/tests/watch-file-pgpmode-next/desc  |  2 +
 21 files changed, 506 insertions(+), 2 deletions(-)

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



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Chris Lamb
Ian Jackson wrote:

> I guess this will become non-experimental in due course ?

Hm, I hope so! Bastien, as you did the initial implementation of 
his, I'm sure you can speak to this better than me. 

(If you think it's ready but don't have time at the moment please
open an another issue — or clone this one — so we don't lose it...)


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Chris Lamb
Bastien,

> If we could add a tag for this it is quite easy
> see data/files/privacy-breaker-websites

I'm sorry but don't understand you, sorry; there is already a check
for this as I explained in my initial reply.

> (we could also add more js library in order to help ftpmaster)

Sure.. If you know of any, please open another issue; I'm not keeping any
"under my hat" :)


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Re: debian/upstream/signing-key.asc in policy 4.1.0

2017-08-28 Thread Osamu Aoki
Hi,

On Sun, Aug 27, 2017 at 08:51:49PM -0300, Henrique de Moraes Holschuh wrote:
> On Wed, 23 Aug 2017, Russ Allbery wrote:
> > Note that this Policy language is carefully written to make it perfectly
> > fine for uscan to support all the things it currently supports, since it
> > only talks about what Policy recommends the maintainer does.  So don't
> > feel any obligation to change what uscan is doing on Policy's account
> > here.
> 
> Actually, the text in 4.1.0.0 might be doing too much.  It reads:
> 
> "If the upstream maintainer of the software provides OpenPGP signatures
> for new releases, including the information required for "uscan" to
> verify signatures for new upstream releases is also recommended. To do
> this, use the "pgpsigurlmangle" option in "debian/watch" to specify
> the location of the upstream signature, and include the key or keys
> used to sign upstream releases in the Debian source package as
> "debian/upstream/signing-key.asc".
> 
> IMO, it should either not be mandating uscan internals, or it should be

In principle, you comment is a very reasonable one.

> very clear about the exact subset of stuff we can use in debian/watch
> (version, etc).  For example, I'd rather use opt="..., pgpmode=auto,..."
> instead of explicitly hardcoding a "pgpsigurlmangle".

The new pgpmode=auto and pgpmode=previous have bugs and fail to function
smoothly ---  #873289 #852537  Excuse me for these bugs.  The fixes have
been committed to git.  I am hoping the next upload of devscripts (and
its backport) will fix them.  So "pgpsigurlmangle" is the only good way
at this moment.
 
> IMHO, just drop everything from "To do this..." to the end of that
> paragraph entirely.  HOW one gets "uscan" to fetch and check upstream
> signatures is a job for the uscan(1) manpage.  Alternatively, just
> mention "debian/watch", and to refer to the uscan documentation in
> package "devscripts".

Once pgpmode=auto becomes noise free, this should be the preferred
choice.  It will be nice to address #833012, too, using s/\?/.asc?/ etc.
to make it really default one.

So for now, the policy text is better for me.

> OTOH, if we really need to mandate a specific level of debian/watch
> support, the current text in policy needs work: it doesn't even tell me
> whether I can use version=3 (supported in oldstable), or version=4
> (supported in oldstable-backports and stable), for example...

The uscan version=3/version=4 difference is not much about enhanced
mangling rules.  It's about how uupdate is invoked and how uupdate
creates the updated source tree.  version=4 uses dpkg-source as back-end
and capable of generating multi-upstream tarball.

If you use new uscan, even with a watch file marked as version=3, it has
access to the enhanced mangling rules.

Osamu



Bug#873458: lintian: Please add autopkgtest-pkg-elpa as a valid value for Testsuite

2017-08-28 Thread Martín Ferrari
On 28/08/17 11:07, Chris Lamb wrote:

>> Can you add pkg-go to the list of valid testsuites?
> 
> This was fixed by Niels in:
> 
>   
> https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=2a279596289a9e7b4f31fc5aba48f2c6c9a18ebc
> 
> I've added the sole remaining value ("autopkgtest-pkg-elpa") in:
> 
>   
> https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=d5f0c0115ef46eb98d1b68f7113aa63d93f72211
> 
> … and renaming this bug to match :)

Cool, we just need to wait for the next release then. Thanks!


-- 
Martín Ferrari (Tincho)



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Ian Jackson
Chris Lamb writes ("Re: lintian: Please check for @import url(http://...) in 
/usr/share/doc"):
> tags 873434 + pending
> retitle 873434 lintian: Please check for @import url(http://...) in 
> /usr/share/doc
> thanks
...
> Thanks for the report. So, this is actually already being checked for
> but as it's an "experimental" tag you need to pass the --display-experimental
> flag. It will then appear as:

Ah.  I guess this will become non-experimental in due course ?

> However, I've taken a moment to add an explicit regression test for
> Google Fonts:

Thanks.

Regards,
Ian.



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Bastien ROUCARIES
BTW acording to https://github.com/google/fonts we could self host and
create an offline version
https://github.com/majodev/google-webfonts-helper

Something for the fonts team ?

On Mon, Aug 28, 2017 at 3:24 PM, Bastien ROUCARIES
 wrote:
> If we could add a tag for this it is quite easy
> see
> data/files/privacy-breaker-websites
>
> (we could also add more js library in order to help ftpmaster)
>
> On Mon, Aug 28, 2017 at 3:18 PM, Chris Lamb  wrote:
>> Bastien,
>>
>>> Do we have alternative packaged for debian ?
>>
>> Given that most usage of Google Fonts is simply to provide "nicer"
>> versions of built-in fonts (and most CSS typically labels a set of
>> fallback font families) simply removing the offending import line
>> will only result in a slightly uglier font.
>>
>> Thus ensuring we have an alternative is not necessary in 99% of cases.
>>
>>> My perl capacity are too low but I will like to become dh_fixprivacy
>>
>> Happy hacking ;-)
>>
>>
>> Best wishes,
>>
>> --
>>   ,''`.
>>  : :'  : Chris Lamb
>>  `. `'`  la...@debian.org / chris-lamb.co.uk
>>`-



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Bastien ROUCARIES
If we could add a tag for this it is quite easy
see
data/files/privacy-breaker-websites

(we could also add more js library in order to help ftpmaster)

On Mon, Aug 28, 2017 at 3:18 PM, Chris Lamb  wrote:
> Bastien,
>
>> Do we have alternative packaged for debian ?
>
> Given that most usage of Google Fonts is simply to provide "nicer"
> versions of built-in fonts (and most CSS typically labels a set of
> fallback font families) simply removing the offending import line
> will only result in a slightly uglier font.
>
> Thus ensuring we have an alternative is not necessary in 99% of cases.
>
>> My perl capacity are too low but I will like to become dh_fixprivacy
>
> Happy hacking ;-)
>
>
> Best wishes,
>
> --
>   ,''`.
>  : :'  : Chris Lamb
>  `. `'`  la...@debian.org / chris-lamb.co.uk
>`-



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Chris Lamb
Bastien,

> Do we have alternative packaged for debian ?

Given that most usage of Google Fonts is simply to provide "nicer"
versions of built-in fonts (and most CSS typically labels a set of
fallback font families) simply removing the offending import line
will only result in a slightly uglier font.

Thus ensuring we have an alternative is not necessary in 99% of cases.

> My perl capacity are too low but I will like to become dh_fixprivacy

Happy hacking ;-)


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#873490: latest-debian-changelog-entry-without-new-date should become an error

2017-08-28 Thread Chris Lamb
tags 873490 + pending
thanks

Fixed in Git:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=a5146df5a79f449e9ae9c44554968418aa1f7900


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Processed: Re: latest-debian-changelog-entry-without-new-date should become an error

2017-08-28 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 873490 + pending
Bug #873490 [lintian] latest-debian-changelog-entry-without-new-date should 
become an error
Added tag(s) pending.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
873490: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=873490
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Bastien ROUCARIES
Do we have alternative packaged for debian ?

BTW if you could get a glimpse at src:imagemagick I have a script that
could help for this kind of problems that use xlst...

My perl capacity are too low but I will like to become dh_fixprivacy

On Mon, Aug 28, 2017 at 10:59 AM, Chris Lamb  wrote:
> tags 873434 + pending
> retitle 873434 lintian: Please check for @import url(http://...) in 
> /usr/share/doc
> thanks
>
> Hi Ian,
>
> Thanks for the report. So, this is actually already being checked for
> but as it's an "experimental" tag you need to pass the --display-experimental
> flag. It will then appear as:
>
>   X: src: binary path/to/foo.html 
> (http://fonts.googleapis.com/css?family=open+sans)
>
> However, I've taken a moment to add an explicit regression test for
> Google Fonts:
>
>   
> https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=636a4cebc93f5321e482563ec0dd00a40fa01a0f
>
> .. as well as one for Typekit (same deal, different mechanism):
>
>   
> https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=0d68e48e6424234f06cfb0873bc746e243577be3
>
>
> Regards,
>
> --
>   ,''`.
>  : :'  : Chris Lamb
>  `. `'`  la...@debian.org / chris-lamb.co.uk
>`-
>



[lintian] 02/02: Upgrade latest-debian-changelog-entry-without-new-date from a warning to an error. (Closes: #873490)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit a5146df5a79f449e9ae9c44554968418aa1f7900
Author: Chris Lamb 
Date:   Mon Aug 28 14:10:47 2017 +0100

Upgrade latest-debian-changelog-entry-without-new-date from a warning to an 
error. (Closes: #873490)
---
 checks/changelog-file.desc  | 2 +-
 debian/changelog| 2 ++
 t/tests/changelog-file-general/tags | 2 +-
 t/tests/legacy-foo++/tags   | 2 +-
 4 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/checks/changelog-file.desc b/checks/changelog-file.desc
index 7eb17f3..df848ac 100644
--- a/checks/changelog-file.desc
+++ b/checks/changelog-file.desc
@@ -196,7 +196,7 @@ Info: The NEWS.Debian file must be valid UTF-8, an encoding 
of the Unicode
   $ mv NEWS.Debian.new NEWS.Debian
 
 Tag: latest-debian-changelog-entry-without-new-date
-Severity: normal
+Severity: important
 Certainty: certain
 Info: The latest Debian changelog entry has either the same or even an
  older date as the entry before.
diff --git a/debian/changelog b/debian/changelog
index 1369bdc..8d124bf 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -170,6 +170,8 @@ lintian (2.5.52) unstable; urgency=medium
 + [CL] Expand the rationale for the
   latest-debian-changelog-entry-without-new-date tag to mention
   possible implications for SOURCE_DATE_EPOCH/reproducible builds.
++ [CL] Upgrade latest-debian-changelog-entry-without-new-date from
+  a warning to an error.  (Closes: #873490)
   * checks/control-file.pm:
 + [NT] Remove check for missing versioned build-depends for dpkg
   and debhlper when using Build-Profiles.  The necessary versions
diff --git a/t/tests/changelog-file-general/tags 
b/t/tests/changelog-file-general/tags
index c5a8859..3b0fb55 100644
--- a/t/tests/changelog-file-general/tags
+++ b/t/tests/changelog-file-general/tags
@@ -1,4 +1,5 @@
 E: changelog-file-general: 
debian-changelog-file-contains-invalid-email-address unknown@unknown
+E: changelog-file-general: latest-debian-changelog-entry-without-new-date
 E: changelog-file-general: possible-missing-colon-in-closes Closes #55
 W: changelog-file-general: changelog-not-compressed-with-max-compression 
changelog.Debian.gz
 W: changelog-file-general: changelog-references-temp-security-identifier 
TEMP-1234567-abcdef
@@ -6,7 +7,6 @@ W: changelog-file-general: 
debian-changelog-file-contains-obsolete-user-emacs-se
 W: changelog-file-general: debian-changelog-line-too-long line 15
 W: changelog-file-general: debian-changelog-line-too-long line 8
 W: changelog-file-general: improbable-bug-number-in-closes 1234
-W: changelog-file-general: latest-debian-changelog-entry-without-new-date
 W: changelog-file-general: latest-debian-changelog-entry-without-new-version
 W: changelog-file-general: misspelled-closes-bug #66
 X: changelog-file-general: bad-intended-distribution intended to experimental 
but uploaded to unstable
diff --git a/t/tests/legacy-foo++/tags b/t/tests/legacy-foo++/tags
index 88a2fd7..c3cb807 100644
--- a/t/tests/legacy-foo++/tags
+++ b/t/tests/legacy-foo++/tags
@@ -11,6 +11,7 @@ E: foo++ source: wrong-debian-qa-address-set-as-maintainer 
Lintian Maintainer 
 E: foo++: debian-changelog-file-contains-invalid-email-address he@unknown
 E: foo++: debian-changelog-file-uses-obsolete-national-encoding at line 11
+E: foo++: latest-debian-changelog-entry-without-new-date
 E: foo++: no-copyright-file
 E: foo++: wrong-debian-qa-address-set-as-maintainer Lintian Maintainer 

 I: foo++ source: duplicate-short-description foo++ foo++-helper
@@ -23,5 +24,4 @@ W: foo++ source: uploader-address-looks-weird Jeroen van 
Wolffelaar

[lintian] branch master updated (d5f0c01 -> a5146df)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  d5f0c01   Recognise "autopkgtest-pkg-ela" as a valid test suite. 
(Closes: #873458)
   new  e83b5ebAdd missing entries in "Test-For" field.
   new  a5146df   Upgrade latest-debian-changelog-entry-without-new-date 
from a warning to an error. (Closes: #873490)

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/changelog-file.desc  |  2 +-
 debian/changelog|  4 
 t/tests/changelog-file-general/tags |  2 +-
 t/tests/legacy-foo++/desc   | 23 +++
 t/tests/legacy-foo++/tags   |  2 +-
 5 files changed, 30 insertions(+), 3 deletions(-)

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



[lintian] 01/02: Add missing entries in "Test-For" field.

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit e83b5eb8b8965f9803e9f826348eff08a85e2284
Author: Chris Lamb 
Date:   Mon Aug 28 14:14:04 2017 +0100

 Add missing entries in "Test-For" field.
---
 debian/changelog  |  2 ++
 t/tests/legacy-foo++/desc | 23 +++
 2 files changed, 25 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 6e967bb..1369bdc 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -129,6 +129,8 @@ lintian (2.5.53) UNRELEASED; urgency=medium
   (Closes: #873434)
 + [CL] Add explicit test for packages including external fonts via
   the Typekit API via  HTML tags.
+  * t/tests/legacy-foo++/tags:
++ [CL] Add missing entries in "Test-For" field.
 
  -- Niels Thykier   Sun, 23 Jul 2017 10:16:35 +
 
diff --git a/t/tests/legacy-foo++/desc b/t/tests/legacy-foo++/desc
index d5b04e6..9b5d4ef 100644
--- a/t/tests/legacy-foo++/desc
+++ b/t/tests/legacy-foo++/desc
@@ -3,3 +3,26 @@ Source: foo++
 Sequence: 
 Version: 5
 Description: Legacy test "foo++"
+Test-For:
+ ancient-standards-version
+ build-info-in-binary-control-file-section
+ changed-by-address-malformed
+ debian-changelog-file-contains-invalid-email-address
+ debian-changelog-file-uses-obsolete-national-encoding
+ debian-changelog-has-wrong-day-of-week
+ debian-control-file-uses-obsolete-national-encoding
+ debian-watch-file-in-native-package
+ debian-watch-file-unknown-version
+ dm-upload-allowed-is-obsolete
+ duplicate-short-description
+ empty-binary-package
+ latest-debian-changelog-entry-without-new-date
+ malformed-dm-upload-allowed
+ no-copyright-file
+ no-md5sums-control-file
+ readme-debian-contains-invalid-email-address
+ uploader-address-is-on-localhost
+ uploader-address-looks-weird
+ uploader-address-malformed
+ uploader-name-missing
+ wrong-debian-qa-address-set-as-maintainer

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



Bug#873490: latest-debian-changelog-entry-without-new-date should become an error

2017-08-28 Thread Adrian Bunk
Package: lintian
Version: 2.5.52
Severity: normal

See #873489 - only a warning for something suggested for
autoreject sounds wrong.



Processed: Re: lintian: Please add autopkgtest-pkg-elpa as a valid value for Testsuite

2017-08-28 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 873458 + pending
Bug #873458 [lintian] lintian: Please add autopkgtest-pkg-go as valid value for 
Testsuite
Added tag(s) pending.
> retitle 873458 lintian: Please add autopkgtest-pkg-elpa as a valid value for 
> Testsuite
Bug #873458 [lintian] lintian: Please add autopkgtest-pkg-go as valid value for 
Testsuite
Changed Bug title to 'lintian: Please add autopkgtest-pkg-elpa as a valid value 
for Testsuite' from 'lintian: Please add autopkgtest-pkg-go as valid value for 
Testsuite'.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
873458: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=873458
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Processed: your mail

2017-08-28 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 873395 + wontfix
Bug #873395 [lintian] lintian: check for non-free dependencies in packages 
targetting main
Added tag(s) wontfix.
> retitle 873395 lintian: check for non-free dependencies in packages targeting 
> main
Bug #873395 [lintian] lintian: check for non-free dependencies in packages 
targetting main
Changed Bug title to 'lintian: check for non-free dependencies in packages 
targeting main' from 'lintian: check for non-free dependencies in packages 
targetting main'.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
873395: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=873395
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#873458: lintian: Please add autopkgtest-pkg-elpa as a valid value for Testsuite

2017-08-28 Thread Chris Lamb
tags 873458 + pending
retitle 873458 lintian: Please add autopkgtest-pkg-elpa as a valid value for 
Testsuite
thanks

Hi Tincho,

> Can you add pkg-go to the list of valid testsuites?

This was fixed by Niels in:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=2a279596289a9e7b4f31fc5aba48f2c6c9a18ebc

I've added the sole remaining value ("autopkgtest-pkg-elpa") in:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=d5f0c0115ef46eb98d1b68f7113aa63d93f72211

… and renaming this bug to match :)


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



[lintian] 01/01: Recognise "autopkgtest-pkg-ela" as a valid test suite. (Closes: #873458)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit d5f0c0115ef46eb98d1b68f7113aa63d93f72211
Author: Chris Lamb 
Date:   Mon Aug 28 10:05:55 2017 +0100

Recognise "autopkgtest-pkg-ela" as a valid test suite. (Closes: #873458)
---
 checks/testsuite.pm | 1 +
 debian/changelog| 2 ++
 2 files changed, 3 insertions(+)

diff --git a/checks/testsuite.pm b/checks/testsuite.pm
index ba24548..d593df0 100644
--- a/checks/testsuite.pm
+++ b/checks/testsuite.pm
@@ -59,6 +59,7 @@ my %KNOWN_RESTRICTIONS = map { $_ => 1 } qw(
 my %KNOWN_TESTSUITES = map { $_ => 1 } qw(
   autopkgtest
   autopkgtest-pkg-dkms
+  autopkgtest-pkg-elpa
   autopkgtest-pkg-go
   autopkgtest-pkg-nodejs
   autopkgtest-pkg-perl
diff --git a/debian/changelog b/debian/changelog
index 9a35397..6e967bb 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -86,6 +86,8 @@ lintian (2.5.53) UNRELEASED; urgency=medium
 + [CL] Warn if we see an unnecessary "Testsuite: autopkgtest" header
   in debian/control.
 + [NT] Recognise "autopkgtest-pkg-go" as a valid test suite.
++ [CL] Recognise "autopkgtest-pkg-elpa" as a valid test suite.
+  (Closes: #873458)
 
   * data/binaries/embedded-libs:
 + [RG] Detect embedded copies of heimdal, libgxps, libquicktime,

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



[lintian] branch master updated (0d68e48e -> d5f0c01)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  0d68e48e  [CL] Add explicit test for packages including external 
fonts via the Typekit API via  HTML tags.
   new  d5f0c01   Recognise "autopkgtest-pkg-ela" as a valid test suite. 
(Closes: #873458)

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/testsuite.pm | 1 +
 debian/changelog| 2 ++
 2 files changed, 3 insertions(+)

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



Processed: Re: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 873434 + pending
Bug #873434 [lintian] lintian: Please check for @import ur(http://...) in 
/usr/share/doc
Added tag(s) pending.
> retitle 873434 lintian: Please check for @import url(http://...) in 
> /usr/share/doc
Bug #873434 [lintian] lintian: Please check for @import ur(http://...) in 
/usr/share/doc
Changed Bug title to 'lintian: Please check for @import url(http://...) in 
/usr/share/doc' from 'lintian: Please check for @import ur(http://...) in 
/usr/share/doc'.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
873434: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=873434
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#873434: lintian: Please check for @import url(http://...) in /usr/share/doc

2017-08-28 Thread Chris Lamb
tags 873434 + pending
retitle 873434 lintian: Please check for @import url(http://...) in 
/usr/share/doc
thanks

Hi Ian,

Thanks for the report. So, this is actually already being checked for
but as it's an "experimental" tag you need to pass the --display-experimental
flag. It will then appear as:

  X: src: binary path/to/foo.html 
(http://fonts.googleapis.com/css?family=open+sans)

However, I've taken a moment to add an explicit regression test for
Google Fonts:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=636a4cebc93f5321e482563ec0dd00a40fa01a0f

.. as well as one for Typekit (same deal, different mechanism):

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=0d68e48e6424234f06cfb0873bc746e243577be3


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



[lintian] 02/02: [CL] Add explicit test for packages including external fonts via the Typekit API via HTML tags.

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 0d68e48e6424234f06cfb0873bc746e243577be3
Author: Chris Lamb 
Date:   Mon Aug 28 09:49:16 2017 +0100

[CL] Add explicit test for packages including external fonts via the 
Typekit API via  HTML tags.
---
 debian/changelog   | 2 ++
 .../debian/src/privacy-breach-generic/typekit.html | 7 +++
 t/tests/files-privacybreach/tags   | 1 +
 3 files changed, 10 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 0f7a058..9a35397 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -125,6 +125,8 @@ lintian (2.5.53) UNRELEASED; urgency=medium
 + [CL] Add explicit test for packages including external fonts via
   the Google Font API. Thanks to Ian Jackson for the report.
   (Closes: #873434)
++ [CL] Add explicit test for packages including external fonts via
+  the Typekit API via  HTML tags.
 
  -- Niels Thykier   Sun, 23 Jul 2017 10:16:35 +
 
diff --git 
a/t/tests/files-privacybreach/debian/src/privacy-breach-generic/typekit.html 
b/t/tests/files-privacybreach/debian/src/privacy-breach-generic/typekit.html
new file mode 100644
index 000..713cfd5
--- /dev/null
+++ b/t/tests/files-privacybreach/debian/src/privacy-breach-generic/typekit.html
@@ -0,0 +1,7 @@
+
+
+
+
+try{Typekit.load();}catch(e){}
+
+
diff --git a/t/tests/files-privacybreach/tags b/t/tests/files-privacybreach/tags
index 0ff1ea9..0f3adbf 100644
--- a/t/tests/files-privacybreach/tags
+++ b/t/tests/files-privacybreach/tags
@@ -73,6 +73,7 @@ X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/pr
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/object.html 
(https://1984.os/hellotrackme.swf)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/source.html 
(https://1984.os/tracking.mp3)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/track.html 
(http://1984.os/notracking.vtt)
+X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/typekit.html 
(//use.typekit.net/uniquecodehere.js)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/video.html 
(ftp://1984.os/tuxistrackingme.ogg)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/videoposter.html 
(ftp://1984.os/tuxistrackingme.ogg)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-google-plus/asyncload.html 
(http://www.example.com)

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



[lintian] 01/02: Add explicit test for packages including external fonts via the Google Font API. Thanks to Ian Jackson for the report. (Closes: #873434)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 636a4cebc93f5321e482563ec0dd00a40fa01a0f
Author: Chris Lamb 
Date:   Mon Aug 28 09:47:41 2017 +0100

Add explicit test for packages including external fonts via the Google Font 
API. Thanks to Ian Jackson for the report. (Closes: #873434)
---
 debian/changelog  | 5 +
 .../debian/src/privacy-breach-generic/googlefontapi.html  | 8 
 t/tests/files-privacybreach/tags  | 1 +
 3 files changed, 14 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 8d141bb..0f7a058 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -121,6 +121,11 @@ lintian (2.5.53) UNRELEASED; urgency=medium
   outputting symbols in a different format on ppc64el.
   (Closes: #869750)
 
+  * t/tests/files-privacybreach/*:
++ [CL] Add explicit test for packages including external fonts via
+  the Google Font API. Thanks to Ian Jackson for the report.
+  (Closes: #873434)
+
  -- Niels Thykier   Sun, 23 Jul 2017 10:16:35 +
 
 lintian (2.5.52) unstable; urgency=medium
diff --git 
a/t/tests/files-privacybreach/debian/src/privacy-breach-generic/googlefontapi.html
 
b/t/tests/files-privacybreach/debian/src/privacy-breach-generic/googlefontapi.html
new file mode 100644
index 000..bd64037
--- /dev/null
+++ 
b/t/tests/files-privacybreach/debian/src/privacy-breach-generic/googlefontapi.html
@@ -0,0 +1,8 @@
+
+
+
+
+   @import url("http://fonts.googleapis.com/css?family=Open+Sans";);
+
+
+
diff --git a/t/tests/files-privacybreach/tags b/t/tests/files-privacybreach/tags
index 1a8c601..0ff1ea9 100644
--- a/t/tests/files-privacybreach/tags
+++ b/t/tests/files-privacybreach/tags
@@ -64,6 +64,7 @@ X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/pr
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/externalimg.xml 
(http://1984.ow/bigbrotheriswatchingyou.png)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/frame.html 
(http://1984.os/trackme_frame_c.htm)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/genericwebsite.html 
(http://www.example.com/trackme.js)
+X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/googlefontapi.html 
(http://fonts.googleapis.com/css?family=open+sans)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/iframe.html 
(http://1984.ow/bigbrotheriswatchingyou.html)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/import.css 
(http://1984.wo/importtracking.css)
 X: files-privacybreach: privacy-breach-generic 
usr/share/files-privacy-breach/privacy-breach-generic/importcss.html 
(http://trackme.css/track.css)

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



[lintian] branch master updated (af5fff4 -> 0d68e48e)

2017-08-28 Thread Chris Lamb
This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  af5fff4   spelling: Add another correction
   new  636a4ce   Add explicit test for packages including external fonts 
via the Google Font API. Thanks to Ian Jackson for the report. (Closes: #873434)
   new  0d68e48e  [CL] Add explicit test for packages including external 
fonts via the Typekit API via  HTML tags.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 debian/changelog  | 7 +++
 .../debian/src/privacy-breach-generic/googlefontapi.html  | 8 
 .../debian/src/privacy-breach-generic/typekit.html| 7 +++
 t/tests/files-privacybreach/tags  | 2 ++
 4 files changed, 24 insertions(+)
 create mode 100644 
t/tests/files-privacybreach/debian/src/privacy-breach-generic/googlefontapi.html
 create mode 100644 
t/tests/files-privacybreach/debian/src/privacy-breach-generic/typekit.html

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