search

overtone / emacs-live

M-x start-hacking
http://overtone.github.com/emacs-live/
Other
1.52k stars 241 forks source link

Clojure test mode broken? #122

Closed wgb closed 11 years ago

wgb commented 11 years ago

With repl connected and tests working on other machines, when I run C-c+, in a test I get: "Wong type argument: stringp, nil."

xavriley commented 11 years ago

after doing

brew install emacs --cocoa

and

git clone git://github.com/overtone/emacs-live.git ~/.emacs.d

(basically following the tutorial here http://paradigmx.net/blog/2013/04/01/clojure-toolchain-reloaded/)

I'm getting C-c C-, is undefined

I'm not too familiar with emacs but it seems like the same issue mentioned here: https://groups.google.com/forum/#!topic/overtone/BYvZ6hh3FOQ

Any suggestions for a workaround?

samaaron commented 11 years ago

Hi there,

On 19 Aug 2013, at 19:56, xavriley notifications@github.com wrote:

I'm getting C-c C-, is undefined

What are you expecting it to do?

Sam

http://sam.aaron.name

xavriley commented 11 years ago

Sorry that last comment was a bit vague.

Sitting inside the clojure-mode pack is the clojure-test-mode file. Because it's sat there in the repo I assumed that it would be loaded by Emacs Live but it turns out it isn't. I was trying use the mapping provided here - https://github.com/overtone/emacs-live/blob/master/packs/live/clojure-pack/lib/clojure-mode/clojure-test-mode.el#L501

I managed to fix it by creating my own emacs live pack which is available here - https://github.com/xavriley/clojure-test-mode-pack

It just requires test mode which is already in the emacs live repo and nothing else. Was a nice experiment in seeing how the packs system works :thumbsup:

samaaron commented 11 years ago

OK, so it sounds like I just need to require test mode in vanilla emacs-live? Would that fix your problems?

xavriley commented 11 years ago

@samaaron - I think that would be good for a general purpose clojure setup, yes. It has to be loaded after nrepl though.

With regards to the error that @wgb found - I just got the same thing after using a package-install that added a newer version of nrepl (2.0.0) to ~/.emacs.d/elpa/ which clashes with the emacs live version (1.8.0). In my case the package was the clojure-cheatsheet

Something doesn't like the new version so it's probably best to avoid any packages that require it for now.