Hiteshsai007 commented on PR #12260: URL: https://github.com/apache/maven/pull/12260#issuecomment-4715500950
### Why the old one failed: On Windows, the integration tests generated the `.mvn/jvm.config` file with a UTF-8 BOM (`\uFEFF`) prepended to it. The old `JvmConfigParser` read this file but did not strip the BOM, passing `\uFEFF-Dprop=val` directly to the Java 21 launcher. The JVM didn't recognize the corrupted `-D` option, failing to set the property and causing test assertions to fail. ### How the new one works: The new parser checks for and strips the leading BOM (`\uFEFF`) character. The clean `-Dprop=val` is passed to the Java 21 launcher, which successfully sets the system property, allowing all integration tests to pass. -- 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]
