[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1025355: lintian: check for non-breaking-space chars in name part of Maintainer/Uploaders fields



Package: lintian
Severity: wishlist

The name part of the Maintainer field for the emacs-cmake-mode package
currently contains non-breaking-space characters instead of spaces.
This has been reported against emacs-cmake-mode as bug report #1025354.

This is unlikely to break any Debian package infrastructure, but it
does mean that naive validation of the Maintainer field will error,
for example the DDPO cron job Maintainer check regex uses spaces
and generates a warning mail when the regex does not match.

The Maintainer field and each item of the Uploaders field should be
checked in both source and binary Debian packages.

Please add a check to prevent this from happening in future.

   $ apt-cache showsrc emacs-cmake-mode | sed -n 's/Maintainer: //p' | sort -u | tee /dev/stderr | tr -d -- '-@<>.a-zA-Z ' | xargs -d '\n' unicode --brief
   Debian Emacsen team <debian-emacsen@lists.debian.org>
     U+00A0 NO-BREAK SPACE
     U+00A0 NO-BREAK SPACE
     U+00A0 NO-BREAK SPACE
   
   $ apt-cache show elpa-cmake-mode | sed -n 's/Maintainer: //p' | sort -u | tee /dev/stderr | tr -d -- '-@<>.a-zA-Z ' | xargs -d '\n' unicode --brief
   Debian Emacsen team <debian-emacsen@lists.debian.org>
     U+00A0 NO-BREAK SPACE
     U+00A0 NO-BREAK SPACE
     U+00A0 NO-BREAK SPACE
   
   $ /srv/qa.debian.org/data/cronjobs/ddpo
   /srv/qa.debian.org/ftp/debian/dists/unstable/main/source/Sources.xz:0: syntax error in emacs-cmake-mode Maintainer: Debian Emacsen team <debian-emacsen@lists.debian.org> at ../extract_archive.pl line 101.
   /srv/qa.debian.org/ftp/debian/dists/testing/main/source/Sources.xz:0: syntax error in emacs-cmake-mode Maintainer: Debian Emacsen team <debian-emacsen@lists.debian.org> at ../extract_archive.pl line 101.
   
   $ grep warn.*error /srv/qa.debian.org/data/ddpo/extract_archive.pl 
           $pkgdata{$package}->{maintainer} =~ /(.+) <(.+)>/ or warn "$fname:$.: syntax error in $package Maintainer: $pkgdata{$package}->{maintainer}";
                           $uploader =~ /(.+) <(.+)>/ or warn "$fname:$.: syntax error in $package Uploaders: $uploader";
   
-- 
bye,
pabs

https://wiki.debian.org/PaulWise

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


Reply to: