michael-o commented on a change in pull request #15:
URL: https://github.com/apache/maven-filtering/pull/15#discussion_r556067456



##########
File path: 
src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
##########
@@ -110,7 +110,8 @@ public void copyFile( File from, File to, boolean 
filtering, List<FileUtils.Filt
         }
         catch ( IOException e )
         {
-            throw new MavenFilteringException( e.getMessage(), e );
+            throw new MavenFilteringException( ( filtering ? "filtering " : 
"copying " ) + from.getPath() + " to "

Review comment:
       The the root cause does not bubble up, then this is crap. Likely the log 
message is not ok. You should not try for this the symptom here. Please look in 
MRESOURCES where this exception is handled.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to