-
Notifications
You must be signed in to change notification settings - Fork 321
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
Even more problems with latest jdt #3188
Comments
@iloveeclipse what is the proper replacement? |
Simply remove it, the option is enabled by default, it isn't option anymore. |
of course now i see a ton of test errors i cannot reproduce locally. |
Caused by: java.lang.Error: Unresolved compilation problem:
|
=> i assume we need to use another ton of reflection :( |
Not sure why Xtext used inlineJsrBytecode at all, but that option is always on for JDT compilation with 4.33+ SDK, and before it was possible to switch it off for target < 1.8. I believe with very old targets (<1.4?) it shouldn't be used either. |
xtext java 11 uses jdt 3.33 |
You mean 4.33? The option there is always true independently what is set. In 4.34 the option is gone. |
jdt core 3.33 |
OMG |
/home/jenkins/agent/workspace/xtext_main/org.eclipse.xtext.xbase.testing/src/org/eclipse/xtext/xbase/testing/InMemoryJavaCompiler.java:[207]
05:21:12 [ERROR] this.compilerOptions.inlineJsrBytecode = true;
05:21:12 [ERROR] ^^^^^^^^^^^^^^^^^
05:21:12 [ERROR] inlineJsrBytecode cannot be resolved or is not a field
05:21:12 [ERROR] 1 problem (1
The text was updated successfully, but these errors were encountered: