Respect attach flag for descriptor set goals

Fixes #59
xolstice · May 13, 2019 · 68c82a4 · 68c82a4
1 parent e35aa69
commit 68c82a4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/main/java/org/xolstice/maven/plugin/protobuf/ProtocCompileDescriptorSetMojo.java b/src/main/java/org/xolstice/maven/plugin/protobuf/ProtocCompileDescriptorSetMojo.java
@@ -115,8 +115,10 @@ protected File getOutputDirectory() {
     @Override
     protected void doAttachGeneratedFiles() {
         final File outputDirectory = getOutputDirectory();
-        final File descriptorSetFile = new File(getOutputDirectory(), descriptorSetFileName);
-        projectHelper.attachArtifact(project, "pb", classifier, descriptorSetFile);
+        if (attach) {
+            final File descriptorSetFile = new File(getOutputDirectory(), descriptorSetFileName);
+            projectHelper.attachArtifact(project, "pb", classifier, descriptorSetFile);
+        }
         buildContext.refresh(outputDirectory);
     }
 }
diff --git a/src/main/java/org/xolstice/maven/plugin/protobuf/ProtocTestCompileDescriptorSetMojo.java b/src/main/java/org/xolstice/maven/plugin/protobuf/ProtocTestCompileDescriptorSetMojo.java
@@ -122,8 +122,10 @@ protected File getOutputDirectory() {
     @Override
     protected void doAttachGeneratedFiles() {
         final File outputDirectory = getOutputDirectory();
-        final File descriptorSetFile = new File(getOutputDirectory(), descriptorSetFileName);
-        projectHelper.attachArtifact(project, "test-pb", classifier, descriptorSetFile);
+        if (attach) {
+            final File descriptorSetFile = new File(getOutputDirectory(), descriptorSetFileName);
+            projectHelper.attachArtifact(project, "test-pb", classifier, descriptorSetFile);
+        }
         buildContext.refresh(outputDirectory);
     }
 }