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

TRAMP improvements #3412

Merged
merged 3 commits into from
Aug 18, 2023
Merged

TRAMP improvements #3412

merged 3 commits into from
Aug 18, 2023

Conversation

vemv
Copy link
Member

@vemv vemv commented Aug 15, 2023

  • Add a Dockerfile for TRAMP development purposes
    • Includes a README.
  • Make TRAMP functionality work when using non-standard ports
    • e.g. when we connect from a SSH port other than 22.
  • Don't lose the CIDER session over TRAMP files

@bbatsov
Copy link
Member

bbatsov commented Aug 18, 2023

The changes look good overall. Do we also need to update/extend https://docs.cider.mx/cider/basics/up_and_running.html#working-with-remote-hosts ?

@vemv vemv merged commit 856a35c into master Aug 18, 2023
26 checks passed
@vemv vemv deleted the dockerfile branch August 18, 2023 10:30
@vemv
Copy link
Member Author

vemv commented Aug 21, 2023

Hey @mmontone @igrishaev ,

No linked CIDER sessions should be far more rare to encounter now (as of CIDER master, as released already into MELPA non- Stable)

  • For vanilla CIDER connections, it generally shouldn't happen
  • For TRAMP connections, we fixed the issue
  • For Docker-backed connections, we also fixed the issue

It would be much helpful if you could try removing the workaround and checking if it works now.

If it didn't, you now can:

  • Try jumping to definition,
  • See it fail with No linked CIDER sessions
  • Then, use the new command M-x cider-debug-sesman-friendly-session-p
  • Copy its output (as visible in the *messages* buffer) and paste it here.

Thanks much - V

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.

CIDER session is lost when navigating to definition
2 participants