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 pages go into wrong directory #2670

Closed
olafhering opened this issue Sep 25, 2019 · 6 comments
Closed

man pages go into wrong directory #2670

olafhering opened this issue Sep 25, 2019 · 6 comments

Comments

@olafhering
Copy link
Contributor

make install shold install the man pages into /usr/share/man.
Right now they go straight into /usr/man, and there seems to be no way to override this directory.

@ghost
Copy link

ghost commented Sep 25, 2019

Indeed. We should add a --mandir option to dune install

@rgrinberg
Copy link
Member

@olafhering do you want to try implementing this? it should be fairly short.

@olafhering
Copy link
Contributor Author

It would be enough to have this via configure.ml. But then, if it exists there, it might be easy to have it also as a global knob at runtime. I will try it, yes.

rgrinberg added a commit to rgrinberg/opam-repository that referenced this issue Dec 13, 2019
…lugin, dune-private-libs and dune-glob (2.1.0)

CHANGES:

- Attach cinaps stanza actions to both `@runtest` and `@cinaps` aliases
  (ocaml/dune#2831, @NathanReb)

- Add variables `%{lib-private...}` and `%{libexec-private...}` for finding
  build paths of files in public and private libraries within the same
  project. (ocaml/dune#2901, @snowleopard)

- Add `--mandir` option to `$ dune install`. This option allows to override the
  installation directory for man pages. (ocaml/dune#2915, fixes ocaml/dune#2670, @rgrinberg)

- Fix `dune --version`. The bootstrap didn't compute the version
  correctly. (ocaml/dune#2929, fixes ocaml/dune#2911, @diml)

- Do not open the log file in `dune clean`. (ocaml/dune#2965, fixes ocaml/dune#2964 and
  ocaml/dune#2921, @diml)

- Support passing two arguments to `=`, `<>`, ... operators in package
  dependencies so that we can have things such as `(<> :os win32)`
  (ocaml/dune#2965, @diml)
rgrinberg added a commit to rgrinberg/opam-repository that referenced this issue Dec 21, 2019
…lugin, dune-private-libs and dune-glob (2.1.0)

CHANGES:

- Attach cinaps stanza actions to both `@runtest` and `@cinaps` aliases
  (ocaml/dune#2831, @NathanReb)

- Add variables `%{lib-private...}` and `%{libexec-private...}` for finding
  build paths of files in public and private libraries within the same
  project. (ocaml/dune#2901, @snowleopard)

- Add `--mandir` option to `$ dune install`. This option allows to override the
  installation directory for man pages. (ocaml/dune#2915, fixes ocaml/dune#2670, @rgrinberg)

- Fix `dune --version`. The bootstrap didn't compute the version
  correctly. (ocaml/dune#2929, fixes ocaml/dune#2911, @diml)

- Do not open the log file in `dune clean`. (ocaml/dune#2965, fixes ocaml/dune#2964 and
  ocaml/dune#2921, @diml)

- Support passing two arguments to `=`, `<>`, ... operators in package
  dependencies so that we can have things such as `(<> :os win32)`
  (ocaml/dune#2965, @diml)
@toots
Copy link
Contributor

toots commented Mar 30, 2021

I concur with this. This is particularly relevant because it is explicitly documented in the FHS here: https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s11.html#usrsharemanManualPages

There is a similar issue with /usr/share/doc/<package> but this one is not referenced in the standard so it's less pressing.

@ejgallego
Copy link
Collaborator

Wait, dune install --mandir already works, right?

@rgrinberg
Copy link
Member

Yeah, not only that but it's also possible to configure the default dir when building dune. This was introduced in 2.1.0 as far as I can tell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment