Bug#787469: lintian: please report warning for debian/.lintian-overrides. if pkg is M-A: same

2018-01-28 Thread Helmut Grohne 
On Sun, Jan 28, 2018 at 11:45:37PM +0100, Mattia Rizzolo wrote:
> On Mon, Jun 01, 2015 at 10:40:12PM +0200, Sebastian Ramacher wrote:
> > #787406 was reported against libav today. It was caused by
> > debian/libavcodec-extra-56.lintian-overrides and
> > debian/libavcodec-extra-56.lintian-overrides.ii386 with differing content.
> > 
> > I think this situation could be detected by lintian and would be great if it
> > could emit a warning refering to [1] (as pointed out by Jakub in the bug
> > report).

I agree, but the present implementation captures too much: A single
arch-specific override is not a problem, as long as there is no other
arch-specific or arch-generic override with different content. Imagine a
package that needs a particular override on x86 and thus ships the same
override for amd64, i386 and x32. Lintian would now flag this
non-problematic case.

> Actually, the problem may be wider: debhelper has a more general feature
> of allowing '.$arch' as a suffix to the config files, that works not
> only for lintian overrides (even if most commonly it's used only for
> lintian overrides and symbol files, from my experience).

Indeed, I've seen it being used for maintainer scripts. Though for
maintainer scripts we know that they get unpacked to
architecture-dependent paths (indeed all control files are). Then there
are files (like .install) that only influence what is being packaged.
Only those that actually end up being shipped in the data.tar matter
here. Ever seen $pkg.menu.$arch or $pkg.logrotate.$arch?

> I believe you should check all files supported by the regular debhelper
> tools that affects the content of data.tar.xz (that means, things like
> d/libfoo.symbols.$arch should be ignored, and perhaps some others).

Given the above, I think we should assume file types as "ok" unless we
know better. Extending the flag list to other types like menu or
logrotate likely makes sense, but I'd keep it as a blacklist rather than
trying to enumerate which files are not problematic.

Helmut

Jenkins build is back to normal : lintian-tests_stretch-backports #128

2018-01-28 Thread jenkins 
See

Build failed in Jenkins: lintian-tests_stretch-backports #127

2018-01-28 Thread jenkins 
See 


--
Started by upstream project "lintian-tests_buster" build number 541
originally caused by:
 Started by upstream project "lintian-tests_sid" build number 2496
 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: 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 128:
stdout: 
stderr: fatal: unable to connect to anonscm.debian.org:
anonscm.debian.org[0: 5.153.231.21]: errno=Connection refused
anonscm.debian.org[1: 2001:41c8:1000:21::21:21]: errno=Network is unreachable


at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1970)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandWithCredentials(CliGitAPIImpl.java:1689)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl.access$300(CliGitAPIImpl.java:71)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl$1.execute(CliGitAPIImpl.java:380)
at 
org.jenkinsci.plugins.gitclient.CliGitAPIImpl$2.execute(CliGitAPIImpl.java:589)
at hudson.plugins.git.GitSCM.retrieveChanges(GitSCM.java:1083)
at hudson.plugins.git.GitSCM.checkout(GitSCM.java:1123)
at hudson.scm.SCM.checkout(SCM.java:495)
at hudson.model.AbstractProject.checkout(AbstractProject.java:1202)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:574)
at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)
at 
hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:499)
at hudson.model.Run.execute(Run.java:1724)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
at hudson.model.ResourceController.execute(ResourceController.java:97)
at hudson.model.Executor.run(Executor.java:421)
ERROR: Error cloning remote repo 'origin'

Bug#787469: lintian: please report warning for debian/.lintian-overrides. if pkg is M-A: same

2018-01-28 Thread Mattia Rizzolo 
On Mon, Jun 01, 2015 at 10:40:12PM +0200, Sebastian Ramacher wrote:
> #787406 was reported against libav today. It was caused by
> debian/libavcodec-extra-56.lintian-overrides and
> debian/libavcodec-extra-56.lintian-overrides.ii386 with differing content.
> 
> I think this situation could be detected by lintian and would be great if it
> could emit a warning refering to [1] (as pointed out by Jakub in the bug
> report).

Actually, the problem may be wider: debhelper has a more general feature
of allowing '.$arch' as a suffix to the config files, that works not
only for lintian overrides (even if most commonly it's used only for
lintian overrides and symbol files, from my experience).

I believe you should check all files supported by the regular debhelper
tools that affects the content of data.tar.xz (that means, things like
d/libfoo.symbols.$arch should be ignored, and perhaps some others).


In other words, my request is to widen the newly committed
multi-arch-same-package-has-arch-specific-overrides to support all
debhelper config files.


I'm CCing Helmut as the leading expert of "Multi-Arch-ness" to double
check on whether I'm writing nonsense :)

