Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

man-db: explicitly pass section string #31825

Merged
merged 1 commit into from Nov 19, 2017

Conversation

phi-gamma
Copy link
Contributor

Work around a bug in the autoconf setup of man-db: The
list of default sections does not include section 0 (zero)
despite ./configure --help advertising it. This causes
header man pages (e. g. time.h from package posix_man_pages)
to be ignored by man(1):

$ file /run/current-system/sw/share/man/man0p/time.h.0p.gz
/run/current-system/sw/share/man/man0p/time.h.0p.gz: gzip compressed data, from Unix
$ man 0p time.h
No manual entry for 0p
No manual entry for time.h

Override the default (as defined in m4/man-arg-sections.m4)
until this is fixed upstream.

Motivation for this change

A category of man pages is not addressable by default.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • [ x] NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Work around a bug in the autoconf setup of man-db: The
list of default sections does not include section 0 (zero)
despite ``./configure --help`` advertising it. This causes
header man pages (e. g. time.h from package posix_man_pages)
to be ignored by man(1):

    $ file /run/current-system/sw/share/man/man0p/time.h.0p.gz
    /run/current-system/sw/share/man/man0p/time.h.0p.gz: gzip compressed data, from Unix
    $ man 0p time.h
    No manual entry for 0p
    No manual entry for time.h

Override the default (as defined in m4/man-arg-sections.m4)
until this is fixed upstream.
@phi-gamma
Copy link
Contributor Author

phi-gamma commented Nov 19, 2017

Patch to man-db has been sent upstream, it really is their bug. But for
now we can bypass it by forcing the correct list in configure.

http://lists.nongnu.org/archive/html/man-db-devel/2017-11/msg00000.html

@vcunat vcunat self-assigned this Nov 19, 2017
@vcunat vcunat added 1.severity: mass-rebuild 9.needs: port to stable A PR needs a backport to the stable release. labels Nov 19, 2017
@vcunat vcunat changed the base branch from master to staging November 19, 2017 17:02
@vcunat vcunat merged commit 9666665 into NixOS:staging Nov 19, 2017
vcunat added a commit that referenced this pull request Nov 21, 2017
(cherry picked from commit b22492e)
It's a trivial bugfix, very unlikely to break anything.
@samueldr samueldr removed the 9.needs: port to stable A PR needs a backport to the stable release. label Apr 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants