yeroc commented on PR #493:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/493#issuecomment-6620904339

   Thanks @olamy. #568 is on master now, so after rebasing, this PR runs 
against `TestCompileGoalAfterTestChangeTest`.
   
   Fixed by restoring `target/classes` with fresh timestamps, the same way 
master restores the jar. `attachedOutputs` still honour `preserveTimestamps`.
   
   Copilot also flagged that `restoreStagedArtifacts` deletes the whole staged 
tree instead of restoring files that weren't rebuilt, because its 
`preVisitDirectory` check matches parent directories. This PR no longer 
triggers that issue, so I'm not sure whether you want it fixed in this PR or in 
a separate issue.
   


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