Felix Lechner pushed to branch master at lintian / lintian


Commits:
598c3763 by Felix Lechner at 2019-12-08T19:19:29Z
Add a new tag gratuitous-manpage to indicate a manpage without an executable.

The severity minor is the lowest setting to trigger a warning. That
way, most people (including the reporter for Bug#583125) are still
alerted to extra manpages.

Added in response to Bug#583125.

Gbp-Dch: ignore

- - - - -
20834ac5 by Felix Lechner at 2019-12-08T19:20:05Z
Issue new tag gratuitous-manpage when a manpage is not accompanied by an 
executable. (Closes: #583125)

For now, the tag is issued only for sections 1 (general commands) and
8 (system administration). The other sections look more complicated.
Perhaps section 6 (games) could be added here.

All existing tests for manpages were adjusted except for one. The test
manpages-dependencies ships a manpage in a package called pecho-data.
In that case, the tag was added as expected.

It is unclear how common it is to ship manpages in a separate -data
package. Policy 12.1 clearly discourages it. I struggled with that
question, and then decided to implement the tag.

I was further unsure about the proper severity. The current setting,
minor, is the lowest setting to trigger a warning. That way, most
people (including the reporter for Bug#583125) are still alerted to
extra manpages.

- - - - -
cc1fc075 by Felix Lechner at 2019-12-08T19:20:05Z
Provide test case with manpages that do not come with executables.

In response to Bug#583125.

Gbp-Dch: ignore

- - - - -
d08dccfe by Felix Lechner at 2019-12-08T19:20:05Z
Fix existing test cases; many did not install executables and triggered new tag 
gratuitous-manpage.

All tests now install executables, except for manpages-dependencies.

In that test, the new tag was added as expected because the manpage
ships in a separate -data package. According to policy 12.1, such
splitting of manpage and executable is discouraged.

These changes were made in response to Bug#583125.

Gbp-Dch: ignore

- - - - -


30 changed files:

- checks/manpages.pm
- + 
t/tags/checks/manpages/bad-link-to-undocumented-manpage/build-spec/debian/install
- t/tags/checks/manpages/manpages-general/build-spec/orig/bin/rstartd → 
t/tags/checks/manpages/bad-link-to-undocumented-manpage/build-spec/orig/bin/sbin-binary
- + t/tags/checks/manpages/bad-so-link-in-manpage/build-spec/debian/install
- t/tags/checks/manpages/manpages-general/build-spec/orig/bin/usr-sbin-binary → 
t/tags/checks/manpages/bad-so-link-in-manpage/build-spec/orig/bin/bad-so
- + t/tags/checks/manpages/fsstnd-dir-in-manual-page/build-spec/debian/install
- + 
t/tags/checks/manpages/fsstnd-dir-in-manual-page/build-spec/orig/bin/mentions-other-standard
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/debian/install
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/debian/manpages
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/fill-values
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/orig/bin/pwd
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/orig/man/addpart.8
- t/tags/checks/manpages/manpages-general/build-spec/orig/man/sketch.1 → 
t/tags/checks/manpages/gratuitous-manpage/build-spec/orig/man/lsmod.8
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/orig/man/pwd.1
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/orig/man/statfs.1
- + t/tags/checks/manpages/gratuitous-manpage/build-spec/orig/sbin/addpart
- + t/tags/checks/manpages/gratuitous-manpage/eval/desc
- + t/tags/checks/manpages/gratuitous-manpage/eval/tags
- + t/tags/checks/manpages/legacy-binary/build-spec/debian/install
- + t/tags/checks/manpages/legacy-binary/build-spec/orig/bin/bad-so
- + t/tags/checks/manpages/manpage-bad-whatis/build-spec/debian/install
- + t/tags/checks/manpages/manpage-bad-whatis/build-spec/orig/bin/bad-whatis
- + t/tags/checks/manpages/manpage-dh-make-template/build-spec/debian/install
- + t/tags/checks/manpages/manpage-dh-make-template/build-spec/orig/bin/program
- + t/tags/checks/manpages/manpage-empty/build-spec/debian/install
- + t/tags/checks/manpages/manpage-empty/build-spec/orig/bin/empty-manpage
- + t/tags/checks/manpages/manpage-errors-from-man/build-spec/debian/install
- + 
t/tags/checks/manpages/manpage-errors-from-man/build-spec/orig/bin/errors-from-man
- + t/tags/checks/manpages/manpage-errors-from-pod2man/build-spec/debian/install
- + 
t/tags/checks/manpages/manpage-errors-from-pod2man/build-spec/orig/bin/errors-from-pod2man


The diff was not included because it is too large.


View it on GitLab: 
https://salsa.debian.org/lintian/lintian/compare/d438eb392e5127da3ec85d0116f1979d93c49da1...d08dccfe097643353e73405002e52b3a1b9ba668

-- 
View it on GitLab: 
https://salsa.debian.org/lintian/lintian/compare/d438eb392e5127da3ec85d0116f1979d93c49da1...d08dccfe097643353e73405002e52b3a1b9ba668
You're receiving this email because of your account on salsa.debian.org.


Reply via email to