desruisseaux commented on issue #109: URL: https://github.com/apache/maven-clean-plugin/issues/109#issuecomment-2825082782
I think that we need to keep the migration to `Path`. Other work such as #243 are built on top of those commits. This work replaces Plexus by `java.nio.file.PathMatcher` and `java.nio.file.FileVisitor`, which allows for example to delegate the handling of symbolic links to Java. It is also part of a larger effort to reduce inconsistent duplications among Maven plugins (e.g., `maven-clean-plugin` and `maven-compiler-plugin` duplicate codes for iterating over files, but with slight behavioural differences that are probably unintentional). Instead, I propose to add a `force` configuration option similar to `rm -f` in Unix. I suggest that the default value should be `false` for safety and for consistency with the standard usage in Unix commands. If agreed, I can volunteer for adding this option on top of #243. -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
