gnodet opened a new pull request, #12297: URL: https://github.com/apache/maven/pull/12297
Fixes #12288. Supersedes #12291 — preserves the original commits by @ascheman with review fixes applied on top. **Original fix (by @ascheman):** Profiles activated through `settings.xml` (`<activeProfiles>` or `<activation><activeByDefault>true</activeByDefault>`) had their `<properties>` block dropped before the resolver session was built, so `aether.*` configuration declared in such a profile silently failed. The same `<properties>` on the same `<profile>` worked when activated via CLI `-P`. **Review fixes applied:** 1. Replaced redundant `request.getActiveProfiles()` call with inactive profile collection, so `-P !profileId` properly excludes `activeByDefault` profiles 2. Renamed IT class to `MavenITgh12288...` following project naming convention -- 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]
