Unverified Commit 803cfd95 authored by Alexey Tsvetkov's avatar Alexey Tsvetkov Committed by GitHub

Rename artifacts for all targets (#414)

Previously we only renamed artifacts for targets,
that can be compiled on a host.
This caused incompatible publications,
when the common publication, published from
Linux, was pointing to wrong macOS & iOS artifacts
parent 9bf726fc
......@@ -364,6 +364,13 @@ kotlin {
}
fun configureNativeTarget(os: OS, arch: Arch, target: KotlinNativeTarget) {
publishing.publications.configureEach {
this as MavenPublication
if (name == target.name) {
artifactId = SkikoArtifacts.nativeArtifactIdFor(os, arch)
}
}
if (!os.isCompatibleWithHost) return
val targetString = "${os.id}-${arch.id}"
......@@ -443,12 +450,6 @@ fun configureNativeTarget(os: OS, arch: Arch, target: KotlinNativeTarget) {
target.compilations.all {
compileKotlinTask.dependsOn(linkTask)
}
publishing.publications.configureEach {
this as MavenPublication
if (name == target.name) {
artifactId = SkikoArtifacts.nativeArtifactIdFor(os, arch)
}
}
}
tasks.withType(JavaCompile::class.java).configureEach {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment