ocaml-batteries-team / batteries-included

Batteries Included project
http://ocaml-batteries-team.github.com/batteries-included/hdoc2/
Other
519 stars 106 forks source link

#man "batteries" in the toplevel point to the wrong address #40

Closed rivercheng closed 14 years ago

rivercheng commented 14 years ago

When '#man "batteries"' is entered in the top level, the url "file:///usr/share/doc/ocaml-batteries/html/index.html" is opened in the default web browser. The real address, however, should be "file:///usr/share/doc/ocaml-batteries/index.html".

thelema commented 14 years ago

Curious. On my system, it opens file:///usr/share/doc/ocaml-batteries/index.html

rivercheng commented 14 years ago

The version I compiled is 1.0.1. The OS is ubuntu 9.10. OCaml version is 3.11.1. Moreover, when I "omake doc", a lot warning appeared like:

Warning: Element Printf not found Warning: Element Format not found Warning: Element Scanf.bscanf not found Warning: Element Scanf.bscanf not found Warning: Element Scanf.bscanf not found .

thelema commented 14 years ago

Those warnings are normal, sometime I'll get around to fixing them. I'm running Ubuntu 9.10 as well, with the apt version of ocaml. Could you paste your batteries_config.ml file? (just the non-comments, of course)

rivercheng commented 14 years ago

The file in the unzipped source directory is attached. In the installed directory, there's only batteries_config.mli and batteries_config.cmi.

let version = "1.0.1";; let documentationroot = "/usr/share/doc/ocaml-batteries/";; let (browser: (, , ) format) = "x-www-browser %s";;

(*The default function to open a www browser.) let default_browse s = let command = Printf.sprintf browser s in Sys.command command let current_browse = ref default_browse

let browse s = !current_browse s let set_browser f = current_browse := f

let max_array_length = Sys.max_array_length let word_size = Sys.word_size let max_string_length= Sys.max_string_length

thelema commented 14 years ago

ah, this was fixed right after v1.0.1. 1.1.0 fixes this issue, commit bbac42e42 specifically