slachiewicz commented on PR #766: URL: https://github.com/apache/maven-invoker-plugin/pull/766#issuecomment-5745921042
Agreed — with `mergeUserSettings` the forked build can be split while `install` writes flat. `install` can't discover that per IT (settings are only one of the sources: `.mvn/maven.config`, `invoker.properties`, `mavenOpts`), so 1d7d3e5 pins the forked builds flat via a user property instead, which outranks settings-profile and system properties; an IT that wants split can override in its user properties. The IT now runs a consumer whose settings.xml enables split, and fails if either pin is removed. Also fixed the IT's own red: enabling split for the outer build hid the plugin snapshot under `cached/`. -- 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]
