giulong commented on PR #436: URL: https://github.com/apache/maven-wrapper/pull/436#issuecomment-5416402951
The issue on Windows and Maven 4 is related to the closing round bracket, which needs to be escaped. Quoting the value in `maven="[3.0,3.10-rc)"` solves on Windows, but isn't a valid syntax for Linux. The solution to cover all combinations, as you can see in the [verify action in my fork](https://github.com/giulong/maven-wrapper/actions/runs/32882936044), was to double the IT and run the same assertions: * [mavenversion_range](https://github.com/apache/maven-wrapper/pull/436/changes#diff-19f58e4079766f1fb04419bd84323c2c439a37befa2e04c4a1a8d5b6029a8482R20) runs with `maven=[3.0,3.10-rc)` on `invoker.os.family = !windows` * [mavenversion_range_windows](https://github.com/apache/maven-wrapper/pull/436/changes#diff-ed5e3d1ea675d92fd04d80ac5141e45c73f5c1fd5dcedfeac2009b4b30f0bc03R20) runs with `maven="[3.0,3.10-rc)"` on `invoker.os.family = windows` Can you run the verify action again? -- 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]
