Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix compability issue #75 with newer piggieback version when doing "exec". #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jumarko
Copy link

@jumarko jumarko commented May 12, 2017

There was an issue when trying to start browser-connected repl:
[{:type java.lang.IllegalArgumentException
:message "No value supplied for key: cemerick.austin.DelegatingExecEnv@1d1e650e"
:at [clojure.lang.PersistentHashMap create "PersistentHashMap.java" 77]}]
:trace
[[clojure.lang.PersistentHashMap create "PersistentHashMap.java" 77]
[cemerick.piggieback$cljs_repl invokeStatic "piggieback.clj" 212]
[cemerick.piggieback$cljs_repl doInvoke "piggieback.clj" 212]
[clojure.lang.RestFn applyTo "RestFn.java" 139]
[clojure.core$apply invokeStatic "core.clj" 650]
[clojure.core$apply invoke "core.clj" 641]
[cemerick.austin.repls$cljs_repl invokeStatic "repls.clj" 49]
[cemerick.austin.repls$cljs_repl doInvoke "repls.clj" 43]

See #75.

There was an issue when trying to start browser-connected repl:
 [{:type java.lang.IllegalArgumentException
   :message "No value supplied for key: cemerick.austin.DelegatingExecEnv@1d1e650e"
   :at [clojure.lang.PersistentHashMap create "PersistentHashMap.java" 77]}]
 :trace
 [[clojure.lang.PersistentHashMap create "PersistentHashMap.java" 77]
  [cemerick.piggieback$cljs_repl invokeStatic "piggieback.clj" 212]
  [cemerick.piggieback$cljs_repl doInvoke "piggieback.clj" 212]
  [clojure.lang.RestFn applyTo "RestFn.java" 139]
  [clojure.core$apply invokeStatic "core.clj" 650]
  [clojure.core$apply invoke "core.clj" 641]
  [cemerick.austin.repls$cljs_repl invokeStatic "repls.clj" 49]
  [cemerick.austin.repls$cljs_repl doInvoke "repls.clj" 43]

See cemerick#75.
@ccfontes
Copy link

ccfontes commented May 9, 2018

Why this wasn't merged yet?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants