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 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 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.
-
Helmut Eller -
Jouni K Seppanen -
lawrence mitchell