Bug#907725: xfonts packages are not using the x11 section in practice

[Russ Allbery]

Chris Lamb  writes:

> Hi Russ,

>> […] standardize the other direction and fix packages.debian.org to remove
>> "fonts" from the definition of the x11 section.  I think this also fits the
>> general classification rubric of preferring the narrower, less general
>> section when a package could fall into multiple sections.

> Sounds good to me.
>  
>> (I'm not sure where to report the packages.debian.org bug.  Do you know?)

> Against the "www.debian.org" pseudo-package.

Thanks, filed as #907782.

-- 
Russ Allbery (r...@debian.org)   

lintian_2.5.99~bpo9+1_amd64.changes ACCEPTED into stretch-backports

[Debian FTP Masters]

Accepted:

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

Format: 1.8
Date: Sat, 01 Sep 2018 20:35:38 +0100
Source: lintian
Binary: lintian
Built-For-Profiles: nocheck
Architecture: source all
Version: 2.5.99~bpo9+1
Distribution: stretch-backports
Urgency: medium
Maintainer: Debian Lintian Maintainers 
Changed-By: Chris Lamb 
Description:
 lintian- Debian package checker
Closes: 907475
Changes:
 lintian (2.5.99~bpo9+1) stretch-backports; urgency=medium
 .
   * Rebuild for stretch-backports.
 .
 lintian (2.5.99) unstable; urgency=medium
 .
   * checks/crufts.desc:
 + [FL] Improve tag description for 'package-does-not-install-examples'.
   * checks/source-copyright.desc:
 + [CL] Correct "may be" typo in the long description for the
   'incomplete-creative-commons-license' tag.
 .
   * data/cruft/warn-file-type:
 + [CL] Match (for example) foo_o.golden with foo.cc to avoid
   source-is-missing false-positives.  (Closes: #907475)
 .
   * t/tests/changes:
 + [FL] Place all tests for the 'changes' suite into their own
   directories.
Checksums-Sha1:
 5b46178163b7c258ad433c5fc6c80ae51746d8f2 3539 lintian_2.5.99~bpo9+1.dsc
 1b1b70a45b3a36644b484015275d8ede2f1e7c0b 1583528 lintian_2.5.99~bpo9+1.tar.xz
 157ad1f5e9de44ab876a75e5d0fe24ba3006d49f 1134366 lintian_2.5.99~bpo9+1_all.deb
 41a679b55a56c2dd34c78af3dcbbe37c97d97059 8650 
lintian_2.5.99~bpo9+1_amd64.buildinfo
Checksums-Sha256:
 f74502426996b215bb517d483c623f2ceb43fc1725b5cca8998c1a41097bcc15 3539 
lintian_2.5.99~bpo9+1.dsc
 b9a8b785c728f451a6b53c22ca0b08527ff3eb94f14f79a060b276932bebee0f 1583528 
lintian_2.5.99~bpo9+1.tar.xz
 56caf9a65b38560a9d2922bb4456f783160a651d176d8246bb5c71e2922dc54d 1134366 
lintian_2.5.99~bpo9+1_all.deb
 0f1367fb3e04657e5e3cd710d45cc5f68f7dd14c6d800fe370a980930da7f75c 8650 
lintian_2.5.99~bpo9+1_amd64.buildinfo
Files:
 e7e1a3a167890c482cff2eedbab59bf4 3539 devel optional lintian_2.5.99~bpo9+1.dsc
 21dd759c80edcfa9fb5f4d75e16ada21 1583528 devel optional 
lintian_2.5.99~bpo9+1.tar.xz
 40b351a612ea6fcfe2cb4be3026d662f 1134366 devel optional 
lintian_2.5.99~bpo9+1_all.deb
 230f1cdecb3fd1581648eac7dcf6440b 8650 devel optional 
lintian_2.5.99~bpo9+1_amd64.buildinfo

-BEGIN PGP SIGNATURE-

iQIzBAEBCAAdFiEEwv5L0nHBObhsUz5GHpU+J9QxHlgFAluK7ccACgkQHpU+J9Qx
HliLGxAAsmKFvNo9nBaS40rKpV7IAVIqpm/+npX+Eyphdbe3Md1TNt9gm2KG0/Rm
ZIVQmRCz5X0uM1+AGCMZ4Wr9QO/R+7phviTdzXfBg0NM/sD6fCjEz9i7Qm+fw/1I
rgwp8CnJEjZQQT4kXnF3WteQRy0j0DMvRL6oc1Q7YOn2eXAIJNmi2PjLuSTTF7pW
VPNotg7fL/TbIcXiBoIbK90ck2JGaFQIXz6Cd7y34ipbywz6xBhRV2w8SO1vxA8d
9z4IgGAASigFmLdbs6rlU+Sfnrc4TMRRdsR2P0zyAeStvi61b7I5J2DmFaWIrwts
q6uE5mnLRJ8+XtoIpGBsDs6PNBkXTChUDQbbLkIMRozI2pN/2fjqyjVqanayA6K9
65d5kUVx9w47+csSzjM82lrANP1nx+VpweZlAfq99pBvZIbjxcHEHzhAZuQAaWRO
Riji1Pp//jTPGxTZCgYVwICCpHASUH7G9U4ohWqlyylSZT84ybgyiwwwdCoMOfKF
Q7oQvWAADBGuFWs6Ky/Rbb6bAuu9SgqDKfvpDJ4tc8asyJJx876lqGIrHwQy4I1X
Lkl4DIb5P2A6eEGcDONm2jq2yUGFdMuHJbONRAJ7hNTwnwignFXPz4lJx32gUYMA
HIvT0D9t83QbE3kV9kq1HCDjjKjm4RD5+Vf1QAh+3HdjFrhLMIU=
=kjso
-END PGP SIGNATURE-


Thank you for your contribution to Debian.

Processing of lintian_2.5.99~bpo9+1_amd64.changes

[Debian FTP Masters]

lintian_2.5.99~bpo9+1_amd64.changes uploaded successfully to localhost
along with the files:
  lintian_2.5.99~bpo9+1.dsc
  lintian_2.5.99~bpo9+1.tar.xz
  lintian_2.5.99~bpo9+1_all.deb
  lintian_2.5.99~bpo9+1_amd64.buildinfo

Greetings,

Your Debian queue daemon (running on host usper.debian.org)

Bug#907725: xfonts packages are not using the x11 section in practice

[Chris Lamb]

Hi Russ,

> […] standardize the other direction and fix packages.debian.org to remove
> "fonts" from the definition of the x11 section.  I think this also fits the
> general classification rubric of preferring the narrower, less general
> section when a package could fall into multiple sections.

Sounds good to me.
 
> (I'm not sure where to report the packages.debian.org bug.  Do you know?)

Against the "www.debian.org" pseudo-package.


Regards,

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

Bug#907667: lintian: should html escape output if --color=html is used

[Chris Lamb]

Hi Niels,

> Though, reminder - if you introduce a new dependency, you will have to
> get DSA to install it on lindsay.d.o before you can upgrade lintian there.

(Oh, I forgot to mention; it's already installed on lindsay)


Regards,

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

Bug#907423: lintian should give an error when both foo-dbg and foo-dbgsym are built

[Adrian Bunk]

On Mon, Aug 27, 2018 at 09:18:43PM +0100, Chris Lamb wrote:
> Hi Adrian,
> 
> > lintian should give an error when both foo-dbg and foo-dbgsym are
> > built
> 
> Could you provide another draft description? Thank you in advance.

Both -dbg and -dbgsym are built for this package.
Only one should be built for the debug symbols.

Usually the -dbg should be dropped in favour of the -dbgsym.

In some cases (e.g. for Python modules) the -dbg contains more than just 
the debug symbols. In these cases the -dbgsym should not be built.

> Regards,

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

Bug#907667: lintian: should html escape output if --color=html is used

[Niels Thykier]

Chris Lamb:
> Hi Niels,
> 
>> Any reason for introducing the CGI dependency over simply applying the
>> same escape rules for the $information variable?
> 
> Only because well-used libraries are preferred, particularly for data
> sanitisation (!) operations.
> 
> Is the extra dependency problematic? We use some far-more esoteric
> libraries than CGI, so I did not think it would be an issue.
> 
> 
> Regards,
> 

If we are consistent with how we perform the quoting, I do not mind the
extra dependency.  Particularly because it should be doable to reduce it
to a suggests given --color=html is not a default (which we can add
later if relevant).

Though, reminder - if you introduce a new dependency, you will have to
get DSA to install it on lindsay.d.o before you can upgrade lintian there.

Thanks,
~Niels

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

[jenkins]

See 

Bug#907667: lintian: should html escape output if --color=html is used

[Chris Lamb]

Hi Niels,

> Any reason for introducing the CGI dependency over simply applying the
> same escape rules for the $information variable?

Only because well-used libraries are preferred, particularly for data
sanitisation (!) operations.

Is the extra dependency problematic? We use some far-more esoteric
libraries than CGI, so I did not think it would be an issue.


Regards,

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

Bug#907667: lintian: should html escape output if --color=html is used

[Niels Thykier]

Chris Lamb:
> tags 907667 + pending
> thanks
> 
> Fixed in Git, pending upload:
> 
>   
> https://salsa.debian.org/lintian/lintian/commit/897c485d61387adc5689f287c7e0404e604136e7
> 
>   debian/changelog  | 5 +
>   debian/control| 2 ++
>   lib/Lintian/Output.pm | 7 +++
>   t/tests/lintian-color-html/debian/debian/docs | 1 +
>   t/tests/lintian-color-html/debian/foo.xml | 1 +
>   t/tests/lintian-color-html/desc   | 8 
>   t/tests/lintian-color-html/tags   | 1 +
>   7 files changed, 21 insertions(+), 4 deletions(-)
> 
> 
> Regards,
> 


Any reason for introducing the CGI dependency over simply applying the
same escape rules for the $information variable?  Possibly we could
extract the html_quote from commands/reporting-html-reports.pm and put
it in L::Util (or similar) and share the code from there.

Alternatively, if we are moving to a dependency to solve this issue,
then we should use it consistently (i.e. remove html_quote from
commands/reporting-html-reports.pm).

Thanks,
~Niels

Build failed in Jenkins: lintian-tests_sid #3183

[jenkins]

See 


Changes:

[lamby] Escape output if --color=html is used. (Closes: #907667)

--
[...truncated 137.37 KB...]
Setting up libfile-listing-perl (6.04-1) ...
Setting up libhttp-message-perl (6.18-1) ...
Setting up g++ (4:8.2.0-1) ...
update-alternatives: using /usr/bin/g++ to provide /usr/bin/c++ (c++) in auto 
mode
Setting up libparams-classify-perl (0.015-1) ...
Setting up git (1:2.19.0~rc1-1) ...
Setting up libhttp-negotiate-perl (6.01-1) ...
Setting up libmailtools-perl (2.18-1) ...
Setting up libgit-wrapper-perl (0.048-1) ...
Setting up libhtml-format-perl (2.12-1) ...
Setting up libemail-valid-perl (1.202-1) ...
Setting up libhttp-cookies-perl (6.04-1) ...
Setting up curl (7.61.0-1) ...
Setting up build-essential (12.5) ...
Setting up libmodule-runtime-perl (0.016-1) ...
Setting up libhttp-daemon-perl (6.01-1) ...
Setting up libhtml-form-perl (6.03-1) ...
Setting up libmime-tools-perl (5.509-1) ...
Setting up libimport-into-perl (1.002005-1) ...
Setting up libmodule-implementation-perl (0.09-1) ...
Setting up libparams-validate-perl (1.29-1) ...
Setting up libmoo-perl (2.003004-1) ...
Setting up libb-hooks-endofscope-perl (0.24-1) ...
Setting up libpackage-stash-perl (0.37-1) ...
Setting up libgetopt-long-descriptive-perl (0.103-2) ...
Setting up libnamespace-clean-perl (0.27-1) ...
Setting up licensecheck (3.0.31-2) ...
Setting up dh-autoreconf (19) ...
Setting up libicu-le-hb0:amd64 (1.0.3+git161113-5) ...
Setting up libwww-perl (6.35-2) ...
Setting up libicu60:amd64 (60.2-6) ...
Setting up liblwp-protocol-https-perl (6.07-2) ...
Setting up libxml2:amd64 (2.9.4+dfsg1-7+b1) ...
Setting up libcroco3:amd64 (0.6.12-2) ...
Setting up libxml-parser-perl (2.44-2+b3) ...
Setting up shared-mime-info (1.9-2) ...
Setting up libxml-libxml-perl (2.0132+dfsg-2) ...
update-perl-sax-parsers: Registering Perl SAX parser XML::LibXML::SAX::Parser 
with priority 50...
update-perl-sax-parsers: Registering Perl SAX parser XML::LibXML::SAX with 
priority 50...
update-perl-sax-parsers: Updating overall Perl SAX parser modules info file...
Replacing config file /etc/perl/XML/SAX/ParserDetails.ini with new version
Setting up libxml-sax-expat-perl (0.51-1) ...
update-perl-sax-parsers: Registering Perl SAX parser XML::SAX::Expat with 
priority 50...
update-perl-sax-parsers: Updating overall Perl SAX parser modules info file...
Replacing config file /etc/perl/XML/SAX/ParserDetails.ini with new version
Setting up gettext (0.19.8.1-7) ...
Setting up libsoap-lite-perl (1.27-1) ...
Setting up intltool-debian (0.35.0+20060710.4) ...
Setting up libxml-simple-perl (2.25-1) ...
Setting up libxmlrpc-lite-perl (0.717-1) ...
Setting up lintian (2.5.99) ...
Setting up po-debconf (1.0.20) ...
Setting up debhelper (11.3.5) ...
Setting up equivs (2.1.0) ...
Setting up dh-strip-nondeterminism (0.042-1) ...
Processing triggers for libc-bin (2.27-5) ...
Processing triggers for ca-certificates (20180409) ...
Updating certificates in /etc/ssl/certs...
0 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
done.
Processing triggers for systemd (239-7) ...
Source: lintian
Section: devel
Priority: optional
Maintainer: Debian Lintian Maintainers 
Uploaders: Josip Rodin ,
   Colin Watson ,
   Russ Allbery ,
   Adam D. Barratt ,
   Raphael Geissert ,
   Niels Thykier ,
   Bastien Roucariès ,
   Axel Beckert ,
   Chris Lamb ,
Build-Depends: aspell ,
   aspell-en ,
   binutils ,
   bzip2 ,
   cdbs ,
   debhelper (>= 11~),
# Prefer the headless variant as it saves about 25% (installed-size), but
# it is not available in Jessie.
   default-jdk-headless  | default-jdk ,
# Use bash as a Jessie indicator (dh-elpa is only used for a test)
   dh-elpa  | bash (<< 4.4) ,
   dh-python ,
   diffstat ,
   docbook-utils,
   docbook-xml,
   dpkg (>= 1.17.14),
   fakeroot ,
   file ,
   gettext ,
   intltool-debian ,
   javahelper ,
   libapt-pkg-perl ,
   libarchive-zip-perl ,
   libcgi-pm-perl 
   libclass-accessor-perl ,
   libclone-perl ,
   libdpkg-perl ,
   libdigest-sha-perl ,
   libemail-valid-perl ,
   libfile-basedir-perl ,
   libhtml-parser-perl ,
   libipc-run-perl ,
   libio-async-perl ,
   liblist-moreutils-perl ,
   libparse-debianchangelog-perl ,
   libtest-minimumversion-perl ,
   libtest-perl-critic-perl ,
   libtest-pod-coverage-perl ,
   libtest-pod-perl ,
   libtest-spelling-perl ,
   libtest-strict-perl 

Bug#907667: lintian: should html escape output if --color=html is used

[Chris Lamb]

tags 907667 + pending
thanks

Fixed in Git, pending upload:

  
https://salsa.debian.org/lintian/lintian/commit/897c485d61387adc5689f287c7e0404e604136e7

  debian/changelog  | 5 +
  debian/control| 2 ++
  lib/Lintian/Output.pm | 7 +++
  t/tests/lintian-color-html/debian/debian/docs | 1 +
  t/tests/lintian-color-html/debian/foo.xml | 1 +
  t/tests/lintian-color-html/desc   | 8 
  t/tests/lintian-color-html/tags   | 1 +
  7 files changed, 21 insertions(+), 4 deletions(-)


Regards,

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

Processed: Re: lintian: should html escape output if --color=html is used

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> tags 907667 + pending
Bug #907667 [lintian] lintian: should html escape output if --color=html is used
Added tag(s) pending.
> thanks
Stopping processing here.

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