Don't create semanticdb files when writing to JAR

Author: WojciechMazur

compiler/src/dotty/tools/dotc/semanticdb/ExtractSemanticDB.scala

@@ -24,6 +24,8 @@ import scala.annotation.{ threadUnsafe => tu, tailrec }
 import scala.PartialFunction.condOpt
 
 import dotty.tools.dotc.{semanticdb => s}
+import dotty.tools.io.AbstractFile
+import dotty.tools.io.JarArchive
 
 /** Extract symbol references and uses to semanticdb files.
  *  See https://scalameta.org/docs/semanticdb/specification.html#symbol-1
@@ -38,7 +40,8 @@ class ExtractSemanticDB extends Phase:
   override val description: String = ExtractSemanticDB.description
 
   override def isRunnable(using Context) =
-    super.isRunnable && ctx.settings.Xsemanticdb.value
+    super.isRunnable && ctx.settings.Xsemanticdb.value && 
+      !ExtractSemanticDB.semanticdbTarget.isInstanceOf[JarArchive]
 
   // Check not needed since it does not transform trees
   override def isCheckable: Boolean = false
@@ -473,21 +476,20 @@ object ExtractSemanticDB:
   val name: String = "extractSemanticDB"
   val description: String = "extract info into .semanticdb files"
 
+  def semanticdbTarget(using Context) =
+      val semanticdbTargetSetting = ctx.settings.semanticdbTarget.value
+      if semanticdbTargetSetting.isEmpty then ctx.settings.outputDir.value
+      else AbstractFile.getDirectory(semanticdbTargetSetting)
+      
   def write(
     source: SourceFile,
     occurrences: List[SymbolOccurrence],
     symbolInfos: List[SymbolInformation],
     synthetics: List[Synthetic],
   )(using Context): Unit =
     def absolutePath(path: Path): Path = path.toAbsolutePath.normalize
-    val semanticdbTarget =
-      val semanticdbTargetSetting = ctx.settings.semanticdbTarget.value
-      absolutePath(
-        if semanticdbTargetSetting.isEmpty then ctx.settings.outputDir.value.jpath
-        else Paths.get(semanticdbTargetSetting)
-      )
     val relPath = SourceFile.relativePath(source, ctx.settings.sourceroot.value)
-    val outpath = semanticdbTarget
+    val outpath = absolutePath(this.semanticdbTarget.jpath)
       .resolve("META-INF")
       .resolve("semanticdb")
       .resolve(relPath)