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

Re: Severity of spelling errors



Russ Allbery wrote:

> Currently, all of the spelling error tags are severity: normal except
> inside binaries:
> 
> changelog-file.desc:Tag: spelling-error-in-changelog
> changelog-file.desc-Severity: normal
> --
> changelog-file.desc:Tag: spelling-error-in-news-debian
> changelog-file.desc-Severity: normal
> --
> copyright-file.desc:Tag: spelling-error-in-copyright
> copyright-file.desc-Severity: normal
> --
> debian-readme.desc:Tag: spelling-error-in-readme-debian
> debian-readme.desc-Severity: normal
> --
> description.desc:Tag: spelling-error-in-description
> description.desc-Severity: normal
> --
> manpages.desc:Tag: manpage-has-spelling-error
> manpages.desc-Severity: normal
> --
> menus.desc:Tag: spelling-error-in-doc-base-title-field
> menus.desc-Severity: normal
> --
> menus.desc:Tag: spelling-error-in-doc-base-abstract-field
> menus.desc-Severity: normal
> 
> However, spelling errors are singled out as an example of a bug that
> should be minor severity by reportbug.  Shouldn't we change those bugs to
> be severity: minor since we're matching Lintian tag severities with bug
> reporting severities?
> 

I just re-checked the severity/certainty->EWI code translation table and
noticed that minor/certain tags are still represented by a warning, so yes,
I think the severity should be updated to reflect reality.

> (BTW, for consistency, I think we should change manpage-has-spelling-error
> to spelling-error-in-manpage.  There are two tag naming conventions that
> we cannot be simultaneously consistent with, but I think the
> spelling-error one is a good one.)
> 

I had a hard time trying to decide which convention I should follow (between
manpage-has-* and spelling-error-in-*). Feel free to rename it to follow
the latter convention.

P.S. have you had time to check some of the infrastructure changes I
proposed?

Cheers,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net



Reply to: