jdee icon indicating copy to clipboard operation
jdee copied to clipboard

Maven backend branch with clojure

Open udalrich opened this issue 6 years ago • 1 comments

My Emacs version: 25.3

My JDEE version (elpa package date): clojure-backend branch

My JDK version: 1.8, 10

My Operating System (Linux/OSX/Windows X/Other): OS X

Stacktrace (if applicable):

How to reproduce (if applicable):

Other info you consider could help to fix it:

A while ago, I took the clojure backend and made it work with the rest of JDEE. It was in a branch (closure-backend) and I never integrated it into master, partly because I wasn't sure how to deploy the various jars.

I have been using it at work by building it locally since then and it generally works for me, although it does create a lot of processes if you project has a lot of modules. (It runs a server in mvn once for each module.)

I now need to be able to use Java 10, which does not work in that branch. I saw comments about supporting it in various issues, so I tried to merge master into my branch. I saw that there now appears to be a jdee-backend that replaces/wraps the bsh. Will that be able to pull things like the class path and the source path from the pom.xml? Does it make sense to try to integrate the closure stuff with the backend?

I haven't looked at the backend code and am hoping someone can just tell me if it works with maven, is incompatible with the clojure approach or what would be a good way forward.

udalrich avatar May 04 '18 17:05 udalrich

jdee-backend is just code interacting with Bsh moved from various places (except jdee-ant and maybe one other place) to separate rest of the code from beanshell. It doesn't add any new code. The code is not fully separated from Bsh yet, but at least it's known where is the communication point with the backend.

In the future jdee-backend will become an interface to a backend (whatever it will be).

jdee-backend doesn't know anything about POM or other build tools - jdee-maven knows that.

pwojnowski avatar May 14 '18 06:05 pwojnowski