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

Add support for InvokeDynamic (indy) #161

Closed
kriegaex opened this issue Jan 8, 2016 · 2 comments
Closed

Add support for InvokeDynamic (indy) #161

kriegaex opened this issue Jan 8, 2016 · 2 comments
Assignees

Comments

@kriegaex
Copy link

kriegaex commented Jan 8, 2016

Hi.

This is more of a question referring to an old Codehaus mailing list thread (is there a new ML?) and to this ticket which I can find nowhere else:

Is invokedynamic supported now?

Background: We currently use invokedynamic support via indy from GMavenPlus and additionally use the Eclipse compiler from IntelliJ IDEA because it is much faster and less problematic with recompilation compared to the standard Groovy compiler. In order for it to be in sync with what we do in Maven I would like to know if indy is now supported by Greclipse. We might then also switch to the Greclipse compiler plugin in Maven.

@aclement
Copy link
Contributor

Hey. Unfortunately no. The groovy compiler is patched for inclusion in groovy-eclipse - patched to improve error reporting, have recoverable parsing, better incremental behaviour, etc. We had to pick which version to patch and picked the non-indy one. If the indy one were patched in the same way then it could be used. (Actually I might be using old information - back when I last looked at this they were separate jars: indy and non-indy. If it is all in one jar now then the patched groovy just needs to be brought up to the latest level and you could do it).

@eric-milles eric-milles changed the title Invokedynamic (indy) support Feature Request: Invokedynamic (indy) support Jun 14, 2017
@eric-milles eric-milles changed the title Feature Request: Invokedynamic (indy) support Feature Request: Support for InvokeDynamic (indy) Jun 14, 2017
@eric-milles
Copy link
Member

I think you could use the indy version of groovy-all in your pom and pass the indy argument to the compiler plugin. Although I have not tried this myself.

@eric-milles eric-milles changed the title Feature Request: Support for InvokeDynamic (indy) Add support for InvokeDynamic (indy) Sep 7, 2017
@eric-milles eric-milles self-assigned this Oct 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants