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

Bug#1025354: emacs-cmake-mode: non-breaking-space chars in name part of Maintainer field



Source: emacs-cmake-mode
Version: 3.25.0+ds-2
Severity: important

The name part of the Maintainer field for the emacs-cmake-mode package
currently contains non-breaking-space characters instead of spaces.

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.

Please fix the space characters so QA folks no longer get cron mail.

$ 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: