[lintian] 01/01: Warn if a "Team upload" (ie. that string is present in the changelog) but the uploader is among the Maintainer/Uploaders. (Closes: #882954)
This is an automated email from the git hooks/post-receive script. lamby pushed a commit to branch master in repository lintian. commit 35480565fd0588b397afc025374936f3c9cd1b9d Author: Chris Lamb Date: Tue Nov 28 20:57:08 2017 +0900 Warn if a "Team upload" (ie. that string is present in the changelog) but the uploader is among the Maintainer/Uploaders. (Closes: #882954) --- checks/nmu.desc| 6 ++ checks/nmu.pm | 1 + debian/changelog | 5 + t/tests/nmu-unnecessary-team-upload/debian/debian/changelog.in | 6 ++ t/tests/nmu-unnecessary-team-upload/desc | 5 + t/tests/nmu-unnecessary-team-upload/tags | 1 + 6 files changed, 24 insertions(+) diff --git a/checks/nmu.desc b/checks/nmu.desc index 1b9fd67..1aed82a 100644 --- a/checks/nmu.desc +++ b/checks/nmu.desc @@ -26,6 +26,12 @@ Info: A team upload (uploading a package from the same team without adding get a NMU revision number. Team uploads are recognized by the string "team upload" on the first line of the changelog file. +Tag: unnecessary-team-upload +Severity: normal +Certainty: certain +Info: The debian/changelog file refers to a "Team upload" but the + uploader is listed amongst the Maintainer/Uploaders. + Tag: source-nmu-has-incorrect-version-number Severity: normal Certainty: certain diff --git a/checks/nmu.pm b/checks/nmu.pm index d5c59a1..ea48e0e 100644 --- a/checks/nmu.pm +++ b/checks/nmu.pm @@ -115,6 +115,7 @@ sub run { } elsif ($changelog_mentions_team_upload) { tag 'team-upload-has-incorrect-version-number', $version if $version_nmuness == 1; +tag 'unnecessary-team-upload' unless $upload_is_nmu; } else { # Local packages may be either NMUs or not. unless ($changelog_mentions_local || $version_local) { diff --git a/debian/changelog b/debian/changelog index d925d96..4d5417d 100644 --- a/debian/changelog +++ b/debian/changelog @@ -2,6 +2,11 @@ lintian (2.5.61) UNRELEASED; urgency=medium XXX: generate tag summary + * checks/nmu.{desc,pm}: ++ [CL] Warn if a "Team upload" (ie. that string is present in the + changelog) but the uploader is among the Maintainer/Uploaders. + (Closes: #882954) + -- Chris Lamb Sun, 26 Nov 2017 11:51:35 +0900 lintian (2.5.60) unstable; urgency=medium diff --git a/t/tests/nmu-unnecessary-team-upload/debian/debian/changelog.in b/t/tests/nmu-unnecessary-team-upload/debian/debian/changelog.in new file mode 100644 index 000..2a31283 --- /dev/null +++ b/t/tests/nmu-unnecessary-team-upload/debian/debian/changelog.in @@ -0,0 +1,6 @@ +{$source} ({$version}) unstable; urgency=low + + * Team upload. + * Suppress "should close ITP bug" messages. (Closes: #123456) + + -- {$author} {$date} diff --git a/t/tests/nmu-unnecessary-team-upload/desc b/t/tests/nmu-unnecessary-team-upload/desc new file mode 100644 index 000..50b3755 --- /dev/null +++ b/t/tests/nmu-unnecessary-team-upload/desc @@ -0,0 +1,5 @@ +Testname: nmu-unnecessary-team-upload +Version: 1.0 +Description: Test for unnecessary team uploads +Test-For: + unnecessary-team-upload diff --git a/t/tests/nmu-unnecessary-team-upload/tags b/t/tests/nmu-unnecessary-team-upload/tags new file mode 100644 index 000..3670d7f --- /dev/null +++ b/t/tests/nmu-unnecessary-team-upload/tags @@ -0,0 +1 @@ +W: nmu-unnecessary-team-upload source: unnecessary-team-upload -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
[lintian] branch master updated (80e6138 -> 3548056)
This is an automated email from the git hooks/post-receive script. lamby pushed a change to branch master in repository lintian. from 80e6138 spelling: Add another correction new 3548056 Warn if a "Team upload" (ie. that string is present in the changelog) but the uploader is among the Maintainer/Uploaders. (Closes: #882954) The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: checks/nmu.desc | 6 ++ checks/nmu.pm | 1 + debian/changelog| 5 + .../nmu-unnecessary-team-upload/debian/debian}/changelog.in | 4 +--- t/tests/nmu-unnecessary-team-upload/desc| 5 + t/tests/nmu-unnecessary-team-upload/tags| 1 + 6 files changed, 19 insertions(+), 3 deletions(-) copy t/{templates/source/skel => tests/nmu-unnecessary-team-upload/debian/debian}/changelog.in (74%) create mode 100644 t/tests/nmu-unnecessary-team-upload/desc create mode 100644 t/tests/nmu-unnecessary-team-upload/tags -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
Bug#882954: lintian: warn about "team upload" but uploader is an uploader
tags 882954 + pending thanks Fixed in Git: https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=35480565fd0588b397afc025374936f3c9cd1b9d Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org / chris-lamb.co.uk `-
Processed: Re: lintian: warn about "team upload" but uploader is an uploader
Processing commands for cont...@bugs.debian.org: > tags 882954 + pending Bug #882954 [lintian] lintian: warn about "team upload" but uploader is an uploader Added tag(s) pending. > thanks Stopping processing here. Please contact me if you need assistance. -- 882954: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=882954 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
[lintian] 01/01: spelling: Add another correction
This is an automated email from the git hooks/post-receive script. pabs pushed a commit to branch master in repository lintian. commit 8d32e05259853789d030cba2c5b6e921035c68cd Author: Paul Wise Date: Tue Nov 28 21:03:20 2017 +0800 spelling: Add another correction --- data/spelling/corrections | 1 + 1 file changed, 1 insertion(+) diff --git a/data/spelling/corrections b/data/spelling/corrections index 159d58e..a688d72 100644 --- a/data/spelling/corrections +++ b/data/spelling/corrections @@ -4129,6 +4129,7 @@ wiil||will wilcards||wildcards wilcard||wildcard willl||will +wil||will wipoing||wiping wirtes||writes wirte||write -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
[lintian] branch master updated (3548056 -> 8d32e05)
This is an automated email from the git hooks/post-receive script. pabs pushed a change to branch master in repository lintian. from 3548056 Warn if a "Team upload" (ie. that string is present in the changelog) but the uploader is among the Maintainer/Uploaders. (Closes: #882954) new 8d32e05 spelling: Add another correction The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: data/spelling/corrections | 1 + 1 file changed, 1 insertion(+) -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
Bug#883041: lintian: Strange no-op false positive of spelling-error-in-manpage
Package: lintian Version: 2.5.60 Severity: normal Hi, checking the screen package with lintian yields three (identical) strange warnings which actually request a no-op change: I: screen: spelling-error-in-manpage usr/share/man/man1/screen.1.gz German German I: screen: spelling-error-in-manpage usr/share/man/man1/screen.1.gz German German I: screen: spelling-error-in-manpage usr/share/man/man1/screen.1.gz German German See also https://lintian.debian.org/full/a...@debian.org.html#screen I suspect that this is caused by one of these two lines, probably the first one: ~/lintian/lintian → git grep German data data/spelling/corrections:german||German data/spelling/corrections-case:german||German -- System Information: Debian Release: buster/sid APT prefers unstable APT policy: (990, 'unstable'), (600, 'testing'), (500, 'unstable-debug'), (500, 'buildd-unstable'), (110, 'experimental'), (1, 'experimental-debug'), (1, 'buildd-experimental') Architecture: amd64 (x86_64) Kernel: Linux 4.13.0-rc7-amd64 (SMP w/8 CPU cores) Locale: LANG=C.UTF-8, LC_CTYPE=C.UTF-8 (charmap=UTF-8), LANGUAGE=C.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: sysvinit (via /sbin/init) Versions of packages lintian depends on: ii binutils 2.29.1-9 ii bzip2 1.0.6-8.1 ii diffstat 1.61-1+b1 ii dpkg 1.19.0.4 ii file 1:5.32-1 ii gettext 0.19.8.1-4 ii intltool-debian 0.35.0+20060710.4 ii libapt-pkg-perl 0.1.33 ii libarchive-zip-perl 1.59-1 ii libclass-accessor-perl0.51-1 ii libclone-perl 0.39-1 ii libdigest-sha-perl5.98-1 ii libdpkg-perl 1.19.0.4 ii libemail-valid-perl 1.202-1 ii libfile-basedir-perl 0.07-1 ii libipc-run-perl 0.96-1 ii liblist-moreutils-perl0.416-1+b3 ii libparse-debianchangelog-perl 1.2.0-12 ii libperl5.26 [libdigest-sha-perl] 5.26.1-2 ii libtext-levenshtein-perl 0.13-1 ii libtimedate-perl 2.3000-2 ii liburi-perl 1.72-2 ii libxml-simple-perl2.24-1 ii libyaml-libyaml-perl 0.63-2+b2 ii man-db2.7.6.1-4 ii patchutils0.3.4-2 ii perl 5.26.1-2 ii t1utils 1.41-2 ii xz-utils 5.2.2-1.3 Versions of packages lintian recommends: ii libperlio-gzip-perl 0.19-1+b4 Versions of packages lintian suggests: ii binutils-multiarch 2.29.1-9 ii dpkg-dev 1.19.0.4 ii libhtml-parser-perl3.72-3+b2 ii libtext-template-perl 1.47-1 -- no debconf information
[lintian] 01/01: Remove "german|German" entry - it is covered by Udata/spelling/corrections-case. (Closes: #883041)
This is an automated email from the git hooks/post-receive script. lamby pushed a commit to branch master in repository lintian. commit 4aca1db8081418f0b11d91fd0c5b66bae1f4ebd0 Author: Chris Lamb Date: Wed Nov 29 11:54:28 2017 +0900 Remove "german|German" entry - it is covered by Udata/spelling/corrections-case. (Closes: #883041) --- data/spelling/corrections | 1 - debian/changelog | 4 t/tests/manpages-general/debian/program.1 | 1 + 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/data/spelling/corrections b/data/spelling/corrections index a688d72..867fd15 100644 --- a/data/spelling/corrections +++ b/data/spelling/corrections @@ -1789,7 +1789,6 @@ genric||generic gental||gentle geometrie||geometry geomtry||geometry -german||German geting||getting gettetx||gettext gived||given diff --git a/debian/changelog b/debian/changelog index 4d5417d..3a8d148 100644 --- a/debian/changelog +++ b/debian/changelog @@ -7,6 +7,10 @@ lintian (2.5.61) UNRELEASED; urgency=medium changelog) but the uploader is among the Maintainer/Uploaders. (Closes: #882954) + * data/spelling/corrections: ++ [CL] Remove "german|German" entry - it is covered by + Udata/spelling/corrections-case. (Closes: #883041) + -- Chris Lamb Sun, 26 Nov 2017 11:51:35 +0900 lintian (2.5.60) unstable; urgency=medium diff --git a/t/tests/manpages-general/debian/program.1 b/t/tests/manpages-general/debian/program.1 index cca9c1f..d370a94 100644 --- a/t/tests/manpages-general/debian/program.1 +++ b/t/tests/manpages-general/debian/program.1 @@ -40,6 +40,7 @@ options starting with two dashes (`-'). A summary of options is included below. For a complete description, see the Info files. This is a speling mistake. +This is the false-positive spelling mistake for the proper noun German. .TP .B \-h, \-\-help Show summary of options. -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
[lintian] branch master updated (8d32e05 -> 4aca1db)
This is an automated email from the git hooks/post-receive script. lamby pushed a change to branch master in repository lintian. from 8d32e05 spelling: Add another correction new 4aca1db Remove "german|German" entry - it is covered by Udata/spelling/corrections-case. (Closes: #883041) The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: data/spelling/corrections | 1 - debian/changelog | 4 t/tests/manpages-general/debian/program.1 | 1 + 3 files changed, 5 insertions(+), 1 deletion(-) -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
Bug#883041: lintian: Strange no-op false positive of spelling-error-in-manpage
tags 883041 + pending thanks >I suspect that this is caused by one of these two lines, probably the > first one: > > data/spelling/corrections:german||German Indeed. Fixed in Git, many thanks! https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=4aca1db8081418f0b11d91fd0c5b66bae1f4ebd0 Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org / chris-lamb.co.uk `-
Processed: Re: lintian: Strange no-op false positive of spelling-error-in-manpage
Processing commands for cont...@bugs.debian.org: > tags 883041 + pending Bug #883041 [lintian] lintian: Strange no-op false positive of spelling-error-in-manpage Added tag(s) pending. > thanks Stopping processing here. Please contact me if you need assistance. -- 883041: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=883041 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
[lintian] 01/01: Fix typo in previous commit
This is an automated email from the git hooks/post-receive script. abe pushed a commit to branch master in repository lintian. commit 5efedf60a17facf326b4a1fa553c848e2569a7ea Author: Axel Beckert Date: Wed Nov 29 04:09:18 2017 +0100 Fix typo in previous commit Gbp-Dch: Ignore --- debian/changelog | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/changelog b/debian/changelog index 3a8d148..e06779e 100644 --- a/debian/changelog +++ b/debian/changelog @@ -9,7 +9,7 @@ lintian (2.5.61) UNRELEASED; urgency=medium * data/spelling/corrections: + [CL] Remove "german|German" entry - it is covered by - Udata/spelling/corrections-case. (Closes: #883041) + data/spelling/corrections-case. (Closes: #883041) -- Chris Lamb Sun, 26 Nov 2017 11:51:35 +0900 -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
[lintian] branch master updated (4aca1db -> 5efedf6)
This is an automated email from the git hooks/post-receive script. abe pushed a change to branch master in repository lintian. from 4aca1db Remove "german|German" entry - it is covered by Udata/spelling/corrections-case. (Closes: #883041) new 5efedf6 Fix typo in previous commit The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: debian/changelog | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
Bug#883041: lintian: Strange no-op false positive of spelling-error-in-manpage
Hi Chris, Chris Lamb wrote: > > I suspect that this is caused by one of these two lines, probably the > > first one: > > > > data/spelling/corrections:german||German > > Indeed. Fixed in Git, many thanks! Thanks for checking and fixing so quickly! Regards, Axel -- ,''`. | Axel Beckert , 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
Bug#883053: dependency-on-python-version-marked-for-end-of-life doesn't match python:any dependency
Package: lintian Version: 2.5.60 As seen with the dput-ng binary, the dependency-on-python-version-marked-for-end-of-life doesn't match the python:any dependency.