[GitHub] [maven-filtering] sebthom commented on a change in pull request #15: [MRESOURCES-268] - Report path of offending file on copy/filter error.

[GitBox]

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



##########
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:
       To illustrate the effect of this PR some screenshots:
   
   1) error message before this PR
   
   
![image](https://user-images.githubusercontent.com/426959/104460261-23e7db00-55ae-11eb-8597-4ea95c5098dc.png)
   
   2) error message with this PR
   
   
![image](https://user-images.githubusercontent.com/426959/104460295-33672400-55ae-11eb-801a-091a62190773.png)
   




----------------------------------------------------------------
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