search

technomancy / slamhound

Slamhound rips your namespace form apart and reconstructs it.
Other
473 stars 38 forks source link

Cannot use slamhound from Emacs (running cider) #97

Open gavinmbell opened 7 years ago

gavinmbell commented 7 years ago

Hello... 'I'd like to be able to use Slamhound from emacs but when I install it I get an error

"Symbol’s function definition is void: nrepl-send-string-sync"

https://github.com/technomancy/slamhound/blob/master/slamhound.el#L60'

s/nrepl-send-string-sync/cider-nrepl-sync-request:eval/ in slamhound.el ? (has been suggested) Is that literally it? (my elisp is not so good)

see: https://github.com/clojure-emacs/cider/issues/1887

gavinmbell commented 7 years ago

Can someone help me get this working :-).... "pretty please with sugar on top"

https://www.youtube.com/watch?v=MykJiy-F82Y :-)

technomancy commented 7 years ago

Sorry; I don't use cider any more and would recommend against using slamhound.el. The whole idea behind that library's existence is ill-considered, and it needs to be replaced with a more generic mechanism for invoking clojure code directly from Emacs commands.

You can try making the patch to have it use cider as a workaround tho.

nivekuil commented 7 years ago

https://github.com/technomancy/slamhound/pull/93 worked for me, except it uses a function replace-in-string which doesn't seem to be a part of GNU Emacs. I used the implementation found here.