write pom files for non-primary mappings if they don't already exist.

Author: LexManos

src/main/java/net/minecraftforge/mcmaven/impl/MinecraftMaven.java

@@ -138,7 +138,15 @@ private void finalize(Artifact module, Mappings mappings, List<Repo.PendingArtif
                 var cache = HashStore.fromFile(target)
                     .add("source", source);
 
-                if (!target.exists() || !cache.isSame()) {
+                boolean write;
+                if ("pom".equals(pending.getArtifact().getExtension())) {
+                    // Write the pom for non-primary mappings if we haven't generated primary mappings yet
+                    write = !target.exists() || (mappings.isPrimary() && !cache.isSame());
+                } else {
+                    write = !target.exists() || !cache.isSame();
+                }
+
+                if (write) {
                     // TODO: [MCMavenizer] Add --api argument to turn class artifacts to api-only targets for a public repo
                     try {
                         org.apache.commons.io.FileUtils.copyFile(source, target);

src/main/java/net/minecraftforge/mcmaven/impl/repo/forge/ForgeRepo.java

@@ -36,7 +36,6 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
-import java.util.Set;
 import java.util.stream.Stream;
 
 // TODO: [MCMavenizer][ForgeRepo] For now, the ForgeRepo needs to be fully complete with everything it has to do.
@@ -148,9 +147,7 @@ private List<PendingArtifact> processV3(String version, Mappings mappings) {
         var classes = pending("Classes", classesTask, name, () -> classVariants(mappings, patcher, extraCoords, mappingCoords));
         var metadata = pending("Metadata", metadata(build, patcher), name.withClassifier("metadata").withExtension("zip"));
 
-        PendingArtifact pom = null;
-        if (mappings.isPrimary())
-            pom = pending("Maven POM", pom(build, patcher, version, extraCoords, mappingCoords), name.withExtension("pom"));
+        var pom = pending("Maven POM", pom(build, patcher, version, extraCoords, mappingCoords), name.withExtension("pom"));
 
         var extraOutput = this.mcpconfig.processExtra(Constants.MC_GROUP + ':' + Constants.MC_CLIENT, patcher.getMCP().getName().getVersion());
         return Stream.concat(

src/main/java/net/minecraftforge/mcmaven/impl/repo/mcpconfig/MCPConfigRepo.java

@@ -123,15 +123,12 @@ public List<PendingArtifact> process(Artifact artifact, Mappings mappings) {
                 var sources = pending("Sources", sourcesTask, name.withClassifier("sources"), sourceVariant(mappings));
                 var classes = pending("Classes", classesTask, name, () -> classVariants(mappings, mcpSide));
                 var metadata = pending("Metadata", metadata(build, mcpSide), name.withClassifier("metadata").withExtension("zip"));
+                var pom = pending("Maven POM", pom(build, side, mcpSide, version), name.withExtension("pom"));
+
                 pending.addAll(List.of(
-                    sources, classes, metadata
+                    sources, classes, metadata, pom
                 ));
 
-                if (mappings.isPrimary()) {
-                    var pom = pending("Maven POM", pom(build, side, mcpSide, version), name.withExtension("pom"));
-                    pending.add(pom);
-                }
-
                 yield pending;
             }
         };