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

Eshell not working on windows #72

Closed
julian-hoch opened this issue May 29, 2024 · 2 comments
Closed

Eshell not working on windows #72

julian-hoch opened this issue May 29, 2024 · 2 comments

Comments

@julian-hoch
Copy link

I use the same config on windows and linux:

(use-package shell-pop
  :config
  (setq shell-pop-shell-type '("eshell" "*eshell*" (lambda () (eshell))))
  (setq shell-pop-default-directory "~/")
  (setq shell-pop-autocd-to-working-dir t)
  (setq shell-pop-full-span t)
  :bind ("C-M-S-s" . shell-pop))

On linux this works and opens eshell, but on windows it opens the windows command line. Not sure whats going on, I can open eshell fine manually with M-x eshell on windows, too.

@syohex
Copy link
Collaborator

syohex commented May 31, 2024

(use-package shell-pop
  :config
  (custom-set-variables
   '(shell-pop-shell-type '("eshell" "*eshell*" (lambda () (eshell)))))
  (setq shell-pop-default-directory "~/")
  (setq shell-pop-autocd-to-working-dir t)
  (setq shell-pop-full-span t)
  :bind ("C-M-S-s" . shell-pop))

You have to use custom-set-variables instead of setq against shell-pop-shell-type. Because the setter function should be called to reflect shell settings but the setter function is not called when using setq.

@julian-hoch
Copy link
Author

Thanks! Learned something new, always appreciated.

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

No branches or pull requests

2 participants