-
Notifications
You must be signed in to change notification settings - Fork 38
Closed
Labels
Description
Java 17 will be released soon. We should make sure that lsif-java works with Java 17 once it's available. Opening this issue to track this topic.
- Wait for Java 17 to become available in Coursier Update index coursier/jvm-index#52
- Test with Zulu 17, it works without issues
- Wait for Temurin 17 to become available Add temurin and update tools, libs coursier/jvm-index#53
- Update
roundToNearestStableRelease
https://github.com/sourcegraph/lsif-java/blob/02ccd70f1c08bba932693e9793d214207979a990/lsif-semanticdb/src/main/java/com/sourcegraph/lsif_semanticdb/JavaVersion.java#L43 - Automatically add missing JVM options to include exports for
jdk.compiler
APIs when indexing Gradle/Maven/sbt projects
--add-exports jdk.compiler/com.sun.tools.javac.model=ALL-UNNAMED
--add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
--add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
--add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
--add-exports jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED
- Make the SemanticDB compiler plugin work with Java 17 (handle breaking changes)
Caused by: java.lang.NoSuchMethodError: 'boolean com.sun.tools.javac.code.Symbol.isLocal()'
at com.sourcegraph.semanticdb_javac.GlobalSymbolsCache.isLocalVariable(GlobalSymbolsCache.java:60)
at com.sourcegraph.semanticdb_javac.GlobalSymbolsCache.uncachedSemanticdbSymbol(GlobalSymbolsCache.java:46)