This is an automated email from the ASF dual-hosted git repository.
gnodet pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven.git
The following commit(s) were added to refs/heads/master by this push:
new 8e86999c6f [#12288] Pass settings.xml profile properties to LRM
(#12297)
8e86999c6f is described below
commit 8e86999c6f7fe74b2c92b3d7a6bd7b74f566d4d2
Author: Guillaume Nodet <[email protected]>
AuthorDate: Wed Jun 17 18:43:34 2026 +0200
[#12288] Pass settings.xml profile properties to LRM (#12297)
Fixes #12288. Supersedes #12291.
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.
Changes:
- Collect activeByDefault profiles for property propagation to LRM
- Respect -P !profileId deactivation for activeByDefault profiles
- Add ITs for both settings.xml activation channels
Co-authored-by: Gerd Aschemann <[email protected]>
---
.../DefaultRepositorySystemSessionFactory.java | 10 +-
...gh12288SettingsProfileAetherPropertiesTest.java | 109 +++++++++++++++++++++
.../settings-profile-aether-properties/pom.xml | 34 +++++++
.../settings-active-by-default.xml | 33 +++++++
.../settings-active-profiles-list.xml | 33 +++++++
5 files changed, 218 insertions(+), 1 deletion(-)
diff --git
a/impl/maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
b/impl/maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
index b25171dcec..aff1423b9c 100644
---
a/impl/maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
+++
b/impl/maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
@@ -433,9 +433,17 @@ private Map<String, String>
getPropertiesFromRequestedProfiles(MavenExecutionReq
HashSet<String> activeProfileId =
new
HashSet<>(request.getProfileActivation().getRequiredActiveProfileIds());
activeProfileId.addAll(request.getProfileActivation().getOptionalActiveProfileIds());
+ // Profiles explicitly deactivated via -P !id must be excluded even if
they
+ // declare activeByDefault=true.
+ HashSet<String> inactiveProfileId =
+ new
HashSet<>(request.getProfileActivation().getRequiredInactiveProfileIds());
+
inactiveProfileId.addAll(request.getProfileActivation().getOptionalInactiveProfileIds());
return request.getProfiles().stream()
- .filter(profile -> activeProfileId.contains(profile.getId()))
+ .filter(profile -> activeProfileId.contains(profile.getId())
+ || (!inactiveProfileId.contains(profile.getId())
+ && profile.getActivation() != null
+ &&
profile.getActivation().isActiveByDefault()))
.map(ModelBase::getProperties)
.flatMap(properties -> properties.entrySet().stream())
.filter(e -> e.getValue() != null)
diff --git
a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITgh12288SettingsProfileAetherPropertiesTest.java
b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITgh12288SettingsProfileAetherPropertiesTest.java
new file mode 100644
index 0000000000..df89e3c790
--- /dev/null
+++
b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITgh12288SettingsProfileAetherPropertiesTest.java
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.it;
+
+import java.io.File;
+
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Integration tests proving that {@code aether.*} properties declared in the
+ * {@code <properties>} block of a {@code settings.xml} profile are honored
+ * by the resolver at local repository manager initialization, regardless of
+ * which settings.xml-only activation channel was used.
+ *
+ * <p>Two activation channels are covered:
+ * <ul>
+ * <li>{@code
<activation><activeByDefault>true</activeByDefault></activation>}
+ * on the profile itself;</li>
+ * <li>{@code
<activeProfiles><activeProfile>...</activeProfile></activeProfiles>}
+ * at the top of {@code settings.xml}.</li>
+ * </ul>
+ *
+ * <p>In both cases the same profile sets:
+ * <pre>
+ * aether.enhancedLocalRepository.split = true
+ * aether.enhancedLocalRepository.localPrefix = it-custom-prefix
+ * </pre>
+ * and the test asserts that {@code mvn install} writes the installed pom
+ * under {@code <localRepo>/it-custom-prefix/<groupId-path>/...} rather
+ * than the flat or default-split layout.
+ *
+ * <p>The same properties on the same profile work correctly when the
+ * profile is activated via {@code -P <id>} on the CLI; only the
+ * settings.xml activation channels fail, which is what these tests guard
+ * against.
+ */
+public class MavenITgh12288SettingsProfileAetherPropertiesTest extends
AbstractMavenIntegrationTestCase {
+
+ @Test
+ public void testActiveByDefaultProfile() throws Exception {
+ runAndAssertCustomPrefix("settings-active-by-default.xml");
+ }
+
+ @Test
+ public void testActiveProfilesList() throws Exception {
+ runAndAssertCustomPrefix("settings-active-profiles-list.xml");
+ }
+
+ private void runAndAssertCustomPrefix(String settingsFile) throws
Exception {
+ File testDir = extractResources("/settings-profile-aether-properties");
+
+ Verifier verifier = newVerifier(testDir.getAbsolutePath());
+ verifier.setAutoclean(false);
+ verifier.deleteDirectory("target");
+
verifier.deleteArtifacts("org.apache.maven.its.settings.profile.aether");
+
+ verifier.addCliArgument("--settings");
+ verifier.addCliArgument(settingsFile);
+ verifier.addCliArgument("install");
+ verifier.execute();
+ verifier.verifyErrorFreeLog();
+
+ File localRepo = new File(verifier.getLocalRepository());
+ String gavRelativePath =
"org/apache/maven/its/settings/profile/aether/test-artifact/1.0/test-artifact-1.0.pom";
+
+ File expectedAtCustomPrefix = new File(localRepo, "it-custom-prefix/"
+ gavRelativePath);
+ File flatLayout = new File(localRepo, gavRelativePath);
+ File defaultSplitPrefix = new File(localRepo, "installed/" +
gavRelativePath);
+
+ assertTrue(
+ expectedAtCustomPrefix.exists(),
+ "Expected install to use custom localPrefix 'it-custom-prefix'
from "
+ + settingsFile
+ + ", but artifact not found at "
+ + expectedAtCustomPrefix);
+
+ assertFalse(
+ flatLayout.exists(),
+ "Found artifact at flat layout "
+ + flatLayout
+ + " — indicates the settings.xml profile properties
did not reach the resolver"
+ + " session config in time for LRM init.");
+
+ assertFalse(
+ defaultSplitPrefix.exists(),
+ "Found artifact at default split-LRM prefix "
+ + defaultSplitPrefix
+ + " — indicates split=true was honored but localPrefix
was silently dropped.");
+ }
+}
diff --git
a/its/core-it-suite/src/test/resources/settings-profile-aether-properties/pom.xml
b/its/core-it-suite/src/test/resources/settings-profile-aether-properties/pom.xml
new file mode 100644
index 0000000000..4e3e07381b
--- /dev/null
+++
b/its/core-it-suite/src/test/resources/settings-profile-aether-properties/pom.xml
@@ -0,0 +1,34 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements. See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership. The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied. See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+<project xmlns="http://maven.apache.org/POM/4.0.0";>
+ <modelVersion>4.0.0</modelVersion>
+
+ <groupId>org.apache.maven.its.settings.profile.aether</groupId>
+ <artifactId>test-artifact</artifactId>
+ <version>1.0</version>
+ <packaging>pom</packaging>
+
+ <name>Maven Integration Test :: Settings Profile Aether Properties</name>
+ <description>
+ Minimal project for proving that aether.enhancedLocalRepository.*
+ properties set in an active-by-default settings.xml profile are honored
+ by the resolver at local repository manager initialization.
+ </description>
+</project>
diff --git
a/its/core-it-suite/src/test/resources/settings-profile-aether-properties/settings-active-by-default.xml
b/its/core-it-suite/src/test/resources/settings-profile-aether-properties/settings-active-by-default.xml
new file mode 100644
index 0000000000..455bcccde2
--- /dev/null
+++
b/its/core-it-suite/src/test/resources/settings-profile-aether-properties/settings-active-by-default.xml
@@ -0,0 +1,33 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements. See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership. The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied. See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0";>
+ <profiles>
+ <profile>
+ <id>aether-split-via-settings</id>
+ <activation>
+ <activeByDefault>true</activeByDefault>
+ </activation>
+ <properties>
+
<aether.enhancedLocalRepository.split>true</aether.enhancedLocalRepository.split>
+
<aether.enhancedLocalRepository.localPrefix>it-custom-prefix</aether.enhancedLocalRepository.localPrefix>
+ </properties>
+ </profile>
+ </profiles>
+</settings>
diff --git
a/its/core-it-suite/src/test/resources/settings-profile-aether-properties/settings-active-profiles-list.xml
b/its/core-it-suite/src/test/resources/settings-profile-aether-properties/settings-active-profiles-list.xml
new file mode 100644
index 0000000000..c40eccb58c
--- /dev/null
+++
b/its/core-it-suite/src/test/resources/settings-profile-aether-properties/settings-active-profiles-list.xml
@@ -0,0 +1,33 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements. See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership. The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied. See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0";>
+ <activeProfiles>
+ <activeProfile>aether-split-via-settings</activeProfile>
+ </activeProfiles>
+ <profiles>
+ <profile>
+ <id>aether-split-via-settings</id>
+ <properties>
+
<aether.enhancedLocalRepository.split>true</aether.enhancedLocalRepository.split>
+
<aether.enhancedLocalRepository.localPrefix>it-custom-prefix</aether.enhancedLocalRepository.localPrefix>
+ </properties>
+ </profile>
+ </profiles>
+</settings>