-- 
regards,
Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540  .''`.
more about me:  https://mapreri.org : :'  :
Launchpad user: https://launchpad.net/~mapreri  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-


signature.asc
Description: PGP signature

Jenkins build is back to normal : lintian-tests_sid #2495

2018-01-28 Thread jenkins 
See

Processed: Re: lintian: new pedantic check about planned features in package descriptions

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

> tags 782990 + pending
Bug #782990 [lintian] lintian: new pedantic check about planned features in 
package descriptions
Added tag(s) pending.
> thanks
Stopping processing here.

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

Bug#782990: lintian: new pedantic check about planned features in package descriptions

2018-01-28 Thread Chris Lamb 
tags 782990 + pending
thanks

Fixed in Git, pending upload:

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


Regards,

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

[lintian] 01/01: Check for packages that mention planned/upcoming features in their long description. (Closes: #782990)

2018-01-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 f444db614d4ac4ed4ef24060cb6856fb61b5b631
Author: Chris Lamb 
Date:   Mon Jan 29 09:21:21 2018 +1100

Check for packages that mention planned/upcoming features in their long 
description. (Closes: #782990)
---
 checks/description.desc| 12 +++
 checks/description.pm  |  8 
 data/description/planned-features  |  8 
 debian/changelog   |  3 +++
 .../debian/debian/control.in   | 23 ++
 t/tests/description-mentions-planned-features/desc |  7 +++
 t/tests/description-mentions-planned-features/tags |  6 ++
 7 files changed, 67 insertions(+)

diff --git a/checks/description.desc b/checks/description.desc
index db64e64..a7cef96 100644
--- a/checks/description.desc
+++ b/checks/description.desc
@@ -244,3 +244,15 @@ Experimental: yes
 Info: Debian users are likely to look for perl modules by their name, e.g.
  Foo::Bar, not by package name (libfoo-bar-perl). To make this easier, the main
  module name should be present in the long package description.
+
+Tag: description-mentions-planned-features
+Severity: wishlist
+Certainty: wild-guess
+Info: This package appears to mention planned or upcoming features of
+ the software.
+ .
+ Package descriptions should not mention features that not yet implemented
+ as they waste the time of people searching for particular keywords. They
+ are also liable to become outdated quickly.
+ .
+ Please remove such statements from the package description.
diff --git a/checks/description.pm b/checks/description.pm
index 8b214fe..6396466 100644
--- a/checks/description.pm
+++ b/checks/description.pm
@@ -29,11 +29,14 @@ use constant DH_MAKE_PERL_TEMPLATE => 'this description was'
 
 use Encode qw(decode);
 
+use Lintian::Data;
 use Lintian::Check
   qw(check_spelling check_spelling_picky spelling_tag_emitter);
 use Lintian::Tags qw(tag);
 use Lintian::Util qw(strip);
 
+my $PLANNED_FEATURES = Lintian::Data->new('description/planned-features');
+
 my $SPELLING_ERROR_IN_SYNOPSIS
   = spelling_tag_emitter('spelling-error-in-description-synopsis');
 my $SPELLING_ERROR_IN_DESCRIPTION
@@ -153,6 +156,11 @@ sub run {
 $flagged_homepage = 1;
 }
 
+foreach my $regex ($PLANNED_FEATURES->all()) {
+tag 'description-mentions-planned-features', "(line $lines)"
+  if m/$regex/i;
+}
+
 if (index(lc($_), DH_MAKE_PERL_TEMPLATE) != -1) {
 tag 'description-contains-dh-make-perl-template';
 }
diff --git a/data/description/planned-features 
b/data/description/planned-features
new file mode 100644
index 000..70c8f14
--- /dev/null
+++ b/data/description/planned-features
@@ -0,0 +1,8 @@
+# Regular expressions (/i) of "planned features"
+
+will soon allow
+soon will allow
+soon will be able
+will soon be able
+will soon support
+soon will support
diff --git a/debian/changelog b/debian/changelog
index 31b87a3..00a99f5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,6 +5,9 @@ lintian (2.5.73) UNRELEASED; urgency=medium
   * checks/cruft.pm:
 + [CL] When looking for the source of "build/foo/bar.min.js", also
   check "src/foo/bar.js".  (Closes: #832027)
+  * checks/description.pm:
++ [CL] Check for packages that mention planned/upcoming features in
+  their long description.  (Closes: #782990)
   * checks/fields.{desc,pm}:
 + [CL] Add a pedantic warning for co-maintained packages that are not
   managed in a revision control system.  (Closes: #884497)
diff --git 
a/t/tests/description-mentions-planned-features/debian/debian/control.in 
b/t/tests/description-mentions-planned-features/debian/debian/control.in
new file mode 100644
index 000..21574e6
--- /dev/null
+++ b/t/tests/description-mentions-planned-features/debian/debian/control.in
@@ -0,0 +1,23 @@
+Source: {$source}
+Priority: optional
+Section: {$section}
+Maintainer: {$author}
+Standards-Version: {$standards_version}
+Build-Depends: {$build_depends}
+Rules-Requires-Root: no
+
+Package: {$source}
+Architecture: {$architecture}
+Depends: $\{misc:Depends\}
+Description: {$description}
+ This is a test package designed to exercise some feature or tag of
+ Lintian.  It is part of the Lintian test suite and may do very odd
+ things.  It should not be installed like a regular package.  It may
+ be an empty package.
+ .
+ Foo will soon allow something.
+ Foo soon will allow something.
+ Foo soon will be able something.
+ Foo will soon be able something.
+ Foo will soon support something.
+ Foo SOON WILL SUPPORT something.
diff --git a/t/tests/description-mentions-planned-features/desc 
b/t/tests/description-mentions-planned-features/desc
new file mode 100644
index 000..fcea236
--- /dev/null
+++ b/t/tests/description-mentions-p

[lintian] branch master updated (bb740f6 -> f444db6)

2018-01-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  bb740f6   Correct reference for Lintian manual.
   new  f444db6   Check for packages that mention planned/upcoming features 
in their long description. (Closes: #782990)

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/description.desc  | 12 
 checks/description.pm|  8 
 data/description/planned-features|  8 
 debian/changelog |  3 +++
 .../debian/debian/control.in |  7 ++-
 t/tests/description-mentions-planned-features/desc   |  7 +++
 t/tests/description-mentions-planned-features/tags   |  6 ++
 7 files changed, 50 insertions(+), 1 deletion(-)
 create mode 100644 data/description/planned-features
 copy t/tests/{description-homepage => 
description-mentions-planned-features}/debian/debian/control.in (71%)
 create mode 100644 t/tests/description-mentions-planned-features/desc
 create mode 100644 t/tests/description-mentions-planned-features/tags

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

Build failed in Jenkins: lintian-tests_sid #2494

2018-01-28 Thread jenkins 
See 


Changes:

[lamby] Warn about Multi-Arch: same packages that ship architecture-specific

--
[...truncated 244.70 KB...]
Adding debian:DigiCert_Global_Root_G2.pem
Adding debian:DigiCert_Global_Root_CA.pem
Adding debian:DigiCert_Assured_ID_Root_G3.pem
Adding debian:DigiCert_Assured_ID_Root_G2.pem
Adding debian:DigiCert_Assured_ID_Root_CA.pem
Adding debian:Deutsche_Telekom_Root_CA_2.pem
Adding debian:DST_Root_CA_X3.pem
Adding debian:DST_ACES_CA_X6.pem
Adding debian:D-TRUST_Root_Class_3_CA_2_EV_2009.pem
Adding debian:D-TRUST_Root_Class_3_CA_2_2009.pem
Adding debian:Cybertrust_Global_Root.pem
Adding debian:Comodo_Trusted_Services_root.pem
Adding debian:Comodo_Secure_Services_root.pem
Adding debian:Comodo_AAA_Services_root.pem
Adding debian:China_Internet_Network_Information_Center_EV_Certificates_Root.pem
Adding debian:Chambers_of_Commerce_Root_-_2008.pem
Adding debian:Certum_Trusted_Network_CA_2.pem
Adding debian:Certum_Trusted_Network_CA.pem
Adding debian:Certum_Root_CA.pem
Adding debian:Certplus_Root_CA_G2.pem
Adding debian:Certplus_Root_CA_G1.pem
Adding debian:Certplus_Class_2_Primary_CA.pem
Adding debian:Certinomis_-_Root_CA.pem
Warning: there was a problem reading the certificate file 
/etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem. Message:
  /etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem (No such file or directory)
Adding debian:Certigna.pem
Adding debian:Camerfirma_Global_Chambersign_Root.pem
Adding debian:Camerfirma_Chambers_of_Commerce_Root.pem
Adding debian:COMODO_RSA_Certification_Authority.pem
Adding debian:COMODO_ECC_Certification_Authority.pem
Adding debian:COMODO_Certification_Authority.pem
Adding debian:CNNIC_ROOT.pem
Adding debian:CFCA_EV_ROOT.pem
Adding debian:CA_Disig_Root_R2.pem
Adding debian:CA_Disig_Root_R1.pem
Adding debian:Buypass_Class_3_Root_CA.pem
Adding debian:Buypass_Class_2_Root_CA.pem
Adding debian:Baltimore_CyberTrust_Root.pem
Adding debian:Autoridad_de_Certificacion_Firmaprofesional_CIF_A62634068.pem
Adding debian:Atos_TrustedRoot_2011.pem
Adding debian:Amazon_Root_CA_4.pem
Adding debian:Amazon_Root_CA_3.pem
Adding debian:Amazon_Root_CA_2.pem
Adding debian:Amazon_Root_CA_1.pem
Adding debian:AffirmTrust_Premium_ECC.pem
Adding debian:AffirmTrust_Premium.pem
Adding debian:AffirmTrust_Networking.pem
Adding debian:AffirmTrust_Commercial.pem
Adding debian:AddTrust_Qualified_Certificates_Root.pem
Adding debian:AddTrust_Public_Services_Root.pem
Adding debian:AddTrust_Low-Value_Services_Root.pem
Adding debian:AddTrust_External_Root.pem
Adding debian:Actalis_Authentication_Root_CA.pem
Adding debian:AC_RAIZ_FNMT-RCM.pem
Adding debian:ACEDICOM_Root.pem
Adding debian:ACCVRAIZ1.pem
done.
Setting up openjdk-8-jdk-headless:amd64 (8u151-b12-1) ...
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/idlj to 
provide /usr/bin/idlj (idlj) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jdeps to 
provide /usr/bin/jdeps (jdeps) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/wsimport to 
provide /usr/bin/wsimport (wsimport) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/rmic to 
provide /usr/bin/rmic (rmic) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jinfo to 
provide /usr/bin/jinfo (jinfo) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jsadebugd to 
provide /usr/bin/jsadebugd (jsadebugd) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/native2ascii 
to provide /usr/bin/native2ascii (native2ascii) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstat to 
provide /usr/bin/jstat (jstat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javac to 
provide /usr/bin/javac (javac) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javah to 
provide /usr/bin/javah (javah) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstack to 
provide /usr/bin/jstack (jstack) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jrunscript to 
provide /usr/bin/jrunscript (jrunscript) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javadoc to 
provide /usr/bin/javadoc (javadoc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jhat to 
provide /usr/bin/jhat (jhat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javap to 
provide /usr/bin/javap (javap) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jar to provide 
/usr/bin/jar (jar) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/xjc to provide 
/usr/bin/xjc (xjc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/schemagen to 
provide /u

[lintian] branch master updated (4575daa -> bb740f6)

2018-01-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  4575daa   Warn about Multi-Arch: same packages that ship 
architecture-specific Lintian overrides. Thanks to Sebastian Ramacher for the 
report. (Closes: #787469)
   new  d81e9bb   Add test for 
multi-arch-same-package-has-arch-specific-overrides.
   new  bb740f6   Correct reference for Lintian manual.

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/fields.desc|  2 +-
 .../debian/debian/control.in  | 11 +--
 ...as-arch-specific-overrides-nonrel.lintian-overrides.amd64} |  0
 ...ckage-has-arch-specific-overrides.lintian-overrides.amd64} |  0
 .../desc  |  6 ++
 .../tags  |  3 +++
 6 files changed, 15 insertions(+), 7 deletions(-)
 copy t/tests/{files-flash-non-free => 
fields-multi-arch-same-package-has-arch-specific-overrides}/debian/debian/control.in
 (83%)
 copy 
t/tests/{patch-systems-quilt-general/debian/debian/patches/file-commented-out 
=> 
fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides-nonrel.lintian-overrides.amd64}
 (100%)
 copy 
t/tests/{patch-systems-quilt-general/debian/debian/patches/file-commented-out 
=> 
fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides.lintian-overrides.amd64}
 (100%)
 create mode 100644 
t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/desc
 create mode 100644 
t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/tags

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

[lintian] 01/02: Add test for multi-arch-same-package-has-arch-specific-overrides.

2018-01-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 d81e9bb7a92b218ec82c04caa74c7c333c7f0202
Author: Chris Lamb 
Date:   Mon Jan 29 08:58:16 2018 +1100

Add test for multi-arch-same-package-has-arch-specific-overrides.
---
 .../debian/debian/control.in   | 28 ++
 ...ecific-overrides-nonrel.lintian-overrides.amd64 |  1 +
 ...arch-specific-overrides.lintian-overrides.amd64 |  1 +
 .../desc   |  6 +
 .../tags   |  3 +++
 5 files changed, 39 insertions(+)

diff --git 
a/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/control.in
 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/control.in
new file mode 100644
index 000..8a659ad
--- /dev/null
+++ 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/control.in
@@ -0,0 +1,28 @@
+Source: {$source}
+Priority: optional
+Section: {$section}
+Maintainer: {$author}
+Standards-Version: {$standards_version}
+Build-Depends: {$build_depends}
+Rules-Requires-Root: no
+
+Package: {$source}
+Architecture: {$architecture}
+Multi-Arch: same
+Depends: $\{misc:Depends\}
+Description: {$description}
+ This is a test package designed to exercise some feature or tag of
+ Lintian.  It is part of the Lintian test suite and may do very odd
+ things.  It should not be installed like a regular package.  It may
+ be an empty package.
+
+Package: {$source}-nonrel
+Architecture: {$architecture}
+Depends: $\{misc:Depends\}
+Description: {$description} (false positive)
+ This is a test package designed to exercise some feature or tag of
+ Lintian.  It is part of the Lintian test suite and may do very odd
+ things.  It should not be installed like a regular package.  It may
+ be an empty package.
+ .
+ This is a false-positive.
diff --git 
a/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides-nonrel.lintian-overrides.amd64
 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides-nonrel.lintian-overrides.amd64
new file mode 100644
index 000..257cc56
--- /dev/null
+++ 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides-nonrel.lintian-overrides.amd64
@@ -0,0 +1 @@
+foo
diff --git 
a/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides.lintian-overrides.amd64
 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides.lintian-overrides.amd64
new file mode 100644
index 000..257cc56
--- /dev/null
+++ 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/debian/debian/fields-multi-arch-same-package-has-arch-specific-overrides.lintian-overrides.amd64
@@ -0,0 +1 @@
+foo
diff --git 
a/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/desc 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/desc
new file mode 100644
index 000..962058d
--- /dev/null
+++ b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/desc
@@ -0,0 +1,6 @@
+Testname: fields-multi-arch-same-package-has-arch-specific-overrides
+Description: Test for Multi-Arch: same package with arch-specific overrides
+Version: 1.0
+Architecture: amd64
+Test-For:
+ multi-arch-same-package-has-arch-specific-overrides
diff --git 
a/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/tags 
b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/tags
new file mode 100644
index 000..46dcd68
--- /dev/null
+++ b/t/tests/fields-multi-arch-same-package-has-arch-specific-overrides/tags
@@ -0,0 +1,3 @@
+W: fields-multi-arch-same-package-has-arch-specific-overrides source: 
multi-arch-same-package-has-arch-specific-overrides 
debian/fields-multi-arch-same-package-has-arch-specific-overrides.lintian-overrides.amd64
+W: fields-multi-arch-same-package-has-arch-specific-overrides-nonrel: 
package-has-long-file-name 79 (88) > 80
+W: fields-multi-arch-same-package-has-arch-specific-overrides: 
package-has-long-file-name 72 (81) > 80

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

[lintian] 02/02: Correct reference for Lintian manual.

2018-01-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 bb740f6bb6110cf19e5b1ffa94e22c6632c2290c
Author: Chris Lamb 
Date:   Mon Jan 29 08:58:45 2018 +1100

Correct reference for Lintian manual.
---
 checks/fields.desc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/checks/fields.desc b/checks/fields.desc
index 4579212..fe4dbbe 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -1414,4 +1414,4 @@ Severity: normal
 Certainty: certain
 Info: The specified file contains architecture-specific Lintian overrides
  but this package is declared as Multi-Arch: same.
-Ref: https://lintian.debian.org/manual/section-2.4.html#section-2.4.3, #787406
+Ref: lintian 2.4.3, #787406

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

Processed: Re: lintian: please report warning for debian/.lintian-overrides. if pkg is M-A: same

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

> tags 787469 + pending
Bug #787469 [lintian] lintian: please report warning for 
debian/.lintian-overrides. if pkg is M-A: same
Added tag(s) pending.
> thanks
Stopping processing here.

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

Bug#787469: lintian: please report warning for debian/.lintian-overrides. if pkg is M-A: same

2018-01-28 Thread Chris Lamb 
tags 787469 + pending
thanks

Fixed in Git, pending upload:

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


Regards,

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

[lintian] 01/01: Warn about Multi-Arch: same packages that ship architecture-specific Lintian overrides. Thanks to Sebastian Ramacher for the report. (Closes: #787469)

2018-01-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 4575daa2fcb96bfab67b8dfa56d474a726cfc191
Author: Chris Lamb 
Date:   Mon Jan 29 08:34:08 2018 +1100

Warn about Multi-Arch: same packages that ship architecture-specific 
Lintian overrides. Thanks to Sebastian Ramacher for the report. (Closes: 
#787469)
---
 checks/fields.desc |  7 +++
 checks/fields.pm   | 11 +++
 debian/changelog   |  3 +++
 3 files changed, 21 insertions(+)

diff --git a/checks/fields.desc b/checks/fields.desc
index e501299..4579212 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -1408,3 +1408,10 @@ Info: Based on the content of the maintainer and 
uploader fields this
  .
  It is recommended that shared maintenance of packages are co-ordinated
  via a revision control system.
+
+Tag: multi-arch-same-package-has-arch-specific-overrides
+Severity: normal
+Certainty: certain
+Info: The specified file contains architecture-specific Lintian overrides
+ but this package is declared as Multi-Arch: same.
+Ref: https://lintian.debian.org/manual/section-2.4.html#section-2.4.3, #787406
diff --git a/checks/fields.pm b/checks/fields.pm
index cef7533..c0a3509 100644
--- a/checks/fields.pm
+++ b/checks/fields.pm
@@ -354,6 +354,17 @@ sub run {
 $arch =~ s/\n//o;
 tag 'illegal-multi-arch-value', $arch, $march if ($arch eq 'all');
 }
+
+}
+
+if ($type eq 'source') {
+for my $bin ($info->binaries) {
+my $arch = $info->binary_field($bin, 'architecture');
+my $fname = "debian/$bin.lintian-overrides.$arch";
+next unless $info->binary_field($bin, 'multi-arch', '') eq 'same';
+tag 'multi-arch-same-package-has-arch-specific-overrides', $fname
+  if $info->index_resolved_path($fname);
+}
 }
 
 if ($type eq 'binary'){
diff --git a/debian/changelog b/debian/changelog
index f2c1d16..31b87a3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -8,6 +8,9 @@ lintian (2.5.73) UNRELEASED; urgency=medium
   * checks/fields.{desc,pm}:
 + [CL] Add a pedantic warning for co-maintained packages that are not
   managed in a revision control system.  (Closes: #884497)
++ [CL] Warn about Multi-Arch: same packages that ship
+  architecture-specific Lintian overrides. Thanks to Sebastian
+  Ramacher for the report.  (Closes: #787469)
   * checks/patch-systems.{desc,pm}:
 + [CL] Emit a pedantic warning for packages that are using the dpatch
   patch system.  (Closes: #884500)

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

[lintian] branch master updated (c2194b5 -> 4575daa)

2018-01-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  c2194b5   Emit a pedantic warning for packages that are using the 
dpatch patch system. (Closes: #884500)
   new  4575daa   Warn about Multi-Arch: same packages that ship 
architecture-specific Lintian overrides. Thanks to Sebastian Ramacher for the 
report. (Closes: #787469)

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.desc |  7 +++
 checks/fields.pm   | 11 +++
 debian/changelog   |  3 +++
 3 files changed, 21 insertions(+)

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

Bug#884498: lintian: Pedantic check for source format 1.0 packages

2018-01-28 Thread Chris Lamb 
Hi Niels,

> I assume 3.0 (native) would be easiest to do here (given the current
> default is the native version of 1.0).

I perhaps should have quoted more of my reply - this is the problematic
bit :)

> I have found it useful to be able to run lintian directly on a test
> artefact for debugging purposes.
[…]
> Rhetorical question: How would we write a test for that tag if lintian
> refuses to emit the tag in the test?

But of course; these were the — to me, too obvious to mention! — issues
that I was covering under the banner of "too horrible" :)


Best wishes,

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

Bug#884498: lintian: Pedantic check for source format 1.0 packages

2018-01-28 Thread Niels Thykier 
Chris Lamb:
> Chris Lamb wrote:
> 
>> Patch attached:
> […]
>> … but this naturally results in a huge number of testsuite failures as
>> the test templates use format 1.0 and thus violate the tag. Changing
>> these templates to use the 3.x series results in other test-specific
>> issues. Any pointers? :)
> 

If the 1.0 format is formally deprecated, then ideally the test suite
would default to a supported (non-deprecated) format instead.  I assume
3.0 (native) would be easiest to do here (given the current default is
the native version of 1.0).

Once the default is handled, then the rest have to be dealt with one at
the time.  Possible over more than one release.

> Is suppose it would be too horrible to make emission of this tag
> require $ENV{'LINTIAN_TEST_ROOT'} to be set...
> 
> 
> Regards,
> 

I cannot recommend it (but nor will I stop you) as lintian now behaves
different based on whether it is being tested or not.  Personally, I
have found it useful to be able to run lintian directly on a test
artefact for debugging purposes.  This would be harder to reason about
if lintian behaves differently when it is being tested and when it isn't.


Rhetorical question: How would we write a test for that tag if lintian
refuses to emit the tag in the test?

Thanks,
~Niels

Bug#774488: lintian: implement feature like Vcswatch (wishlist)

2018-01-28 Thread Chris Lamb 
tags 774488 + moreinfo
thanks

Dear Hideki-san!

> "Error: The Vcs URL git://git.debian.org/pkg-nlp-ja/mecab-naist-jdic.git is 
> deprecated/defunct, git://anonscm.debian.org/pkg-nlp-ja/mecab-naist-jdic.git
> was tried instead. Please update the Vcs field in debian/control."
  ^

You mean, it accesses the internet to check the URI?


Best wishes,

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

Bug#770021: marked as done (lintian: check for FIXME)

2018-01-28 Thread Debian Bug Tracking System 
Your message dated Sun, 28 Jan 2018 17:17:32 +0530
with message-id 
<1517140052.2813838.1250723040.2865d...@webmail.messagingengine.com>
and subject line 
has caused the Debian Bug report #770021,
regarding lintian: check for FIXME
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
770021: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=770021
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: lintian
Version: 2.5.30
Severity: wishlist

Dear Maintainer,

When packaging, I add the 'FIXME' word in some points to review after
an initial result/package test. I think that it can be util for other
maintainers.

I put below an example, simulating that we need update a field
because the upstream has a new homepage and the d/control is using
an old URL:

Source: mac-robber
Section: utils
Priority: optional
Maintainer: Debian Forensics 
Uploaders: Joao Eriberto Mota Filho 
Build-Depends: debhelper (>= 9)
Standards-Version: 3.9.5
Homepage: FIXME http://www.sleuthkit.org/mac-robber
Vcs-Browser: http://anonscm.debian.org/cgit/forensics/mac-robber.git
Vcs-Git: git://anonscm.debian.org/forensics/mac-robber.git

Thanks for your attention.

Regards,

Eriberto

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.16-3-amd64 (SMP w/4 CPU cores)
Locale: LANG=pt_BR.UTF-8, LC_CTYPE=pt_BR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
--- End Message ---
--- Begin Message ---
Version: 2.5.52

+ [CL] Check for "FIXME" and similar placeholders in various files
  under debian/. (Closes: #846009)


Regards,

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

Processed: your mail

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

> severity 756574 wishlist
Bug #756574 [lintian] lintian: Please consider removing the tag 
license-problem-gfdl-non-official-text.
Severity set to 'wishlist' from 'minor'
> severity 664211 wishlist
Bug #664211 [lintian] add check for wrong dependency on debconf
Severity set to 'wishlist' from 'normal'
> severity 683060 wishlist
Bug #683060 [lintian] lintian: *-command-not-in-package could check for 
alternatives in dependent packages
Severity set to 'wishlist' from 'normal'
> severity 743226 wishlist
Bug #743226 [lintian] lintian.debian.org: link filenames to corresponding 
sources.debian.org entries
Severity set to 'wishlist' from 'normal'
> severity 751947 wishlist
Bug #751947 [lintian] Prefer SVG output in doxygen generated documentation
Severity set to 'wishlist' from 'normal'
> severity 796352 wishlist
Bug #796352 [lintian] please warn about a missing dependency on the gfortran 
module version
Severity set to 'wishlist' from 'normal'
> severity 813741 wishlist
Bug #813741 [lintian] please add a check that libraries are installed in 
/usr/lib instead of /usr/lib/DEB_HOST_MULTIARCH
Severity set to 'wishlist' from 'normal'
> severity 834607 wishlist
Bug #834607 [lintian] lintian: /usr/lib/ vs /usr/libexec
Severity set to 'wishlist' from 'normal'
> severity 843932 wishlist
Bug #843932 [lintian] dev-pkg-without-shlib-symlink: Please consider *.so links 
in /lib
Severity set to 'wishlist' from 'normal'
> severity 880430 wishlist
Bug #880430 [lintian] lintian: check for invalid arguments to 
dpkg-maintscript-helper
Severity set to 'wishlist' from 'normal'
> severity 843662 wishlist
Bug #843662 [lintian] lintian.d.o: Missing visual clue for arch tags
Severity set to 'wishlist' from 'normal'
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
664211: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=664211
683060: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683060
743226: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=743226
751947: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=751947
756574: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=756574
796352: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=796352
813741: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=813741
834607: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=834607
843662: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=843662
843932: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=843932
880430: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=880430
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#884498: lintian: Pedantic check for source format 1.0 packages

2018-01-28 Thread Chris Lamb 
Chris Lamb wrote:

> Patch attached:
[…]
> … but this naturally results in a huge number of testsuite failures as
> the test templates use format 1.0 and thus violate the tag. Changing
> these templates to use the 3.x series results in other test-specific
> issues. Any pointers? :)

Is suppose it would be too horrible to make emission of this tag
require $ENV{'LINTIAN_TEST_ROOT'} to be set...


Regards,

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

Processed (with 11 errors): your mail

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

> tags 756574 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #756574 [lintian] lintian: Please consider removing the tag 
license-problem-gfdl-non-official-text.
Requested to add no tags; doing nothing.
> tags 664211 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #664211 [lintian] add check for wrong dependency on debconf
Requested to add no tags; doing nothing.
> tags 683060 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #683060 [lintian] lintian: *-command-not-in-package could check for 
alternatives in dependent packages
Requested to add no tags; doing nothing.
> tags 743226 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #743226 [lintian] lintian.debian.org: link filenames to corresponding 
sources.debian.org entries
Requested to add no tags; doing nothing.
> tags 751947 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #751947 [lintian] Prefer SVG output in doxygen generated documentation
Requested to add no tags; doing nothing.
> tags 796352 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #796352 [lintian] please warn about a missing dependency on the gfortran 
module version
Requested to add no tags; doing nothing.
> tags 813741 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #813741 [lintian] please add a check that libraries are installed in 
/usr/lib instead of /usr/lib/DEB_HOST_MULTIARCH
Requested to add no tags; doing nothing.
> tags 834607 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ignore wheezy wheezy-ignore 
jessie jessie-ignore stretch stretch-ignore buster buster-ignore bullseye 
bullseye-ignore.

Bug #834607 [lintian] lintian: /usr/lib/ vs /usr/libexec
Requested to add no tags; doing nothing.
> tags 843932 + wishlist
Unknown tag/s: wishlist.
Recognized are: patch wontfix moreinfo unreproducible fixed potato woody sid 
help security upstream pending sarge sarge-ignore experimental d-i confirmed 
ipv6 lfs fixed-in-experimental fixed-upstream l10n newcomer a11y etch 
etch-ignore lenny lenny-ignore squeeze squeeze-ign

Bug#884500: lintian: Pedantic check for packages using dpatch

2018-01-28 Thread Chris Lamb 
tags 884500 + pending
thanks

Fixed in Git:

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


Regards,

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

Processed: Re: lintian: Pedantic check for packages using dpatch

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

> tags 884500 + pending
Bug #884500 [lintian] lintian: Pedantic check for packages using dpatch
Added tag(s) pending.
> thanks
Stopping processing here.

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

[lintian] branch master updated (a1370a6 -> c2194b5)

2018-01-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  a1370a6   Update existing tests for new 
co-maintained-package-with-no-vcs-headers tag.
   new  5d68fb7   Correct changelog ordering.
   new  c2194b5   Emit a pedantic warning for packages that are using the 
dpatch patch system. (Closes: #884500)

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/patch-systems.desc | 10 ++
 checks/patch-systems.pm   |  1 +
 debian/changelog  |  9 ++---
 t/tests/patch-systems-dpatch-description/desc |  3 +++
 t/tests/patch-systems-dpatch-description/tags |  1 +
 5 files changed, 21 insertions(+), 3 deletions(-)

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

[lintian] 02/02: Emit a pedantic warning for packages that are using the dpatch patch system. (Closes: #884500)

2018-01-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 c2194b556346247f37e28f75f1d76115b005dcce
Author: Chris Lamb 
Date:   Sun Jan 28 19:16:18 2018 +1100

Emit a pedantic warning for packages that are using the dpatch patch 
system. (Closes: #884500)
---
 checks/patch-systems.desc | 10 ++
 checks/patch-systems.pm   |  1 +
 debian/changelog  |  3 +++
 t/tests/patch-systems-dpatch-description/desc |  3 +++
 t/tests/patch-systems-dpatch-description/tags |  1 +
 5 files changed, 18 insertions(+)

diff --git a/checks/patch-systems.desc b/checks/patch-systems.desc
index 47301bc..e8f9b6f 100644
--- a/checks/patch-systems.desc
+++ b/checks/patch-systems.desc
@@ -208,3 +208,13 @@ Info: The specified patch is present under the 
debian/patches
  .
0001_fix-foo.patch
# 0002_fix-bar.patch
+
+Tag: package-uses-deprecated-dpatch-patch-system
+Severity: pedantic
+Certainty: certain
+Info: The dpatch patch system has been deprecated and superceded by the
+ "3.0 (quilt)" source format.
+ .
+ Please migrate the patches in the debian/patches directory and
+ the 00list file to use this source format.
+Ref: dpatch(1), dpkg-source(1)
diff --git a/checks/patch-systems.pm b/checks/patch-systems.pm
index fb2e629..134a650 100644
--- a/checks/patch-systems.pm
+++ b/checks/patch-systems.pm
@@ -60,6 +60,7 @@ sub run {
 #- dpatch
 if ($build_deps->implies('dpatch')) {
 my $list_file;
+tag 'package-uses-deprecated-dpatch-patch-system';
 $uses_patch_system++;
 $list_file = $dpdir->resolve_path('00list') if $dpdir;
 #check for a debian/patches file:
diff --git a/debian/changelog b/debian/changelog
index ec51588..f2c1d16 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -8,6 +8,9 @@ lintian (2.5.73) UNRELEASED; urgency=medium
   * checks/fields.{desc,pm}:
 + [CL] Add a pedantic warning for co-maintained packages that are not
   managed in a revision control system.  (Closes: #884497)
+  * checks/patch-systems.{desc,pm}:
++ [CL] Emit a pedantic warning for packages that are using the dpatch
+  patch system.  (Closes: #884500)
   * checks/source-copyright.{desc,pm}:
 + [CL] Warn about packages that specify a Files-Excluded header without
   a valid Format header as the former will be ignored by uscan(1).
diff --git a/t/tests/patch-systems-dpatch-description/desc 
b/t/tests/patch-systems-dpatch-description/desc
index 48e8ed0..f1c9de7 100644
--- a/t/tests/patch-systems-dpatch-description/desc
+++ b/t/tests/patch-systems-dpatch-description/desc
@@ -2,7 +2,10 @@ Testname: patch-systems-dpatch-description
 Version: 1.0
 Description: Test patch-system tags for a dpatch package
 Extra-Build-Depends: dpatch
+Options: --pedantic -I
+Skeleton: pedantic
 Test-For:
  dpatch-index-references-non-existent-patch
  dpatch-missing-description
+ package-uses-deprecated-dpatch-patch-system
  patch-file-present-but-not-mentioned-in-series
diff --git a/t/tests/patch-systems-dpatch-description/tags 
b/t/tests/patch-systems-dpatch-description/tags
index d2e60e8..06f9751 100644
--- a/t/tests/patch-systems-dpatch-description/tags
+++ b/t/tests/patch-systems-dpatch-description/tags
@@ -3,4 +3,5 @@ E: patch-systems-dpatch-description source: 
dpatch-index-references-non-existent
 E: patch-systems-dpatch-description source: 
dpatch-index-references-non-existent-patch 
01_some_other_patch_thats_not_in_the_package.dpatch
 I: patch-systems-dpatch-description source: dpatch-missing-description 
02_i_dont_have_a_description.patch
 I: patch-systems-dpatch-description source: dpatch-missing-description 
04_i_dont_have_a_description_either.patch
+P: patch-systems-dpatch-description source: 
package-uses-deprecated-dpatch-patch-system
 W: patch-systems-dpatch-description source: 
patch-file-present-but-not-mentioned-in-series 
03_specified_without_dpatch.dpatch

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

[lintian] 01/02: Correct changelog ordering.

2018-01-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 5d68fb7613dc38f30596da291cd36eb03fa40705
Author: Chris Lamb 
Date:   Sun Jan 28 19:16:29 2018 +1100

Correct changelog ordering.
---
 debian/changelog | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 2934b45..ec51588 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,9 @@ lintian (2.5.73) UNRELEASED; urgency=medium
 
   XXX: generate tag summary
 
+  * checks/cruft.pm:
++ [CL] When looking for the source of "build/foo/bar.min.js", also
+  check "src/foo/bar.js".  (Closes: #832027)
   * checks/fields.{desc,pm}:
 + [CL] Add a pedantic warning for co-maintained packages that are not
   managed in a revision control system.  (Closes: #884497)
@@ -12,9 +15,6 @@ lintian (2.5.73) UNRELEASED; urgency=medium
 + [CL] Warn when a "Files: *" DEP-5 paragraph exists but it is not the
   first paragraph. Thank to Christoph Biedl for the report and idea.
   (Closes: #879235)
-  * checks/cruft.pm:
-+ [CL] When looking for the source of "build/foo/bar.min.js", also
-  check "src/foo/bar.js".  (Closes: #832027)
 
   * data/files/python-generic-modules:
 + [CL] Detect "backports" (and "backport") as overly generic Python

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

Jenkins build is back to normal : lintian-tests_buster #538

2018-01-28 Thread jenkins 
See

Jenkins build is back to normal : lintian-tests_sid #2492

2018-01-28 Thread jenkins 
See

Processed: notfound 884502 in lintian/2.5.39

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

> # Remove found; else the BTS thinks the found version is a regression
> notfound 884502 lintian/2.5.39
Bug #884502 {Done: Chris Lamb } [lintian] lintian: Pedantic 
check for packages modifying files outside of debian/
No longer marked as found in versions lintian/2.5.39.
> thanks
Stopping processing here.

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

Bug#884502: marked as done (lintian: Pedantic check for packages modifying files outside of debian/)

2018-01-28 Thread Debian Bug Tracking System 
Your message dated Sun, 28 Jan 2018 13:32:11 +0530
with message-id 
<1517126531.2739970.1250622760.73c14...@webmail.messagingengine.com>
and subject line Re: lintian: Pedantic check for packages modifying files 
outside of debian/
has caused the Debian Bug report #884502,
regarding lintian: Pedantic check for packages modifying files outside of 
debian/
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
884502: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884502
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: lintian
Version: 2.5.39
Severity: wishlist

Hi,

In the thread https://lists.debian.org/debian-devel/2015/12/msg00383.html, and 
specifically in
https://lists.debian.org/20151228134852.ga17...@xanadu.blop.info

there was a discussion about adding several pedantic warnings for:
qa-comaint_but_no_vcs.txt
qa-format_10.txt (3153 packages) (probably below than pedantic)
qa-helper_classic_debhelper.txt (3647 packages) (probably below than pedantic)
qa-helper_not_debhelper.txt (144 packages)
qa-patch_dpatch.txt (170 packages)
qa-patch_modified-files-outside-debian.txt (1156 packages)
qa-patch_more_than_one.txt (201 packages)
qa-patch_other.txt (51 packages)
qa-patch_quilt.txt (445 packages)
qa-patch_simple-patchsys.txt (129 packages)
qa-vcs_but_not_git_or_svn.txt (290 packages) (probably below than pedantic)
qa-vcs_more_than_one_declared_vcs.txt (1 package)

I'm not copying/summaryzing the discussion as it's still ongoing on -devel@.

- Lucas

-- System Information:
Debian Release: stretch/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'stable-updates'), (400, 'stable'), (300, 
'unstable'), (150, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386, armhf

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

Versions of packages lintian depends on:
ii  binutils   2.25.90.20151209-1
ii  bzip2  1.0.6-8
ii  diffstat   1.60-1
ii  file   1:5.25-2
ii  gettext0.19.6-1
ii  hardening-includes 2.8+nmu2
ii  intltool-debian0.35.0+20060710.4
ii  libapt-pkg-perl0.1.29+b4
ii  libarchive-zip-perl1.55-1
ii  libclass-accessor-perl 0.34-1
ii  libclone-perl  0.38-1
ii  libdpkg-perl   1.18.3
ii  libemail-valid-perl1.198-1
ii  libfile-basedir-perl   0.07-1
ii  libipc-run-perl0.94-1
ii  liblist-moreutils-perl 0.413-1
ii  libparse-debianchangelog-perl  1.2.0-8
ii  libtext-levenshtein-perl   0.13-1
ii  libtimedate-perl   2.3000-2
ii  liburi-perl1.69-1
ii  man-db 2.7.5-1
ii  patchutils 0.3.4-1
ii  perl [libdigest-sha-perl]  5.20.2-6
ii  t1utils1.38-4
ii  xz-utils   5.1.1alpha+20120614-2.1

Versions of packages lintian recommends:
ii  dpkg1.18.3
ii  libperlio-gzip-perl 0.19-1
ii  perl5.20.2-6
ii  perl-modules [libautodie-perl]  5.20.2-6

Versions of packages lintian suggests:
pn  binutils-multiarch 
ii  dpkg-dev   1.18.3
ii  libhtml-parser-perl3.71-2
ii  libtext-template-perl  1.46-1
ii  libyaml-perl   1.15-1

-- no debconf information
--- End Message ---
--- Begin Message ---
Version: 2.2.0

Hi,

> The package has modified files outside the debian/ directory (not tracked
> using patches).

I believe this is direct-changes-in-diff-but-no-patch-system.


Regards,

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

Build failed in Jenkins: lintian-tests_sid #2491

2018-01-28 Thread jenkins 
See 

--
[...truncated 244.45 KB...]
Adding debian:EC-ACC.pem
Adding debian:E-Tugra_Certification_Authority.pem
Adding debian:DigiCert_Trusted_Root_G4.pem
Adding debian:DigiCert_High_Assurance_EV_Root_CA.pem
Adding debian:DigiCert_Global_Root_G3.pem
Adding debian:DigiCert_Global_Root_G2.pem
Adding debian:DigiCert_Global_Root_CA.pem
Adding debian:DigiCert_Assured_ID_Root_G3.pem
Adding debian:DigiCert_Assured_ID_Root_G2.pem
Adding debian:DigiCert_Assured_ID_Root_CA.pem
Adding debian:Deutsche_Telekom_Root_CA_2.pem
Adding debian:DST_Root_CA_X3.pem
Adding debian:DST_ACES_CA_X6.pem
Adding debian:D-TRUST_Root_Class_3_CA_2_EV_2009.pem
Adding debian:D-TRUST_Root_Class_3_CA_2_2009.pem
Adding debian:Cybertrust_Global_Root.pem
Adding debian:Comodo_Trusted_Services_root.pem
Adding debian:Comodo_Secure_Services_root.pem
Adding debian:Comodo_AAA_Services_root.pem
Adding debian:China_Internet_Network_Information_Center_EV_Certificates_Root.pem
Adding debian:Chambers_of_Commerce_Root_-_2008.pem
Adding debian:Certum_Trusted_Network_CA_2.pem
Adding debian:Certum_Trusted_Network_CA.pem
Adding debian:Certum_Root_CA.pem
Adding debian:Certplus_Root_CA_G2.pem
Adding debian:Certplus_Root_CA_G1.pem
Adding debian:Certplus_Class_2_Primary_CA.pem
Adding debian:Certinomis_-_Root_CA.pem
Warning: there was a problem reading the certificate file 
/etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem. Message:
  /etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem (No such file or directory)
Adding debian:Certigna.pem
Adding debian:Camerfirma_Global_Chambersign_Root.pem
Adding debian:Camerfirma_Chambers_of_Commerce_Root.pem
Adding debian:COMODO_RSA_Certification_Authority.pem
Adding debian:COMODO_ECC_Certification_Authority.pem
Adding debian:COMODO_Certification_Authority.pem
Adding debian:CNNIC_ROOT.pem
Adding debian:CFCA_EV_ROOT.pem
Adding debian:CA_Disig_Root_R2.pem
Adding debian:CA_Disig_Root_R1.pem
Adding debian:Buypass_Class_3_Root_CA.pem
Adding debian:Buypass_Class_2_Root_CA.pem
Adding debian:Baltimore_CyberTrust_Root.pem
Adding debian:Autoridad_de_Certificacion_Firmaprofesional_CIF_A62634068.pem
Adding debian:Atos_TrustedRoot_2011.pem
Adding debian:Amazon_Root_CA_4.pem
Adding debian:Amazon_Root_CA_3.pem
Adding debian:Amazon_Root_CA_2.pem
Adding debian:Amazon_Root_CA_1.pem
Adding debian:AffirmTrust_Premium_ECC.pem
Adding debian:AffirmTrust_Premium.pem
Adding debian:AffirmTrust_Networking.pem
Adding debian:AffirmTrust_Commercial.pem
Adding debian:AddTrust_Qualified_Certificates_Root.pem
Adding debian:AddTrust_Public_Services_Root.pem
Adding debian:AddTrust_Low-Value_Services_Root.pem
Adding debian:AddTrust_External_Root.pem
Adding debian:Actalis_Authentication_Root_CA.pem
Adding debian:AC_RAIZ_FNMT-RCM.pem
Adding debian:ACEDICOM_Root.pem
Adding debian:ACCVRAIZ1.pem
done.
Setting up openjdk-8-jdk-headless:amd64 (8u151-b12-1) ...
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/idlj to 
provide /usr/bin/idlj (idlj) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jdeps to 
provide /usr/bin/jdeps (jdeps) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/wsimport to 
provide /usr/bin/wsimport (wsimport) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/rmic to 
provide /usr/bin/rmic (rmic) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jinfo to 
provide /usr/bin/jinfo (jinfo) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jsadebugd to 
provide /usr/bin/jsadebugd (jsadebugd) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/native2ascii 
to provide /usr/bin/native2ascii (native2ascii) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstat to 
provide /usr/bin/jstat (jstat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javac to 
provide /usr/bin/javac (javac) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javah to 
provide /usr/bin/javah (javah) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstack to 
provide /usr/bin/jstack (jstack) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jrunscript to 
provide /usr/bin/jrunscript (jrunscript) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javadoc to 
provide /usr/bin/javadoc (javadoc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jhat to 
provide /usr/bin/jhat (jhat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javap to 
provide /usr/bin/javap (javap) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jar to provide 
/usr/bin/jar (jar) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/xjc to provide 
/usr/bin/x