[slime-devel] [PATCH] slime-path needed for compiling slime-changelog-date in slime.el
Hi, Here's a simple patch without which I couldn't compile slime.el in XEmacs. It complained about the variable slime-path not being defined when compiling slime-changelog-date. -- Jouni K Seppänen
Jouni K Seppanen <jks@iki.fi> writes:
Here's a simple patch without which I couldn't compile slime.el in XEmacs. It complained about the variable slime-path not being defined when compiling slime-changelog-date.
This is a bit a tricky. Your patch works when SLIME is in Emacs' load-path, but that's not necessarily the case during byte-compilation. Compiling slime-changelog-date requires something like *compile-truename* and the closest Emacs variable for this is byte-compile-current-file. Unfortunately this is not present in all Emacs versions I committed the patch anyway. If we ever have a proper install script we can bind byte-compile-current-file before compiling. Helmut.
Helmut Eller wrote: [...]
This is a bit a tricky. Your patch works when SLIME is in Emacs' load-path, but that's not necessarily the case during byte-compilation. Compiling slime-changelog-date requires something like *compile-truename* and the closest Emacs variable for this is byte-compile-current-file. Unfortunately this is not present in all Emacs versions
How about `load-file-name', or does that not work when byte-compiling? [...] -- lawrence mitchell <wence@gmx.li>
lawrence mitchell <wence@gmx.li> writes:
How about `load-file-name', or does that not work when byte-compiling?
I haven't tried it. All my Emacsen here bind byte-compile-current-file. I assume load-file-name is only bound during load-time and not (necessarily) during compile-time. But we could use it as a fallback if byte-compile-current-file is not bound. Helmut.
participants (3)
-
Helmut Eller -
Jouni K Seppanen -
lawrence mitchell