-
Notifications
You must be signed in to change notification settings - Fork 399
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
can't build project after latest jdt core updates (ArrayBinding cannot be cast to class ReferenceBinding) #3199
Comments
Looks like #3200 fixed it |
Actually, it looks like this is still broken (tested with b363a86). In my earlier test I forgot to switch back to the development version of eclipse.jdt.ls |
Narrowed it down to: @Test
public void test_compile_error() throws Exception {
Comparator<Object[]> ordering = CompoundOrdering.of(Arrays.asList(
OrderingByPosition.arrayOrdering(1, false, false),
OrderingByPosition.arrayOrdering(0, false, false)
));
} Which triggers the error. The code compiles with javac. It also compiles with the eclipse compiler if changing to: @Test
public void test_ok() throws Exception {
var orderings = Arrays.asList(
OrderingByPosition.arrayOrdering(1, false, false),
OrderingByPosition.arrayOrdering(0, false, false)
);
Comparator<Object[]> ordering = CompoundOrdering.of(orderings);
} https://github.com/mfussenegger/eclipse-compiler-bug contains the full trimmed reproduction. |
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199 (cherry picked from commit edda49c)
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199 (cherry picked from commit edda49c) # Conflicts: # server/src/test/java/io/crate/execution/engine/sort/OrderingByPositionTest.java
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199 (cherry picked from commit edda49c) # Conflicts: # server/src/test/java/io/crate/execution/engine/sort/OrderingByPositionTest.java
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199 (cherry picked from commit edda49c) # Conflicts: # server/src/test/java/io/crate/execution/engine/sort/OrderingByPositionTest.java
For some reason `OrderingByPositionTest` triggered a bug in the eclipse compiler, causing it to crash. See eclipse-jdtls/eclipse.jdt.ls#3199 (cherry picked from commit edda49c)
Using the current master (5c5f2e3), eclipse.jdt.ls is no longer able to load and build crate.
Update: Workaround was merged: crate/crate#16416, see #3199 (comment) for a reproduction.
The logs show the following errors:
I suspect this is caused by one of the recent jdt core/target platform updates, but I cannot compile the older commits of eclipse.jdt.ls so I can't tell for sure. (dependencies disappeared(?)).
After deleting OrderingByPositionTest the project builds again.
I can try to extract a smaller reproduction if necessary, but I suspected that with the full project, the file that causes the error and the full stacktrace someone more familiar with the code might already have an idea what's causing the problem.
The text was updated successfully, but these errors were encountered: