From da05f1f4a39fa7c23d4af46e2139859bda80abe5 Mon Sep 17 00:00:00 2001 From: Eugene Yokota Date: Sun, 28 Apr 2024 00:07:47 -0400 Subject: [PATCH] Adjust the API name entry for nested classes **Problem** Some build pipelining tests fail on the latest sbt RC. ``` Error: (sbt-test / scripted) Failed tests: Error: pipelining/Yjava-tasty-fromjavaobject Error: pipelining/Yjava-tasty-paths ``` This is likely caused by inconsistent capturing of APIs from Java sources in ExtractAPI vs AnalyzingJavaCompiler in Zinc. **Solution** This adjusts the API name entry for Java nested classes. --- .../tools/dotc/sbt/ExtractDependencies.scala | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/compiler/src/dotty/tools/dotc/sbt/ExtractDependencies.scala b/compiler/src/dotty/tools/dotc/sbt/ExtractDependencies.scala index 352636f681c3..fc7099c1a19a 100644 --- a/compiler/src/dotty/tools/dotc/sbt/ExtractDependencies.scala +++ b/compiler/src/dotty/tools/dotc/sbt/ExtractDependencies.scala @@ -105,8 +105,24 @@ object ExtractDependencies { val name: String = "sbt-deps" val description: String = "sends information on classes' dependencies to sbt" + /** Construct String name for the given sym. + * See https://github.com/sbt/zinc/blob/v1.9.6/internal/zinc-apiinfo/src/main/scala/sbt/internal/inc/ClassToAPI.scala#L86-L99 + * + * For a Java nested class M of a class C returns C's canonical name + "." + M's simple name. + */ def classNameAsString(sym: Symbol)(using Context): String = - sym.fullName.stripModuleClassSuffix.toString + def isJava(sym: Symbol)(using Context): Boolean = + Option(sym.source) match + case Some(src) => src.toString.endsWith(".java") + case None => false + def classNameAsString0(sym: Symbol)(using Context): String = + sym.fullName.stripModuleClassSuffix.toString + def javaClassNameAsString(sym: Symbol)(using Context): String = + if sym.owner.isClass && !sym.owner.isRoot then + javaClassNameAsString(sym.owner) + "." + sym.name.stripModuleClassSuffix.toString + else classNameAsString0(sym) + if isJava(sym) then javaClassNameAsString(sym) + else classNameAsString0(sym) /** Report an internal error in incremental compilation. */ def internalError(msg: => String, pos: SrcPos = NoSourcePosition)(using Context): Unit =