elharo commented on issue #557: URL: https://github.com/apache/maven-jar-plugin/issues/557#issuecomment-5021217996
After further investigation, this is not actually a bug. The Plexus `DirectoryScanner.scandir()` builds paths using `File.separator` (e.g., `META-INF\\versions\\9\\test.txt` on Windows), not forward slashes. The detection logic in `AbstractJarMojo.java` uses `File.separatorChar` to build the prefix, which is consistent on all platforms. Both sides use the platform separator, so the `startsWith` check works correctly everywhere. The integration test from PR #563 passes on Windows CI without any code changes, confirming this. Closing as not a bug — the existing code is correct. -- 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]
