Parse cljs files

Description

Add a function clojure.tools.namespace.find/find-clojurescript-sources-in-dir to parse cljs files

Environment

None

Activity

Show:
Yehonathan Sharvit
December 7, 2014, 5:19 PM

a better patch: clj files are also considered as cljs source files

Stuart Sierra
December 8, 2014, 9:37 PM

I am reluctant to add anything related to ClojureScript files until a final decision has been made regarding Feature Expressions, which may include alternative file extensions. http://dev.clojure.org/display/design/Feature+Expressions

Yehonathan Sharvit
July 1, 2015, 7:20 AM

Now that reader conditionals is part of clojure, could your reconsider support cljs files parsing?

Stuart Sierra
July 24, 2015, 8:35 PM

I am interested in finding ways to support ClojureScript files in tools.namespace.

However, I don't think these patches really address the scope of the problem.

For better or for worse, tools.namespace has a layered structure: c.t.n.repl calls c.t.n.dir which calls c.t.n.file which calls c.t.n.parse and so on.

Some of the top layers (e.g. c.t.n.repl) don't make sense in ClojureScript. But the layered structure makes it awkward to pass options from the higher layers such as c.t.n.file down to the bottom layers where the Clojure/ClojureScript distinction actually matters, such as c.t.n.parse.

I think the solution will probably involve deprecating some of the higher-level namespaces such as c.t.n.file and replacing them with more composable primitives in the lower layers.

Stuart Sierra
July 24, 2015, 8:52 PM

More notes about this problem now captured in a new ticket: TNS-35.

Declined

Assignee

Stuart Sierra

Reporter

Yehonathan Sharvit

Labels

None

Approval

None

Patch

Code

Priority

Minor