#14136 closed enhancement (fixed)
slime 20070503 Enhancement to support Emacs.app
Reported by: | jrhope | Owned by: | easye |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.6.0 |
Keywords: | slime emacs-app | Cc: | |
Port: |
Description
Here is a Portfile patch to enable slime to use Emacs.app for compilation and as the installation destination (i.e. to use 'emacs-app' instead of 'emacs' as the depends_lib).
Attachments (3)
Change History (11)
Changed 17 years ago by jrhope
Attachment: | slime-Portfile.diff added |
---|
comment:1 Changed 17 years ago by easye
A very much reworked version of the MacPorts SLIME portfile that:
- updates the sources to CVS HEAD of 20080203, which reflects the major refactoring into a 'contrib' subdirectory that occured in the last couple of months
- now has an 'app' variant that will install with the 'emacs-app' port
- removed the obsolescent 'devel' variant ('emacs-devel' is no longer in the MacPorts
- updated style to comply with what is outlined in http://guide.macports.org/#development.practices (no tabs, 4 column indents, Emacs/vim initial modeline)
- rewrote the sample '.emacs' initialization code message to conform with the new style enabling SLIME to more easily serve as the frontend to multiple Lisp (or with varying commandline options)
Changed 17 years ago by easye
comment:2 follow-up: 5 Changed 17 years ago by easye
Additional cleanups before re-requesting commit:
- now works with 'emacs+carbon' by finding the correct Emacs binary
- if an Emacs binary is not found (should always but to be safe), don't bytecompile the *.el files
- disambiguate global variables by prepending 'slime_' to all of them
- cleaned up .emacs initialization code examples
- now passes port lint target
comment:4 Changed 17 years ago by easye
And
- changed the dependency on installed Lisp so that '+sbcl' is now the default_variant
comment:5 Changed 17 years ago by jrhope
Replying to evenson@panix.com:
- now works with 'emacs+carbon' by finding the correct Emacs binary
The emacs+carbon fix breaks emacs-app support. That code needs to be moved into the else clause of if variant_isset app. Also, the app variant needs to modify depends_lib. Finally, #12115 is closed, so Emacs.app is now able to find slime in ${prefix}/share/emacs/site-lisp.
I'm attaching a modification of your Portfile.
comment:6 Changed 17 years ago by easye
Owner: | changed from macports-tickets@… to easieste@… |
---|---|
Status: | new → assigned |
Moving to close this out now that I have commit rights (yeah!)
comment:7 Changed 17 years ago by easye
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:8 Changed 16 years ago by (none)
Milestone: | Port Enhancements |
---|
Milestone Port Enhancements deleted
adds a variant to use emacs-app instead of emacs