Bug#892905: lintian: pkg-doc examples need not reside under /usr/share/doc/pkg-doc/

2018-03-14 Thread Ferenc Wágner
Package: lintian
Version: 2.5.79
Severity: normal

Dear Maintainer,

Debian Policy Manual §12.3 recommends installing documentation files
(and thus examples) under the main package documentation directory
instead of the documentation directory of the separate -doc package.
However, current Lintian emits the package-does-not-install-examples tag
in such cases.  Please consider relaxing the directory name regex.

Thanks,
Feri.


Bug#891892: lintian: False positive: statically-linked-binary despite the Build-Depends on golang-go

2018-03-14 Thread Arnaud

On 03/14/2018 11:41 AM, Chris Lamb wrote:
> Ah, sorry, I should/could have told you that :(  Sorry for causing you
> extra work..

No worries, I need to practice my debian-fu anyway :)

>
>> let me just clarify that with the go packaging team, and fix the control
>> file to start with.
> Can you clarify? As in, shall we close this bug?

Yep we can close this bug, there's no reason to give `golang-x.x-go` in
the dependencies anyway as far as I know now, and I fixed my packages.

For reference:
http://lists.alioth.debian.org/pipermail/pkg-go-maintainers/Week-of-Mon-20180312/020080.html

Cheers,

  Arnaud



Bug#891892: marked as done (lintian: False positive: statically-linked-binary despite the Build-Depends on golang-go)

2018-03-14 Thread Debian Bug Tracking System
Your message dated Wed, 14 Mar 2018 19:19:09 +0700
with message-id <4552ca4f-4320-c6a6-9462-3dc6ccb6d...@gmail.com>
and subject line 
has caused the Debian Bug report #891892,
regarding lintian: False positive: statically-linked-binary despite the 
Build-Depends on golang-go
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.)


-- 
891892: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=891892
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: lintian
Version: 2.5.67~bpo9+1
Severity: normal

Dear Maintainer,

Package concerned (should be pushed in Debian experimental soon)

https://salsa.debian.org/elboulangero-guest/golang-gogottrpc

Build the package with

gbp clone https://salsa.debian.org/elboulangero-guest/golang-gogottrpc.git
cd golang-gogottrpc
gbp buildpackage --git-pbuilder --git-dist=sid

Then run lintian

lintian ../golang-gogottrpc*changes
E: gogottrpc: statically-linked-binary usr/bin/protoc-gen-gogottrpc
W: gogottrpc: binary-without-manpage usr/bin/protoc-gen-gogottrpc

The statically-linked-binary warninf shouldn't be there, as the field
Build-Depends contains:

golang-go (>= 2:1.9~) | golang-1.9-go

Best regards,

  Arnaud

-- System Information:
Debian Release: 9.3
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-6-amd64 (SMP w/8 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)

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  dpkg  1.18.24
ii  file  1:5.30-1+deb9u1
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  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-3+deb9u2
ii  libtext-levenshtein-perl  0.13-1
ii  libtimedate-perl  2.3000-2
ii  liburi-perl   1.71-1
ii  libxml-simple-perl2.22-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-3+deb9u2
ii  t1utils   1.39-2
ii  xz-utils  5.2.2-1.2+b1

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

Versions of packages lintian suggests:
ii  binutils-multiarch 2.28-5
ii  dpkg-dev   1.18.24
ii  libhtml-parser-perl3.72-3
pn  libtext-template-perl  

-- no debconf information
--- End Message ---
--- Begin Message ---
Not applicable, the Build-Depends line for a go package should be simply
`golang-any (>= 2:1.10~)`.--- End Message ---


Bug#892967: lintian: considers .rtf to be a text file, causing false positives

2018-03-14 Thread Adam Borowski
Package: lintian
Version: 2.5.79
Severity: normal

Hi!
RTF is an old file format from Microsoft, that was pushed as a means of
"cross-platform interchange".  It has never been really widespread, but,
because its simplicity, most Word-like programs support it.  Thus, it's
an odd but not entirely irrational decision to ship some documents this
way.

