Am Samstag, den 12.11.2011, 18:43 -0800 schrieb Steve Langasek: > On Sat, Nov 12, 2011 at 04:04:35PM -0800, Evan Broder wrote: > > On Sat, Nov 12, 2011 at 2:01 PM, Benjamin Drung <bdr...@ubuntu.com> wrote: > > > Am Dienstag, den 08.11.2011, 17:38 -0800 schrieb Evan Broder: > > >> If there is more that lintian.uw.o could be doing for you, please let me > > >> know! > > > > Thanks for setting this service up. > > > > I compared my profiles [1] and [2] and discovered false positive > > > results: debian-news-entry-has-unknown-version and > > > non-standard-file-perm > > > Thanks for pointing these out. I believe there is an explanation for > > both of them; adjusting to correct for it may be difficult, though. > > > > debian-news-entry-has-unknown-version complains about normal Debian > > > changelog entry versions. > > > debian-news-entry-has-unknown-version is a check on the binary > > package, so it's comparing those version numbers against the changelog > > in the binary package. Starting with natty, pkgbinarymangler strips > > all but the 10 most recent changelog entries. In all cases, the > > versions lintian is complaining about are not included in the shrunk > > changelog. > > > I'm not sure what to do with that - maybe pkgbinarymangler should > > remove NEWS entries not included in the shrunk changelog? > > This would negatively impact the purpose of said NEWS files, which is to > notify users of changes in the software since the previous version they had > installed. There's absolutely no guarantee that the relevant news was > introduced within the last 10 most recent uploads - especially if the user > is doing an LTS->LTS upgrade. > > I'd rather we suppress (or otherwise ignore) this warning instead.
We should split the check into two different checks. The first should check if the NEWS entry version number is older than the oldest changelog entry version. This check should be disabled on Ubuntu. The second should check if the NEWS entry version number is newer than the oldest changelog entry version and matches one of the changelog entries. This second check should be enabled on Ubuntu. -- Benjamin Drung Debian & Ubuntu Developer
signature.asc
Description: This is a digitally signed message part
-- ubuntu-devel mailing list ubuntu-devel@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel
