Summary: | app-emacs/ocaml-mode: Autoloading failed to define function inferior-caml-mode-font-hook | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | sgbmyr |
Component: | [OLD] Development | Assignee: | Gentoo Team for the ML programming language family <ml> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | emacs, sgbmyr |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | 50ocaml-mode-gentoo.el patch |
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. |
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.