SLIME support (delivered with ABCL) needs updating

[ehuelsmann]

The default implementation is WAY outdated. This issue can be closed if the implementation runs verified correctly against what is the HEAD of SLIME when closing the issue.

(Implementing additional features is nice, but not required for this issue.)

[Mark Evenson]

I'll take the SLIME support issues (since I usually debug them anyways).

SLIME from CVS HEAD works pretty well with ABCL at the moment, but the main part of this ticket is to remove the

slime-loader.lisp	slime.lisp
swank-abcl.lisp		swank-loader.lisp	swank-protocol.lisp	swank-xcl.lisp
swank-allegro.lisp	swank-package.lisp	swank-sbcl.lisp		swank.lisp

files from src/org/armedbear/lisp directory, and verify that everything still works.

[Mark Evenson]

There are also the files

SlimeInputStream.java
SlimeOutputStream.java

that are also (?) unused.

[Mark Evenson]

Answering previous comment: no, the functions

  EXTENSIONS:MAKE-SLIME-INPUT-STREAM
  EXTENSIONS:MAKE-SLIME-OUTPUT-STREAM

are definitely used (see swank-abcl.lisp:27 ff.)

As a side note, this is probably a good place to hook the creation of UTF-8 SLIME interaction in support of other input methods.

[Mark Evenson]

Remove uneeded slime files

[Mark Evenson]

I've been running (and rebuilding) ABCL and SLIME for a week or so without the files

slime-loader.lisp	slime.lisp
swank-abcl.lisp		swank-loader.lisp	swank-protocol.lisp	swank-xcl.lisp
swank-allegro.lisp	swank-package.lisp	swank-sbcl.lisp		swank.lisp

The attached patch removes them from the tree.

Reassigning to Erik to remove from trunk.

[ehuelsmann]

Removal of the files mentioned above has been completed in commit r11375. It would be interesting to see to what extent ABCL needs to be modified to work with swank-gray.lisp.

As soon as it does, the classes SlimeInputStream? and SlimeOutputStream? can be removed.

[ehuelsmann]

BTW, swank-gray compiles without warnings on the latest of trunk (r11375).

[ehuelsmann]

But:

(require 'asdf)
(require 'gray-streams)
(asdf:oos 'asdf:load-op :swank)

breaks the asdf operation.

See ticket #20.

[Mark Evenson]

The build system for ABCL should be further "tightened" with the 'remove-old-slime.patch', removing the references to the removed files in 'build.xml' as well as 'compile-system.lisp'.

[Mark Evenson]

Remove uneeded slime files from build

[Mark Evenson]

Replying to ehuelsmann:

But:

(require 'asdf)
(require 'gray-streams)
(asdf:oos 'asdf:load-op :swank)

breaks the asdf operation.

See ticket #20.

The patch [attached to ticket #20][1] allows this operation to succeed.

[1]: ​http://trac.common-lisp.net/armedbear/attachment/ticket/20/pprint-gray-streams.lisp

[ehuelsmann]

All slime artifacts have been removed. remove-slime-build.patch committed in r11376. Closing this issue. (We don't need to deliver Slime with ABCL; Slime CVS changes too often to keep up. Users should download their own.)