o Updated to 'plexus-utils-3.0.23'.
authorChristian Schulte <schulte@apache.org>
Thu, 5 May 2016 03:22:53 +0000 (03:22 +0000)
committerChristian Schulte <schulte@apache.org>
Thu, 5 May 2016 03:22:53 +0000 (03:22 +0000)
o Updated to stop suppressing exceptions incorrectly when closing resources.
  Most of the time the 'IOUtils.closeQuietly' methods of 'commons-io' and
  the 'IOUtil.close' method of 'plexus-utils' are used incorrectly. They
  are meant to be used in 'finally' blocks to not suppress an exception
  already thrown in the 'try' block. The documentation of the
  'IOUtils.closeQuietly' methods explicitly contains usage examples.
  As soon as 'commons-io' or 'plexus-utils' is targetted at Java 1.7, those
  methods should get deprecated and people should be told to use the
  try-with-resources statement instead.

git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1742353 13f79535-47bb-0310-9956-ffa450edef68

pom.xml
src/main/java/org/apache/maven/plugin/gpg/SignAndDeployFileMojo.java

diff --git a/pom.xml b/pom.xml
index 4604965..a8fb658 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -94,7 +94,7 @@ under the License.
     <dependency>
       <groupId>org.codehaus.plexus</groupId>
       <artifactId>plexus-utils</artifactId>
-      <version>3.0.20</version>
+      <version>3.0.23</version>
     </dependency>
     <dependency>
       <groupId>org.sonatype.plexus</groupId>
index 6739dff..9090bde 100644 (file)
@@ -538,7 +538,10 @@ public class SignAndDeployFileMojo
         try
         {
             reader = ReaderFactory.newXmlReader( pomFile );
-            return new MavenXpp3Reader().read( reader );
+            final Model model = new MavenXpp3Reader().read( reader );
+            reader.close();
+            reader = null;
+            return model;
         }
         catch ( FileNotFoundException e )
         {
@@ -576,8 +579,12 @@ public class SignAndDeployFileMojo
             tempFile.deleteOnExit();
 
             fw = WriterFactory.newXmlWriter( tempFile );
+
             new MavenXpp3Writer().write( fw, model );
 
+            fw.close();
+            fw = null;
+
             return tempFile;
         }
         catch ( IOException e )