Processed: Re: Bug#975690: lintian: detect invalid Uploaders fields that are missing separating commas

2020-11-25 Thread Debian Bug Tracking System
Processing control commands:

> block -1 by 509935
Bug #975690 [lintian] lintian: detect invalid Uploaders fields that are missing 
separating commas
975690 was not blocked by any bugs.
975690 was not blocking any bugs.
Added blocking bug(s) of 975690: 401452 and 509935

-- 
975690: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=975690
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#975690: lintian: detect invalid Uploaders fields that are missing separating commas

2020-11-25 Thread Felix Lechner
Control: block -1 by 509935

Hi.

On Tue, Nov 24, 2020 at 10:51 PM Paul Wise  wrote:
>
> Please detect this and emit an error about it, probably it should also
> get onto the ftp-master lintian reject list.

Lintian uses the standard Perl module Email::Address::XS [1] to parse
all address fields. (There are others, but this was most capable in
parsing Unicode "phrases".) The module states that it "parses input
string[s] sequentially according to RFC 5322 grammar." In the given
package, the last two uploaders are simply discarded:

$ bin/lintian -T mail-contact -T trimmed-field --no-tag-display-limit
/mirror/debian/pool/main/y/yubico-piv-tool/yubico-piv-tool_2.1.1-2.dsc
C: yubico-piv-tool source: mail-contact Maintainer "Debian
Authentication Maintainers"

C: yubico-piv-tool source: mail-contact Uploaders "Alessio Di Mauro"

C: yubico-piv-tool source: mail-contact Uploaders "Dain Nilsson"

C: yubico-piv-tool source: trimmed-field Maintainer Debian
Authentication Maintainers

C: yubico-piv-tool source: trimmed-field Uploaders Alessio Di Mauro
, Dain Nilsson  Klas Lindfors
, nicoo ,
[some output omitted]

Perhaps the RFC 5322 grammar is too lenient with respect to errors,
and focuses instead on sending to at least some recipients.

Kind regards,
Felix Lechner

[1] https://metacpan.org/pod/Email::Address::XS#parse



Bug#975690: lintian: detect invalid Uploaders fields that are missing separating commas

2020-11-24 Thread Stuart Prescott
I think that Debian needs to know what the format of Uploaders is supposed to 
be, before it is reasonable to hope that lintian can check that it is correct. 
(well ok, policy often works the other way around, but there needs to be the 
rough consensus first rather than lintian driving policy)

Perhaps there is a rough consensus in these discussions so far:

https://bugs.debian.org/401452
https://bugs.debian.org/509935
https://bugs.debian.org/962277

