[lintian] branch master updated (d951d71 -> a1775d5)

2018-03-07 Thread Paul Wise
This is an automated email from the git hooks/post-receive script.

pabs pushed a change to branch master
in repository lintian.

  from  d951d71   Don't emit orig-tarball-missing-upstream-signature when 
the package provides a "foo.tar.asc" for a "foo.tar.gz". We previously only 
checked for a "foo.tar.gz.asc". Thanks to Uwe Kleine-König for the report. 
(Closes: #892255)
   new  a1775d5   spelling: Add another correction

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:
 data/spelling/corrections | 1 +
 1 file changed, 1 insertion(+)

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



[lintian] 01/01: spelling: Add another correction

2018-03-07 Thread Paul Wise
This is an automated email from the git hooks/post-receive script.

pabs pushed a commit to branch master
in repository lintian.

commit a1775d579605d8849d45c8c127c345552b0b4682
Author: Paul Wise 
Date:   Thu Mar 8 14:30:41 2018 +0800

spelling: Add another correction
---
 data/spelling/corrections | 1 +
 1 file changed, 1 insertion(+)

diff --git a/data/spelling/corrections b/data/spelling/corrections
index 6781039..ad207e9 100644
--- a/data/spelling/corrections
+++ b/data/spelling/corrections
@@ -1690,6 +1690,7 @@ featue||feature
 featues||features
 feauture||feature
 feautures||features
+feeback||feedback
 feeded||fed
 fetaure||feature
 fetaures||features

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



Processed: Re: lintian: orig-tarball-missing-upstream-signature with signed .tar

2018-03-07 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 892255 + pending
Bug #892255 [lintian] lintian: orig-tarball-missing-upstream-signature with 
signed .tar
Added tag(s) pending.
> thanks
Stopping processing here.

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



Bug#892255: lintian: orig-tarball-missing-upstream-signature with signed .tar

2018-03-07 Thread Chris Lamb
tags 892255 + pending
thanks

Fixed in Git, pending upload:

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


Regards,

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



[lintian] 01/01: Don't emit orig-tarball-missing-upstream-signature when the package provides a "foo.tar.asc" for a "foo.tar.gz". We previously only checked for a "foo.tar.gz.asc". Thanks to Uwe Klei

2018-03-07 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 d951d71b164f99c287c4e244eaa15f306e7cb703
Author: Chris Lamb 
Date:   Wed Mar 7 22:08:24 2018 -0800

Don't emit orig-tarball-missing-upstream-signature when the package 
provides a "foo.tar.asc" for a "foo.tar.gz". We previously only checked for a 
"foo.tar.gz.asc". Thanks to Uwe Kleine-König for the report. (Closes: #892255)
---
 checks/changes-file.pm   | 6 --
 debian/changelog | 5 +
 t/tests/changes-upstream-signature-missing/pre_build | 4 
 3 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/checks/changes-file.pm b/checks/changes-file.pm
index 3256ccb..6ca14e4 100644
--- a/checks/changes-file.pm
+++ b/checks/changes-file.pm
@@ -23,6 +23,8 @@ use strict;
 use warnings;
 use autodie;
 
+use List::MoreUtils qw(none);
+
 use Lintian::Tags qw(tag);
 use Lintian::Check qw(check_maintainer);
 use Lintian::Data;
@@ -198,11 +200,11 @@ sub run {
 # Ensure all orig tarballs have a signature if we have an upstream
 # signature.
 if (   $has_signing_key
-&& $file =~ m/\.orig(-[A-Za-z\d-]+)?\.tar\./
+&& $file =~ m/(^.*\.orig(?:-[A-Za-z\d-]+)?\.tar)\./
 && $file !~ m/\.asc$/
 && !$info->repacked) {
 tag 'orig-tarball-missing-upstream-signature', $file
-  unless exists $files->{"$file.asc"};
+  if none { exists $files->{"$_.asc"} } ($file, $1);
 }
 
 # check section
diff --git a/debian/changelog b/debian/changelog
index 8b74ebd..ff109d3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,11 @@ lintian (2.5.79) UNRELEASED; urgency=medium
 
   XXX: generate tag summary
 
+  * checks/changes-file.pm:
++ [CL] Don't emit orig-tarball-missing-upstream-signature when the
+  package provides a "foo.tar.asc" for a "foo.tar.gz".  We previously
+  only checked for a "foo.tar.gz.asc".  Thanks to Uwe Kleine-König for
+  the report.  (Closes: #892255)
   * checks/changelog-file.pm:
 + [CL] Ignore entries that end with ":" to avoid false-positives in the
   debian-changelog-line-too-short.  Thanks to Mattia Rizollo for the
diff --git a/t/tests/changes-upstream-signature-missing/pre_build 
b/t/tests/changes-upstream-signature-missing/pre_build
index 0afee4d..2d5a158 100755
--- a/t/tests/changes-upstream-signature-missing/pre_build
+++ b/t/tests/changes-upstream-signature-missing/pre_build
@@ -12,3 +12,7 @@ cp ${DIR}/../${NAME}_${VERSION}.orig.tar.gz 
${DIR}/../${NAME}_${VERSION}.orig-co
 # Don't emit if we have a signature
 cp ${DIR}/../${NAME}_${VERSION}.orig.tar.gz 
${DIR}/../${NAME}_${VERSION}.orig-signed.tar.gz
 touch ${DIR}/../${NAME}_${VERSION}.orig-signed.tar.gz.asc
+
+# Don't emit if we have .tar.asc (NB. not a .tar.gz.asc)
+cp ${DIR}/../${NAME}_${VERSION}.orig.tar.gz 
${DIR}/../${NAME}_${VERSION}.orig-noext.tar.gz
+touch ${DIR}/../${NAME}_${VERSION}.orig-noext.tar.gz.asc

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



[lintian] branch master updated (742199a -> d951d71)

2018-03-07 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  742199a   spelling: Add another correction
   new  d951d71   Don't emit orig-tarball-missing-upstream-signature when 
the package provides a "foo.tar.asc" for a "foo.tar.gz". We previously only 
checked for a "foo.tar.gz.asc". Thanks to Uwe Kleine-König for the report. 
(Closes: #892255)

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/changes-file.pm   | 6 --
 debian/changelog | 5 +
 t/tests/changes-upstream-signature-missing/pre_build | 4 
 3 files changed, 13 insertions(+), 2 deletions(-)

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



[lintian] branch master updated (fb18b04 -> 742199a)

2018-03-07 Thread Paul Wise
This is an automated email from the git hooks/post-receive script.

pabs pushed a change to branch master
in repository lintian.

  from  fb18b04   checks/cruft.pm: Use \Q & \E to "quotemeta" the embedded 
variable in regular expression. Thanks, Niels!
   new  742199a   spelling: Add another correction

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:
 data/spelling/corrections | 1 +
 1 file changed, 1 insertion(+)

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



[lintian] 01/01: spelling: Add another correction

2018-03-07 Thread Paul Wise
This is an automated email from the git hooks/post-receive script.

pabs pushed a commit to branch master
in repository lintian.

commit 742199a303ce2d2c00021429b7fc99be5af7c3fb
Author: Paul Wise 
Date:   Thu Mar 8 11:30:13 2018 +0800

spelling: Add another correction
---
 data/spelling/corrections | 1 +
 1 file changed, 1 insertion(+)

diff --git a/data/spelling/corrections b/data/spelling/corrections
index b1dbda5..6781039 100644
--- a/data/spelling/corrections
+++ b/data/spelling/corrections
@@ -3812,6 +3812,7 @@ sythesis||synthesis
 tage||stage
 taht||that
 targetted||targeted
+taylored||tailored
 teached||taught
 technlogy||technology
 teh||the

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



Bug#892304: lintian: Warn about "old" X-Python3-Version fields?

2018-03-07 Thread Chris Lamb
Package: lintian
Version: 2.5.77
Severity: wishlist

Hi,

Should we warn about "old" X-Python3-Version fields? For example, I
just saw a new package from a sponsee with:

  X-Python3-Version: >= 3.2

This seems a little silly given that 3.2 is only in wheezy, jessie has
3.4 and stretch has 3.5.

If the idea is fundamentally sound, we could even avoid most of the
"What versions should we I: or P: at..." bikeshedding by having an
"ancient" and "old" tags with differing severities.  :)

Thoughts?


Best wishes,

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



Re: [lintian] 02/02: Check all subdirectories under /usr/share/doc/foo to test whether we ship example files, not just /usr/share/doc/foo/examples/.

2018-03-07 Thread Chris Lamb
Hey Niels,

> While you are updating that code path, could you add the missing the \Q
> \E around that $name (or quotemeta)?

Good spot. Updated in:

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


Best wishes,

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



[lintian] 01/01: checks/cruft.pm: Use \Q & \E to "quotemeta" the embedded variable in regular expression. Thanks, Niels!

2018-03-07 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 fb18b0416dee3c31b6595fe590e56c81c5dd915e
Author: Chris Lamb 
Date:   Wed Mar 7 09:18:39 2018 -0800

checks/cruft.pm: Use \Q & \E to "quotemeta" the embedded variable in 
regular expression. Thanks, Niels!
---
 checks/cruft.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/checks/cruft.pm b/checks/cruft.pm
index 5a7ef79..9406ee0 100644
--- a/checks/cruft.pm
+++ b/checks/cruft.pm
@@ -1575,7 +1575,8 @@ sub _ships_examples {
 # If we have an -examples package, assume we ship examples.
 return 1 if $name =~ m{-examples$};
 my @files = $binpkg->info->sorted_index;
-return 1 if any { m{^usr/share/doc/$name/(.+/)?examples/$} } @files;
+return 1
+  if any { m{^usr/share/doc/\Q$name\E/(.+/)?examples/$} } @files;
 }
 return;
 }

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



[lintian] branch master updated (cfb0964 -> fb18b04)

2018-03-07 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  cfb0964   spelling: Add another correction
   new  fb18b04   checks/cruft.pm: Use \Q & \E to "quotemeta" the embedded 
variable in regular expression. Thanks, Niels!

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/cruft.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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



[lintian] branch master updated (a9155a4 -> cfb0964)

2018-03-07 Thread Paul Wise
This is an automated email from the git hooks/post-receive script.

pabs pushed a change to branch master
in repository lintian.

  from  a9155a4   Ignore entries that end with ":" to avoid false-positives 
in the debian-changelog-line-too-short. Thanks to Mattia Rizollo for the 
report. (Closes: #892197)
   new  cfb0964   spelling: Add another correction

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:
 data/spelling/corrections-multiword | 1 +
 1 file changed, 1 insertion(+)

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



[lintian] 01/01: spelling: Add another correction

2018-03-07 Thread Paul Wise
This is an automated email from the git hooks/post-receive script.

pabs pushed a commit to branch master
in repository lintian.

commit cfb09643c0dcdbb0043639b62bcc612e1e3a1bc2
Author: Paul Wise 
Date:   Wed Mar 7 20:16:55 2018 +0800

spelling: Add another correction
---
 data/spelling/corrections-multiword | 1 +
 1 file changed, 1 insertion(+)

diff --git a/data/spelling/corrections-multiword 
b/data/spelling/corrections-multiword
index 06e8058..d08bdf8 100644
--- a/data/spelling/corrections-multiword
+++ b/data/spelling/corrections-multiword
@@ -16,6 +16,7 @@
 (?i)gnu library public licen[cs]e||GNU Library General Public License
 (?i)gnu public licen[cs]e||GNU General Public License
 (?i)is not enable||is not enabled
+(?i)lest strict||less strict
 (?i)let's you||lets you
 (?i)none existent||non-existent
 (?i)none? the less||nonetheless

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



Bug#892255: lintian: orig-tarball-missing-upstream-signature with signed .tar

2018-03-07 Thread Uwe Kleine-König
Package: lintian
Version: 2.5.78
Severity: normal

Hello,

uwe@taurus:~/debpkg/microcom$ xzcat microcom_2017.03.0.orig.tar.xz | gpg 
--verify microcom_2017.03.0.orig.tar.asc -
gpg: Signature made Tue 06 Mar 2018 02:21:38 PM CET
gpg:using RSA key 7E722A169018ACFF3E74A40BC1FC1478ADCAEC09
gpg: Good signature from "Uwe Kleine-König " [ultimate]
gpg: aka "Uwe Kleine-König " [ultimate]
gpg: aka "Uwe Kleine-König " [ultimate]
gpg: aka "Uwe Kleine-König " [ultimate]
gpg: aka "Uwe Kleine-König " 
[ultimate]
gpg: aka "Uwe Kleine-König " 
[ultimate]
Primary key fingerprint: 0D25 11F3 22BF AB1C 1580  266B E2DC DD91 3266 9BD6
 Subkey fingerprint: 7E72 2A16 9018 ACFF 3E74  A40B C1FC 1478 ADCA EC09
uwe@taurus:~/debpkg/microcom$ grep microcom_2017.03.0.orig.tar.asc 
microcom_2017.03.0-1_source.changes
 f26feeaf212c5be8fa203f2102ac68024ab4cda0010f0b84c8a4415fd9c6471d 310 
microcom_2017.03.0.orig.tar.asc
 bea45ee0c144df466f1340ea45771353e6aa5e49 310 microcom_2017.03.0.orig.tar.asc
 918dc12abfcb768c5563351a21b144f6 310 - - microcom_2017.03.0.orig.tar.asc
uwe@taurus:~/debpkg/microcom$ lintian microcom_2017.03.0-1_source.changes
W: microcom changes: orig-tarball-missing-upstream-signature 
microcom_2017.03.0.orig.tar.xz

It would be great if lintian considered a signature on the extracted
orig.tar that is shipped in the changes file.

This is (slightly) related to https://bugs.debian.org/882694 .

Best regards
Uwe

-- System Information:
Debian Release: buster/sid
  APT prefers testing
  APT policy: (700, 'testing'), (600, 'unstable'), (500, 'unstable-debug'), 
(500, 'stable'), (499, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.15.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages lintian depends on:
ii  binutils  2.30-5
ii  bzip2 1.0.6-8.1
ii  diffstat  1.61-1+b1
ii  dpkg  1.19.0.5
ii  file  1:5.32-2
ii  gettext   0.19.8.1-4
ii  intltool-debian   0.35.0+20060710.4
ii  libapt-pkg-perl   0.1.33
ii  libarchive-zip-perl   1.60-1
ii  libclass-accessor-perl0.51-1
ii  libclone-perl 0.39-1
ii  libdpkg-perl  1.19.0.5
ii  libemail-valid-perl   1.202-1
ii  libfile-basedir-perl  0.07-1
ii  libipc-run-perl   0.96-1
ii  liblist-moreutils-perl0.416-1+b3
ii  libparse-debianchangelog-perl 1.2.0-12
ii  libperl5.26 [libdigest-sha-perl]  5.26.1-5
ii  libtext-levenshtein-perl  0.13-1
ii  libtimedate-perl  2.3000-2
ii  liburi-perl   1.73-1
ii  libxml-simple-perl2.24-1
ii  libyaml-libyaml-perl  0.69+repack-1
ii  man-db2.8.2-1
ii  patchutils0.3.4-2
ii  perl  5.26.1-5
ii  t1utils   1.41-2
ii  xz-utils  5.2.2-1.3

Versions of packages lintian recommends:
ii  libperlio-gzip-perl  0.19-1+b4

Versions of packages lintian suggests:
ii  binutils-multiarch 2.30-5
ii  dpkg-dev   1.19.0.5
ii  libhtml-parser-perl3.72-3+b2
ii  libtext-template-perl  1.47-1

-- no debconf information