fix: man dir for contexts (#7655)

it was set to bin/ accidentally Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
ocaml · Apr 28, 2023 · 5ae2b3a · 5ae2b3a
1 parent 7467cbb
commit 5ae2b3a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,6 +1,9 @@
 Unreleased
 ----------
 
+- Correctly set `MANPATH` in `dune exec`. Previously, we would use the `bin/`
+  directory of the context. (#7655, @rgrinberg)
+
 - Allow overriding the `ocaml` binary with findlib configuration (#7648,
   @rgrinberg)
 

diff --git a/src/dune_rules/local_install_path.ml b/src/dune_rules/local_install_path.ml
@@ -9,7 +9,7 @@ let lib_root ~context = Path.Build.relative (dir ~context) "lib"
 
 let bin_dir ~context = Path.Build.relative (dir ~context) "bin"
 
-let man_dir ~context = Path.Build.relative (dir ~context) "bin"
+let man_dir ~context = Path.Build.relative (dir ~context) "man"
 
 let lib_dir ~context ~package =
   Path.Build.relative (lib_root ~context) (Package.Name.to_string package)