Bug#966295: lintian: False positive with bogus email address
Control: found -1 2.104.0
Hi,
Eriberto Mota wrote:
> lintian produces a false positive in mmorph package. I think the problem is a
> comma used by an old maintainer.
>
> E: mmorph: bogus-mail-host-in-debian-changelog Thomas Bushnell, BSG <tb@becket.net>
Just ran into it, too, with the package screen:
E: screen: bogus-mail-host-in-debian-changelog Eric Gillespie, Jr. <epg@debian.org>
Anthony Fok wrote:
> I ran into the same problem with the lilypond package too, though
> after adding double quotes around the display name of the email
> address like so:
>
> "Thomas Bushnell, BSG" <tb@debian.org>
>
> the Lintian errors went away.
Yes, but since
a) this is emitted for ancient changelog entries which never were a
problem, and
b) there is no support in the debian/changelog for having more than
one e-mail address in the changelog entry signature line,
I consider this really to be a false positive.
Will override it in the screen package until this is fixed in Lintian.
> This makes sense to me because the comma is used for separating
> multiple email recipients, so Lintian is right in splitting it up at
> the comma, i.e. trying to parse the string "Thomas Busnell" as an
> email address, and "BSG <tb@debian.org>" as another.
Yeah, but it does not make sense at the point where lintian has found
that address as there are no multiple addresses allowed in a
debian/changelog signature:
https://www.debian.org/doc/debian-policy/ch-source.html#debian-changelog-debian-changelog
Actually, there are _no_ restrictions at all in the Debian Policy how
how a "maintainer name" must look like.
So after now having read the according part in the Debian Policy, for
me this is very clearly a false positive.
IMHO lintian should only check for valid e-mail address syntax between
"<" and ">", otherwise it does not adhere to the Debian Policy.
Regards, Axel
--
,''`. | Axel Beckert <abe@debian.org>, https://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
`- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
Reply to: