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-login-shells custom doesn't match type #689
Comments
@Thaodan (use-package vterm
:ensure t
:config
(setq vterm-tramp-shells '(("docker" "/bin/sh")
("ssh" "/bin/bash")))) At least for me this worked... |
That sounds more like a workaround than a fix. |
Yes you are completely right, it is definitely not the fix... (defcustom vterm-tramp-shells '(("docker" "/bin/sh"))
"The shell that gets run in the vterm for tramp.
`vterm-tramp-shells' has to be a list of pairs of the format:
\(TRAMP-METHOD SHELL)"
:type '(alist :key-type string :value-type string)
:group 'vterm) and should be: (defcustom vterm-tramp-shells '(("docker" "/bin/sh"))
"The shell that gets run in the vterm for tramp.
`vterm-tramp-shells' has to be a list of pairs of the format:
\(TRAMP-METHOD SHELL)"
:type '(alist (:key-type string :value-type string))
:group 'vterm) At least when I do this change on the source code, this works: (use-package vterm
:ensure t
:config
(setopt vterm-tramp-shells '(("ssh" "/bin/bash")
("docker" "/bin/sh")))
) But I must confess I have no idea if this is right |
I am not familiar with |
setopt is basically setf but with check against custom types. |
@Sbozzolo I have to confess that I am also not very familiar with :type '(alist :key-type string :value-type string) I played around a little bit more with (use-package vterm
:bind (:map vterm-mode-map
("C-s" . isearch-forward))
:config
(setq vterm-max-scrollback 100000)
;; Include the title in vterm and multi-vterm buffers
;; setting multi-vterm-buffer-name isn't enough.
(setq vterm-buffer-name-string "*vterm %s*")
(setopt vterm-tramp-shells '(("docker" . "/bin/sh")
("ssh" . "/bin/zsh")))) to see if it works without the warning. If yes, than the only thing that maybe should be changed is the default value of the (defcustom vterm-tramp-shells '(("docker" "/bin/sh")) to: (defcustom vterm-tramp-shells '(("docker" . "/bin/sh")) |
While customizing
tramp-login-shells
I have noticed that the set custom type doesn't match the format of the variable used in vterm.For example this is the warning my Emacs showed me:
My configuration:
The text was updated successfully, but these errors were encountered: