ccw-ide / ccw

Counterclockwise is an Eclipse plugin helping developers write Clojure code
https://github.com/laurentpetit/ccw/wiki/GoogleCodeHome
Eclipse Public License 1.0
220 stars 50 forks source link

Cannot use Luminus projects with CCW REPL #604

Closed laurentpetit closed 9 years ago

laurentpetit commented 9 years ago

Original issue 563 created by laurentpetit on 2013-04-06T22:46:37.000Z:

What steps will reproduce the problem? 1.Create a new clojure project named 'test', with lein template 'luminus'

  1. First issue: When I press finish, I see the 'test' project being created in the Project Explorer, but the 'New Clojure Project' modal window does not disappear. I have to close the window and convert the project to a Leiningen project manually for things to work as expected. 3.Second issue: When I attempt to run test.repl in the CCW REPL (Clojure > Load file in REPL), I get the following output:

;; Clojure 1.5.0

;; Switching to test.repl namespace

nil

FileNotFoundException Could not locate clabango/parser__init.class or clabango/parser.clj on classpath: clojure.lang.RT.load (RT.java:443)

What is the expected output? What do you see instead?

Running the 'lein ring server' command in a seperate terminal instance for the same project works fine, it should be the same when using CCW.

What version of the product are you using? On what operating system?

Running Juno Service Release 2 on OSX Mountain Lion.

Please provide any additional information below.

Before the clabango dependency was added to Luminus things were working fine.

Managed to get it working by following these steps:

I deleted the project from eclipse and recreated a new one from terminal, using 'lein new luminus test'.

Then I created a new project matching the path and converted to a lein project. Updated lein references and voila!

laurentpetit commented 9 years ago

Comment #1 originally posted by laurentpetit on 2013-04-06T22:51:05.000Z:

<empty>

laurentpetit commented 9 years ago

Comment #2 originally posted by laurentpetit on 2014-05-22T08:15:04.000Z:

<empty>

laurentpetit commented 9 years ago

Comment #3 originally posted by laurentpetit on 2014-05-22T15:07:41.000Z:

Issue 541 has been merged into this issue.

laurentpetit commented 9 years ago

Comment #4 originally posted by laurentpetit on 2014-05-22T20:56:40.000Z:

Ok, so it was the Luminus lein new template plugin which was not working well with lein new's --to-dir option. I fixed it locally, and submitted a Pull Request to the luminus github project.

I also took the opportunity to make the "New Clojure Wizard" support 'lein new' template parameters, so that 'lein new luminus my-project +cljs +mongodb' can be used

laurentpetit commented 9 years ago

Comment #5 originally posted by laurentpetit on 2014-05-24T14:29:42.000Z:

Unfortunately, the fix you've submitted doesn't work because you can't use the dir dynamic var outside the scope of ->files.

laurentpetit commented 9 years ago

Comment #6 originally posted by laurentpetit on 2014-07-10T20:13:55.000Z:

<empty>

laurentpetit commented 9 years ago

Comment #7 originally posted by laurentpetit on 2014-07-10T20:16:14.000Z:

Issue 541 has been merged into this issue.

laurentpetit commented 9 years ago

Comment #8 originally posted by laurentpetit on 2014-07-21T21:31:58.000Z:

Change priority to Critical to give visibility to the issue, mostly because it has been started a long time ago and should be finished!

laurentpetit commented 9 years ago

Comment #9 originally posted by laurentpetit on 2014-08-21T20:36:44.000Z:

<empty>

laurentpetit commented 9 years ago

Comment #10 originally posted by laurentpetit on 2014-09-25T21:29:07.000Z:

<empty>

laurentpetit commented 9 years ago

Comment #11 originally posted by laurentpetit on 2014-11-18T21:25:16.000Z:

<empty>

laurentpetit commented 9 years ago

CCW and Luminus seem to finally be compatible, as per the tests I did