#30 An attempt to avoid "file being used by another process" situation

xolstice · Oct 15, 2017 · 55de313 · 55de313
1 parent bf7bc21
commit 55de313
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/main/java/org/xolstice/maven/plugin/protobuf/AbstractProtocMojo.java b/src/main/java/org/xolstice/maven/plugin/protobuf/AbstractProtocMojo.java
@@ -333,8 +333,8 @@ abstract class AbstractProtocMojo extends AbstractMojo {
     protected boolean includeDependenciesInDescriptorSet;
 
     /**
-     * If {@code true} and {@code writeDescriptorSet} has been set, do not strip SourceCodeInfo 
-     * from the FileDescriptorProto. This results in vastly larger descriptors that include information 
+     * If {@code true} and {@code writeDescriptorSet} has been set, do not strip SourceCodeInfo
+     * from the FileDescriptorProto. This results in vastly larger descriptors that include information
      * about the original location of each decl in the source file as well as surrounding comments.
      *
      * @since 0.4.4
@@ -983,6 +983,11 @@ protected File resolveBinaryArtifact(final Artifact artifact) throws MojoExecuti
             targetFileName = sourceFileName;
         }
         final File targetFile = new File(protocPluginDirectory, targetFileName);
+        if (targetFile.exists()) {
+            // The file must have already been copied in a prior plugin execution/invocation
+            getLog().debug("Executable file already exists: " + targetFile.getAbsolutePath());
+            return targetFile;
+        }
         try {
             FileUtils.forceMkdir(protocPluginDirectory);
         } catch (final IOException e) {