Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefer loading SLIME from the local-projects directory. #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gonzojive
Copy link

Modifiy the .el template to look in the local-projects directory before the
dist/ directory of quicklisp.

Modifiy the .el template to look in the local-projects directory before the
dist/ directory of quicklisp.
@hraban
Copy link

hraban commented Nov 2, 2021

@quicklisp would you please reconsider this? Slime has a bug with emacs 28 that's fixed in their github master since jan 2021, but the latest release was in dec 2020. It's effectively broken atm, and this is the one package we can't override in quicklisp.

@hraban
Copy link

hraban commented Nov 2, 2021

I just dug a little deeper and apparently there's a(nother) layer of indirection built into slime loading. Through a file called swank.txt. You can load local slime like this:

cd ~/quicklisp
mv dists/quicklisp/installed/systems/swank.txt{,.orig}
echo local-projects/slime/swank.asd > dists/quicklisp/installed/systems/swank.txt

So if you're struggling with this error ( slime/slime#594 ): "error '(wrong-number-of-arguments (3 . 4) 2)'" in function "define-obsolete-variable-alias", you can fix it by running the above snippet, and:

mkdir -p ~/quicklisp/local-projects
cd ~/quicklisp/local-projects
git clone https://github.com/slime/slime

and you should be good to go with Emacs 28, Slime, Quicklisp, and the quicklisp slime helper

@quicklisp
Copy link
Owner

quicklisp commented Nov 2, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants