<< Back to previous view

[CLJS-579] Use leiningen to handle clojurescript dependency resolution && classpath string generation Created: 28/Aug/13  Updated: 05/Sep/13

Status: Open
Project: ClojureScript
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Enhancement Priority: Minor
Reporter: Nicola Mometto Assignee: Nicola Mometto
Resolution: Unresolved Votes: 0
Labels: None

Attachments: Text File 0001-let-lein-handle-classpath-string-generation-dependen.patch    

 Description   

After every bump in one of clojurescript's dependency, it is needed to manually run script/clean && script/bootstrap in order to use the new versions.

By leveraging `lein classpath`, we can automate this process removing the need for any manual intervention.



 Comments   
Comment by Nicola Mometto [ 28/Aug/13 7:45 PM ]

The following patch also removes script/bootstrap script/clean and script/test-compile since those files are no more relevant.

I haven't tested the batch version of the scripts since I don't have windows on my machine, it would be a good idea to have somebody test those before merging.

Comment by David Nolen [ 04/Sep/13 11:07 PM ]

Is there a revised minimal patch? Thanks!

Comment by Nicola Mometto [ 05/Sep/13 6:10 AM ]

http://dev.clojure.org/jira/browse/CLJS-578 only adds project.clj, is this what you're asking for?





[CLJS-837] Cache namespace env Created: 08/Aug/14  Updated: 11/Aug/14

Status: Open
Project: ClojureScript
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Enhancement Priority: Minor
Reporter: Nicola Mometto Assignee: Nicola Mometto
Resolution: Unresolved Votes: 1
Labels: None

Attachments: Text File 0001-add-support-for-cached-env.patch    

 Description   

See: https://github.com/clojure/tools.analyzer.js/blob/master/src/main/clojure/clojure/tools/analyzer/js.clj#L524-L548



 Comments   
Comment by Nicola Mometto [ 11/Aug/14 8:09 AM ]

I've written an initial patch based on the same approach I used for t.a.js, it's attached as 0001-add-support-for-cached-env.patch
Currently I'm seeing an OOM exception when I invoke backup-env, after loading cljs.core, which is exactly what David told me he was getting when toying with a similar patch.

At a quick glance, it seems like the analyzer's ::namespace map contains significant more fields per var map than tools.analyzer.js's one, so that's probably why backup-env works fine for t.a.js but not for cljs





Generated at Tue Oct 21 20:53:23 CDT 2014 using JIRA 4.4#649-r158309.