feat(migrate): Add path conventions for package configuration files

Apply the path conventions from the ort-config repository also to package configuration files. Signed-off-by: Martin Nonnenmacher <[email protected]>
oss-review-toolkit · Sep 21, 2023 · 4388504 · 4388504
1 parent 62a0e5b
commit 4388504
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/plugins/commands/migrate/src/main/kotlin/MigrateCommand.kt b/plugins/commands/migrate/src/main/kotlin/MigrateCommand.kt
@@ -28,13 +28,15 @@ import com.github.ajalt.clikt.parameters.types.file
 import java.io.File
 
 import org.ossreviewtoolkit.model.FileFormat
+import org.ossreviewtoolkit.model.Identifier
 import org.ossreviewtoolkit.model.PackageCuration
 import org.ossreviewtoolkit.model.config.PackageConfiguration
 import org.ossreviewtoolkit.model.readValue
 import org.ossreviewtoolkit.model.yamlMapper
 import org.ossreviewtoolkit.plugins.commands.api.OrtCommand
 import org.ossreviewtoolkit.plugins.packagecurationproviders.ortconfig.toCurationPath
 import org.ossreviewtoolkit.plugins.packagemanagers.nuget.utils.getIdentifierWithNamespace
+import org.ossreviewtoolkit.utils.common.encodeOr
 import org.ossreviewtoolkit.utils.common.expandTilde
 import org.ossreviewtoolkit.utils.common.getCommonParentFile
 import org.ossreviewtoolkit.utils.common.safeMkdirs
@@ -92,6 +94,7 @@ class MigrateCommand : OrtCommand(
             }
         }.toMap()
 
+        val configsDir = getCommonParentFile(pkgConfigFiles.keys)
         candidateFiles -= pkgConfigFiles.keys
         echo("Skipping ${candidateFiles.size} files of unknown format.")
 
@@ -129,7 +132,18 @@ class MigrateCommand : OrtCommand(
             }
 
             if (configWithFixedId != config) {
-                configYamlMapper.writeValue(file, configWithFixedId)
+                val oldPath = file.relativeTo(configsDir).path
+                val newName = if (configWithFixedId.sourceArtifactUrl != null) "source-artifact" else "vcs"
+                val newPath = "${configWithFixedId.id.toPath(emptyValue = "_")}/$newName.yml"
+                val newFile = configsDir.resolve(newPath)
+
+                // TODO: Maybe make this optional to support layouts that do not follow ort-config conventions.
+                if (newPath != oldPath) {
+                    configsDir.resolve(oldPath).delete()
+                    newFile.parentFile.safeMkdirs()
+                }
+
+                configYamlMapper.writeValue(newFile, configWithFixedId)
             }
         }
     }