gnodet commented on issue #316:
URL: 
https://github.com/apache/maven-resources-plugin/issues/316#issuecomment-5833765290

   This issue is fixed upstream in apache/maven-filtering#397.
   
   The root cause: `DefaultMavenResourcesFiltering` emitted `"Copying file X"` 
unconditionally *before* calling the copy, so the log was always emitted even 
when the change-detection strategy (content-hash, timestamp, NEVER) decided to 
leave the destination untouched.
   
   The fix in maven-filtering#397:
   - `MavenFileFilter` gains a new `copyFileWithResult()` method that returns 
`true` when the destination was actually written, `false` when it was skipped.
   - `DefaultMavenResourcesFiltering` calls `copyFileWithResult()` and moves 
the debug log **after** the copy, emitting either `"Copying file X"` (written) 
or `"Skipping file X (up to date)"` (skipped) based on the actual outcome.
   
   The fix will ship in maven-filtering 4.0.0-beta-2 and will be picked up by 
maven-resources-plugin once PR #527 bumps `mavenFilteringVersion`.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to