Skip to content
This repository has been archived by the owner on Jun 4, 2022. It is now read-only.

Cannot require clojure.tools.reader #209

Closed
moxaj opened this issue Jul 17, 2017 · 5 comments
Closed

Cannot require clojure.tools.reader #209

moxaj opened this issue Jul 17, 2017 · 5 comments
Labels
upstream

Comments

@moxaj
Copy link
Contributor

moxaj commented Jul 17, 2017

With latest lumo (1.6):

cljs.user=> (require 'clojure.tools.reader)
No such macros namespace: cljs.tools.reader, could not locate cljs/tools/reader.clj or cljs/tools/reader.cljc

However, the following works:

cljs.user=> (require 'cljs.tools.reader)
nil
@anmonteiro anmonteiro added the bug label Jul 17, 2017
@moxaj
Copy link
Contributor Author

moxaj commented Jul 20, 2017

I suspect https://dev.clojure.org/jira/browse/CLJS-2069 may be relevant.

@moxaj
Copy link
Contributor Author

moxaj commented Jul 21, 2017

Upstream bug: https://dev.clojure.org/jira/browse/CLJS-2266

@moxaj moxaj closed this as completed Jul 21, 2017
@anmonteiro
Copy link
Owner

Still an issue in Lumo even if also upstream.

@anmonteiro anmonteiro reopened this Jul 21, 2017
@anmonteiro anmonteiro added upstream and removed bug labels Jul 21, 2017
@moxaj
Copy link
Contributor Author

moxaj commented Jul 21, 2017

Oh, sorry, not familiar with the policies :)

kanaka added a commit to kanaka/mal that referenced this issue Jul 27, 2017
@kanaka
[clojure] Fixes to support lumo 1.6.0.
62fc786 
- Pass down the CLJ_MODE value for DOCKERIZED builds in addition to
  run/tests.
- Correct deps so that node_modules is installed for CLJ_MODE=cljs for
  all steps.
- Lumo 1.6.0 is currently not able to install globally as root
  (anmonteiro/lumo#206) so add the local
  node_modules/.bin to the PATH in the run script.
- Reader conditionals needed for clojure.reader/cljs.reader until this
  is fixed: anmonteiro/lumo#209
@anmonteiro
Copy link
Owner

fixed in 1.7.0

micfan pushed a commit to micfan/make-a-lisp that referenced this issue Dec 2, 2018
@kanaka
[clojure] Fixes to support lumo 1.6.0.
c8e2a15 
- Pass down the CLJ_MODE value for DOCKERIZED builds in addition to
  run/tests.
- Correct deps so that node_modules is installed for CLJ_MODE=cljs for
  all steps.
- Lumo 1.6.0 is currently not able to install globally as root
  (anmonteiro/lumo#206) so add the local
  node_modules/.bin to the PATH in the run script.
- Reader conditionals needed for clojure.reader/cljs.reader until this
  is fixed: anmonteiro/lumo#209
luelista pushed a commit to luelista/mal that referenced this issue Mar 10, 2024
@kanaka
[clojure] Fixes to support lumo 1.6.0.
0c05fc9 
- Pass down the CLJ_MODE value for DOCKERIZED builds in addition to
  run/tests.
- Correct deps so that node_modules is installed for CLJ_MODE=cljs for
  all steps.
- Lumo 1.6.0 is currently not able to install globally as root
  (anmonteiro/lumo#206) so add the local
  node_modules/.bin to the PATH in the run script.
- Reader conditionals needed for clojure.reader/cljs.reader until this
  is fixed: anmonteiro/lumo#209
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
upstream
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@anmonteiro @moxaj