Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix the nrepl configuration #103

Merged
merged 3 commits into from Jan 19, 2015
Merged

Fix the nrepl configuration #103

merged 3 commits into from Jan 19, 2015

Conversation

mbossenbroek
Copy link
Contributor

@theJohnnyBrown could you try this fix and verify that you can start a repl?

@mbossenbroek
Copy link
Contributor Author

Apologies for the whitespace changes. It's just this in the subprojects section:

   dependencies {
        development 'org.clojure:tools.nrepl:0.2.3'
    }

    clojureRepl {
        port = "9999"
    }

@theJohnnyBrown
Copy link
Contributor

Thanks! FYI the latest version of nrepl is 0.2.7

After adding those lines clojureRepl produces this result:

% ./gradlew clojureRepl
Script '/Users/jbrown/projects/PigPen/gradle/convention.gradle': line 24
The ConfigurationContainer.add() method has been deprecated and is scheduled to be removed in Gradle 2.0. Please use the create() method instead.
The TaskContainer.add() method has been deprecated and is scheduled to be removed in Gradle 2.0. Please use the create() method instead.
:pigpen:clojureRepl
Repl started on port 9999
> Building 0% > :pigpen:clojureRepl

It's been displaying that 0% for a few minutes now.

The repl server starts, and I can connect to it, e.g. lein repl :connect 127.0.0.1:9999, but it looks like the classpath might not be set up right:

   user=> (require '[pigpen.core :as pig])

ClassNotFoundException pigpen.PigPenException  java.net.URLClassLoader$1.run (URLClassLoader.java:372)

@mbossenbroek
Copy link
Contributor Author

@theJohnnyBrown Found a different approach, seems to work better. Verified that I can not only connect to it, but that the classpath is set up right. Let me know if you see anything weird with this one.

mbossenbroek added a commit that referenced this pull request Jan 19, 2015
Fix the nrepl configuration
@mbossenbroek mbossenbroek merged commit f469cc4 into master Jan 19, 2015
@mbossenbroek mbossenbroek deleted the fix-nrepl branch January 19, 2015 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants