atom-slime icon indicating copy to clipboard operation
atom-slime copied to clipboard

Better docker integration

Open sjlevine opened this issue 8 years ago • 7 comments

May need to expose host, port, etc.

sjlevine avatar Jul 28 '16 20:07 sjlevine

Feel free to assign this to me and remove the "help wanted" label.

haz avatar Jul 29 '16 13:07 haz

Something to think about with this is that when running in a docker image, M-. will return file names that are meaningless to Atom.

daewok avatar Aug 03 '16 18:08 daewok

@daewok -- aye, that's why the local (host) machine needs to have symlinks setup to mirror that of what docker sees. The custom setup we have in mind should be in our private wiki...

haz avatar Aug 03 '16 18:08 haz

That would work for GNU/Linux users (but would probably be annoying to set up). What about Windows users, with their silly drive letters?

daewok avatar Aug 03 '16 18:08 daewok

They would simply need to find a different set of instructions ;). Perhaps the issue is misnamed -- we don't want to do anything fully general for docker integration into atom; rather we just want to expose those settings necessary for those using MERS-site instructions to follow along with minimal effort.

haz avatar Aug 03 '16 18:08 haz

Fair enough, but it'll still be an eventual issue for this project I feel, even outside of Docker integration. For people running production code, it's not uncommon to connect to a remote swank server to do debugging, and the filesystems are likely different there as well. I'd advocate adding to the atom-slime roadmap some hooks for translating between local and remote file names. (For reference, see slime-to-lisp-filename-function and slime-from-lisp-filename-function).

daewok avatar Aug 03 '16 19:08 daewok

Brilliant idea. I'd suggest we keep this issue open even after pull request #38 goes through.

haz avatar Aug 04 '16 14:08 haz