On Tue, Dec 2, 2014 at 9:02 PM, Stas Boukarev stassats@gmail.com wrote:
And on the off chance that you actually want to use the contrib (and if it still works) a way which doesn't require any changes to sbcl is to redefine the make-pretty-stream function, returning a subclass of pretty-stream from it. Consequently, a clean approach would be to have (defvar sb-ext:*pretty-stream-maker* #'make-pretty-stream) in SBCL.
Oh, the thing is, swank contrib loading is currently all or nothing. So, Fabrice probably doesn't want slime-presentation-streams specifically, and I had assumed it was needed by the more basic slime-presentations contrib, which it's not.
So, the first step is to disable this dangerous patching. (Done and pushed to SLIME's repo.)
Then, we can look into re-enable this functionality using the clean approach outlined above. Opened https://github.com/slime/slime/issues/213 to track that. Patches welcome, yada yada. :-)
Thanks Stas!
Luís Oliveira luismbo@gmail.com writes:
Oh, the thing is, swank contrib loading is currently all or nothing. So, Fabrice probably doesn't want slime-presentation-streams specifically
FWIW, there is code in the repo to fix exactly this problem:
https://github.com/slime/slime/tree/choose-swank-loading-method
If you don't add a contrib to slime-contribs, the lisp-side contrib doesn't get loaded. The code was developed between me and Faré.
SLY is using (a minor variation) of this and it works nicely, with no special configuration needed from the user.
João