cstamas commented on issue #12522: URL: https://github.com/apache/maven/issues/12522#issuecomment-5068460289
I am afraid that analysis is a bit off. The "snapshot" vs "live" may be true for old Plexus, but am pretty much sure the Sisu Plexus (plexus-shim, drop in replacement for old plexus built on top of Sisu) is also injecting "live" collections. In fact, there is a telling comment done by Igor Fedorenko still present in Maven 3.10.x codebase: https://github.com/apache/maven/blob/9ec1f357cbff391d5f3ef6319db6e1294d75c52d/maven-core/src/main/java/org/apache/maven/lifecycle/internal/CompoundProjectExecutionListener.java#L31 OTOH, the proposed fix seems sound. -- 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]
