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

Clojure layer does not respect (setq cider-repl-popup-stacktraces nil) var in user-config() #7068

Closed
aeberts opened this issue Sep 8, 2016 · 3 comments

Comments

@aeberts
Copy link

aeberts commented Sep 8, 2016

Description

The variable cider-repl-popup-stacktraces is ignored when set to nil in user-config().

Reproduction guide

  • Start Emacs
  • Set cider-repl-popup-stacktraces to nil in user-config()
  • Start a repl using , s i
  • Enter something in the repl that will cause an error e.g. (/ 1 0)

Observed behaviour:

  • The *cider-error* buffer is displayed in a popup window.

Expected behaviour:

  • No *cider-error* buffer is displayed.

System Info

  • OS: gnu/linux
  • Emacs: 24.5.1
  • Spacemacs: 0.105.22
  • Spacemacs branch: master (rev. 9f9faa4)
  • Graphic display: t
  • Distribution: spacemacs
  • Editing style: hybrid
  • Completion: helm
  • Layers:
(osx
 (auto-completion :variables auto-completion-enable-snippets-in-popup)
 better-defaults emacs-lisp
 (org :variables org-enable-github-support t)
 git github markdown clojure python html erc
 (shell :variables shell-default-height 30 shell-default-position 'bottom)
 spell-checking syntax-checking version-control themes-megapack)

Code that may be source of the problem

from: /.emacs.d/layers/+lang/clojure/packages.el

(defun clojure/pre-init-popwin ()
  (spacemacs|use-package-add-hook popwin
    :post-config
    (push '("*cider-error*" :dedicated t :position bottom :stick t :noselect nil :height 0.4)
          popwin:special-display-config)
    (push '("*cider-doc*" :dedicated t :position bottom :stick t :noselect nil :height 0.4)
          popwin:special-display-config)))
@mahinshaw
Copy link
Contributor

@aeberts - cider-repl-popup-stacktraces is a deprecated var in cider refer to Changelog

@aeberts
Copy link
Author

aeberts commented Oct 11, 2016

@mahinshaw Thanks for the heads-up. I can confirm that removing:

(setq cider-repl-popup-stacktraces nil)

and adding:

(setq cider-show-error-buffer 'except-in-repl)

to the user-config section of the .spacemacs file solves the problem on my setup and the error buffer is no longer shown when an error appears in the repl.

@d12frosted
Copy link
Collaborator

Great to hear! Closing this issue as being solved. 💃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Bugs
Not reproduced
Development

No branches or pull requests

4 participants