cheers
Stuart
(who would welcome a resolution too: see https://bugs.debian.org/686638)

-- 
Stuart Prescotthttp://www.nanonanonano.net/   stu...@nanonanonano.net
Debian Developer   http://www.debian.org/ stu...@debian.org
GPG fingerprint90E2 D2C1 AD14 6A1B 7EBB 891D BBC1 7EBB 1396 F2F7



Bug#975690: lintian: detect invalid Uploaders fields that are missing separating commas

2020-11-24 Thread Paul Wise
Package: lintian
Version: 2.103.0
Severity: wishlist
The yubico-piv-tool package recently introduced an invalid Uploaders
fields that is missing a single comma in the middle of the list.

   $ apt-cache showsrc yubico-piv-tool | grep -E '^$|^Version|^Uploaders'
   Version: 2.0.0-2
   Uploaders: nicoo , Alessio Di Mauro , 
Klas Lindfors , Dain Nilsson 
   
   Version: 2.1.1-1
   Uploaders: Alessio Di Mauro , Dain Nilsson 
 Klas Lindfors , nicoo ,
   
   Version: 2.1.1-2
   Uploaders: Alessio Di Mauro , Dain Nilsson 
 Klas Lindfors , nicoo ,

 ^

This is a violation of Debian Policy 5.6.3:

   https://www.debian.org/doc/debian-policy/ch-controlfields.html#uploaders
   
   List of the names and email addresses of co-maintainers of the package,
   if any. The format of each entry is the same as that of the Maintainer
   field, and multiple entries *must be comma separated*.

Please detect this and emit an error about it, probably it should also
get onto the ftp-master lintian reject list.

-- System Information:
Debian Release: bullseye/sid
  APT prefers testing-debug
  APT policy: (900, 'testing-debug'), (900, 'testing'), (800, 
'unstable-debug'), (800, 'unstable'), (790, 'buildd-unstable'), (700, 
'experimental-debug'), (700, 'experimental'), (690, 'buildd-experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 5.9.0-2-amd64 (SMP w/4 CPU threads)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8), LANGUAGE=en_AU:en
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages lintian depends on:
ii  binutils    2.35.1-2
ii  bzip2   1.0.8-4
ii  diffstat    1.63-1
ii  dpkg    1.20.5
ii  dpkg-dev    1.20.5
ii  file    1:5.38-5
ii  gettext 0.19.8.1-10
ii  gpg 2.2.20-1
ii  intltool-debian 0.35.0+20060710.5
ii  libapt-pkg-perl 0.1.36+b4
ii  libarchive-zip-perl 1.68-1
ii  libcapture-tiny-perl    0.48-1
ii  libclass-xsaccessor-perl    1.19-3+b6
ii  libclone-perl   0.45-1+b1
ii  libconfig-tiny-perl 2.24-1
ii  libcpanel-json-xs-perl  4.25-1+b1
ii  libdata-dpath-perl  0.58-1
ii  libdata-validate-domain-perl    0.10-1
ii  libdevel-size-perl  0.83-1+b2
ii  libdpkg-perl    1.20.5
ii  libemail-address-xs-perl    1.04-1+b3
ii  libfile-basedir-perl    0.08-1
ii  libfile-find-rule-perl  0.34-1
ii  libfont-ttf-perl    1.06-1
ii  libhtml-html5-entities-perl 0.004-1
ii  libipc-run3-perl    0.048-2
ii  libjson-maybexs-perl    1.004003-1
ii  liblist-compare-perl    0.55-1
ii  liblist-moreutils-perl  0.416-1+b6
ii  liblist-utilsby-perl    0.11-1
ii  libmoo-perl 2.004000-1
ii  libmoox-aliases-perl    0.001006-1
ii  libnamespace-clean-perl 0.27-1
ii  libpath-tiny-perl   0.114-1
ii  libperlio-gzip-perl 0.19-1+b7
ii  libproc-processtable-perl   0.59-2+b1
ii  libsereal-decoder-perl  4.018+ds-1+b1
ii  libsereal-encoder-perl  4.018+ds-1+b1
ii  libtext-glob-perl   0.11-1
ii  libtext-levenshteinxs-perl  0.03-4+b8
ii  libtext-markdown-discount-perl  0.12-1+b1
ii  libtext-xslate-perl 3.5.8-1+b1
ii  libtime-duration-perl   1.21-1
ii  libtime-moment-perl 0.44-1+b3
ii  libtimedate-perl    2.3300-1
ii  libtry-tiny-perl    0.30-1
ii  libtype-tiny-perl   1.012000-1
ii  libunicode-utf8-perl    0.62-1+b2
ii  liburi-perl 5.05-1
ii  libxml-libxml-perl  2.0134+dfsg-2+b1
ii  libyaml-libyaml-perl    0.82+repack-1+b1
ii  lzip    1.21-8
ii  lzop    1.04-2
ii  man-db  2.9.3-2
ii  patchutils  0.4.2-1
ii  perl [libdigest-sha-perl]   5.32.0-5
ii  t1utils 1.41-4
ii  unzip   6.0-25
ii  xz-utils    5.2.4-1+b1

lintian recommends no packages.

Versions of packages lintian suggests:
ii  binutils-multiarch 2.35.1-2
ii  libtext-template-perl  1.59-1

-- no debconf information

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part