Created attachment 332042 [details, diff] 50ocaml-mode-gentoo.el patch Steps to reproduce: 1. Start emacs 2. Open an Ocaml file 3. Start the the ocaml subshell Now the minibuffer displays: Autoloading failed to define function inferior-caml-mode-font-hook This is probably due to the fact that caml-font.el does not define a function with such a name. Most probably the function 'inferior-caml-set-font-lock' should be called. I attached a patch that resolves this problem.
Thanks. The name of the function had changed between versions 3.10.0 and 3.10.2. I wonder why nobody had noticed this before. > Created attachment 332042 [details, diff] [details, diff] > 50ocaml-mode-gentoo.el patch Hm, I believe that the "add-hook" in the last line of the file should be updated for the new function name, too (and I've done this for -r1). Or is there any special reason why it shouldn't? Anyway, ocaml-mode-3.12.1-r1 committed to CVS. Please test (and report back).
(In reply to comment #1) > Thanks. The name of the function had changed between versions 3.10.0 and > 3.10.2. > I wonder why nobody had noticed this before. > > > Created attachment 332042 [details, diff] [details, diff] [details, diff] > > 50ocaml-mode-gentoo.el patch > > Hm, I believe that the "add-hook" in the last line of the file should be > updated for the new function name, too (and I've done this for -r1). Or is > there any special reason why it shouldn't? > No special reason, must have missed that line when I was re-doing the diff for the upload. > Anyway, ocaml-mode-3.12.1-r1 committed to CVS. Please test (and report back).
(In reply to comment #2) > (In reply to comment #1) > > Thanks. The name of the function had changed between versions 3.10.0 and > > 3.10.2. > > I wonder why nobody had noticed this before. > > > > > Created attachment 332042 [details, diff] [details, diff] [details, diff] [details, diff] > > > 50ocaml-mode-gentoo.el patch > > > > Hm, I believe that the "add-hook" in the last line of the file should be > > updated for the new function name, too (and I've done this for -r1). Or is > > there any special reason why it shouldn't? > > > > No special reason, must have missed that line when I was re-doing the diff > for the upload. > > > Anyway, ocaml-mode-3.12.1-r1 committed to CVS. Please test (and report back). So I did thorough testing and it works. (Sorry for taking so long, didn't quite realize the "and report back" part...)
Thanks for reporting and testing.