A package that I'm reviewing includes several RTF files in its documentation
(I've attached one), it triggers a false positive for
license-problem-gfdl-invariants (despite correctly declaring no invariants
nor front or back cover texts).

I guess that RTF files could possibly trigger a number of other false
positives in lintian tests that expect real text.  As adding proper support
for RTF is obviously not worth the effort, it would be nice if you regarded
them to be binary (despite, like .ps, being technically ASCII).


Meow!
-- System Information:
Debian Release: buster/sid
  APT prefers unstable-debug
  APT policy: (500, 'unstable-debug'), (500, 'unstable'), (500, 'testing'), 
(150, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.16.0-rc5-debug-00026-gca682bdc52ca (SMP w/6 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.30-7
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:
pn  binutils-multiarch 
ii  dpkg-dev   1.19.0.5
ii  libhtml-parser-perl3.72-3+b2
pn  libtext-template-perl  

-- no debconf information
{\rtf1\ansi\ansicpg1252\cocoartf102{\fonttbl\f2\fnil Bitstream Charter;\f0\fnil 
Droid Sans Mono;\f1\fnil FreeSans;}
{\colortbl;\red0\green0\blue200;\red0\green0\blue0;\red255\green0\blue0;}
{{\NeXTGraphic iconoGimp3.tif \width1816 
\height1309}\pard\tx560\tx1120\tx1680\tx2240\tx2800\tx3360\tx3920\tx4480\tx5040\tx5600\tx6160\tx6720\ql\f0\fs24
 \'AC}\f1\fs48  \b\cf1 \'A1Welcome to FisicaLab!\cf0\fs24\b0\cf2 \par
\par
\fs20\i Copyright (C)  2009, 2010, 2012  German A. Arias.\par
Permission is granted to copy, distribute and/or modify this document\par
under the terms of the GNU Free Documentation License, Version 1.3\par
or any later version published by the Free Software Foundation;\par
with no Invariant Sections, no Front-Cover Texts, and no Back-Cover 
Texts.\par
A copy of the license is included in the section entitled "GNU\par
Free Documentation License".\fs24\i0 \par
\par
\par
FisicaLab is an educational application to solve physics problems. Its main 
objective is let the user to focus in physics concepts, leaving aside the 
mathematical details (FisicaLab take care of them). This allows the user to 
become familiar with the physical concepts without running the risk of getting 
lost in mathematical details. And so, when the user gain confidence in applying 
physical concepts, will be better prepared to solve the problems by hand (with 
pen and paper). FisicaLab is easy to use and very intuitive. However, in order 
to take advantage of all its features, we recommend you read first these help 
files.\par
\par
FisicaLab display to two windows, one named \b Modules and elements\b0  and 
other named \b Chalkboard\b0 . The first of these windows, contain all modules 
that can be used to solve problems. These modules are grouped by: kinematics, 
static, dynamics, ... (see image below). You can select one of these groups 
with the buttons at the top of the window, marked with (1) in the image. When 
you leave the mouse\rquote s cursor above one of these buttons, a 

Bug#892905: lintian: pkg-doc examples need not reside under /usr/share/doc/pkg-doc/

2018-03-14 Thread Chris Lamb
tags 892905 + moreinfo
thanks

Hi Ferenc,

> Please consider relaxing the directory name regex.

We currently look for paths called "examples" under usr/share/doc/$pkg.
What are you suggesting we look for instead? :)

(The path must surely include "examples" somewhere, otherwise we can't
distinguish them between that and "regular" documentation.)


Regards,

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



Processed: Re: Bug#892905: lintian: pkg-doc examples need not reside under /usr/share/doc/pkg-doc/

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

> tags 892905 + moreinfo
Bug #892905 [lintian] lintian: pkg-doc examples need not reside under 
/usr/share/doc/pkg-doc/
Added tag(s) moreinfo.
> thanks
Stopping processing here.

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



[lintian] branch master updated (b880cc5 -> 9dd5cd5)

2018-03-14 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  b880cc5   Drop apache2-module-depends-on-real-apache2-package tag 
as there are separate tags for missing apache2-api-* dependencies. Thanks to 
Thijs Kinkhorst et al. (Closes: #796285)
   new  9dd5cd5   Strip \par elements from files prior to license checks to 
avoid false-positives when checking .rtf files. Thanks to Adam Borowski for the 
report and testcase. (Closes: #892967)

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 +
 debian/changelog   |   4 +
 .../debian/src/false-positive.rtf  | 125 +
 3 files changed, 132 insertions(+)
 create mode 100644 t/tests/cruft-gfdl-invariants/debian/src/false-positive.rtf

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



[lintian] 01/01: Strip \par elements from files prior to license checks to avoid false-positives when checking .rtf files. Thanks to Adam Borowski for the report and testcase. (Closes: #892967)

2018-03-14 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 9dd5cd5f7484f30026d1f63e362e13903a678ac1
Author: Chris Lamb 
Date:   Wed Mar 14 19:36:08 2018 -0700

Strip \par elements from files prior to license checks to avoid 
false-positives when checking .rtf files. Thanks to Adam Borowski for the 
report and testcase. (Closes: #892967)
---
 checks/cruft.pm|   3 +
 debian/changelog   |   4 +
 .../debian/src/false-positive.rtf  | 125 +
 3 files changed, 132 insertions(+)

diff --git a/checks/cruft.pm b/checks/cruft.pm
index 9406ee0..2409fe3 100644
--- a/checks/cruft.pm
+++ b/checks/cruft.pm
@@ -1363,6 +1363,9 @@ sub _clean_block {
 # po comment may include html tag
 $text =~ s/\"\s?\v\#~\s?\"//gxms;
 
+# strip .rtf paragraph marks (#892967)
+$text =~ s/\\par\b//gxms;
+
 $text =~ s/\\url[{][^}]*?[}]/ /gxms;  # (la)?tex url
 $text =~ s/\\emph[{]/ /gxms;  # (la)?tex emph
 $text =~ s/\\href[{][^}]*?[}]
diff --git a/debian/changelog b/debian/changelog
index b7791c9..e20e2ed 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -6,6 +6,10 @@ lintian (2.5.80) UNRELEASED; urgency=medium
 + [CL] Drop apache2-module-depends-on-real-apache2-package tag as there
   are separate tags for missing apache2-api-* dependencies.  Thanks to
   Thijs Kinkhorst et al.  (Closes: #796285)
+  * checks/cruft.pm:
++ [CL] Strip \par elements from files prior to license checks to
+  avoid false-positives when checking .rtf files.  Thanks to Adam
+  Borowski for the report and testcase.  (Closes: #892967)
   * checks/fields.pm:
 + [CL] Only check dependency fields in binary packages for
   mail-transport-agent-dependency-does-not-specify-default-mta etc.
diff --git a/t/tests/cruft-gfdl-invariants/debian/src/false-positive.rtf 
b/t/tests/cruft-gfdl-invariants/debian/src/false-positive.rtf
new file mode 100644
index 000..19560b4
--- /dev/null
+++ b/t/tests/cruft-gfdl-invariants/debian/src/false-positive.rtf
@@ -0,0 +1,125 @@
+{\rtf1\ansi\ansicpg1252\cocoartf102{\fonttbl\f2\fnil Bitstream 
Charter;\f0\fnil Droid Sans Mono;\f1\fnil FreeSans;}
+{\colortbl;\red0\green0\blue200;\red0\green0\blue0;\red255\green0\blue0;}
+{{\NeXTGraphic iconoGimp3.tif \width1816 
\height1309}\pard\tx560\tx1120\tx1680\tx2240\tx2800\tx3360\tx3920\tx4480\tx5040\tx5600\tx6160\tx6720\ql\f0\fs24
 \'AC}\f1\fs48  \b\cf1 \'A1Welcome to FisicaLab!\cf0\fs24\b0\cf2 \par
+\par
+\fs20\i Copyright (C)  2009, 2010, 2012  German A. Arias.\par
+Permission is granted to copy, distribute and/or modify this document\par
+under the terms of the GNU Free Documentation License, Version 1.3\par
+or any later version published by the Free Software Foundation;\par
+with no Invariant Sections, no Front-Cover Texts, and no Back-Cover 
Texts.\par
+A copy of the license is included in the section entitled "GNU\par
+Free Documentation License".\fs24\i0 \par
+\par
+\par
+FisicaLab is an educational application to solve physics problems. Its main 
objective is let the user to focus in physics concepts, leaving aside the 
mathematical details (FisicaLab take care of them). This allows the user to 
become familiar with the physical concepts without running the risk of getting 
lost in mathematical details. And so, when the user gain confidence in applying 
physical concepts, will be better prepared to solve the problems by hand (with 
pen and paper). FisicaLab is [...]
+\par
+FisicaLab display to two windows, one named \b Modules and elements\b0  and 
other named \b Chalkboard\b0 . The first of these windows, contain all modules 
that can be used to solve problems. These modules are grouped by: kinematics, 
static, dynamics, ... (see image below). You can select one of these groups 
with the buttons at the top of the window, marked with (1) in the image. When 
you leave the mouse\rquote s cursor above one of these buttons, a label with 
the group name is displayed. [...]
+\par
+\cf0\f0{{\NeXTGraphic FisicaLabPanel.jpg \width7680 
\height10760}\'AC}\f2\cf2 \par
+\pard\ql\pard\tx0\tx560\tx1120\tx1680\tx2240\tx2800\tx3360\tx3920\tx4480\tx5040\tx5600\tx6160\ql
 \par
+\f1 The window named \b Chalkboard\b0  (see image below), have at the top two 
buttons, marked with (7). The button at the left let you solve the problem, and 
the other is to clean the chalkboard. The black box marked with (8) is the 
chalkboard, where you add the elements to set the problems. You need keep in 
mind, although you can\rquote t see, that the chalkboard is a grid formed with 
cells of 50x50 pixels. By default the chalkboard size is 26x18 cells. In \b 
Preferences\b0  panel you c [...]
+\par
+\pard\ql\pard\tx560\tx1120\tx1680\tx2240\tx2800\tx3360\tx3920\tx4480\tx5040\tx5600\tx6160\tx6720\ql
 \cf0\f0{{\NeXTGraphic Fisi

Bug#892967: lintian: considers .rtf to be a text file, causing false positives

2018-03-14 Thread Chris Lamb
tags 892967 + pending
thanks

> […]

So, turns out we can actually fix / "parse" .rtf files quite easily:

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

.. so no need to ignore them just yet. :)


Best wishes,

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



Processed: Re: lintian: considers .rtf to be a text file, causing false positives

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

> tags 892967 + pending
Bug #892967 [lintian] lintian: considers .rtf to be a text file, causing false 
positives
Added tag(s) pending.
> thanks
Stopping processing here.

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



lintian.debian.org not updating

2018-03-14 Thread Scott Talbert

Hi,

I'm assuming you are aware already, but it seems that lintian.debian.org 
is not performing periodic scans.  It appears to have not run since around 
February 20.


Scott