Skip to content

Commit 252ab5e

Browse files
gnodetclaude
gnodet
and
claude
committed
[MNG-11642] Use qualified exports for non-API impl packages
Only org.apache.maven.api.* packages are Maven 4's public API. All implementation packages (org.apache.maven.impl.*, org.apache.maven.di.impl, org.apache.maven.slf4j, org.apache.maven.jline) are now qualified exports, restricted to the specific internal modules that need them. Changes: - maven-impl: qualify all org.apache.maven.impl.* exports to maven-core, maven-cli, maven-testing, maven-compat, maven-embedder; remove unused org.apache.maven.impl.model.profile export - maven-di: qualify org.apache.maven.di.impl export - maven-logging: qualify org.apache.maven.slf4j export - maven-jline: qualify org.apache.maven.jline export Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 164fc78 commit 252ab5e

4 files changed

Lines changed: 98 additions & 18 deletions

File tree

‎impl/maven-di/src/main/java/module-info.java‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,5 +22,10 @@
2222
requires org.apache.maven.api.annotations;
2323

2424
exports org.apache.maven.di;
25-
exports org.apache.maven.di.impl;
25+
exports org.apache.maven.di.impl to
26+
org.apache.maven.impl,
27+
org.apache.maven.core,
28+
org.apache.maven.testing,
29+
org.apache.maven.cling,
30+
org.apache.maven.embedder;
2631
}

‎impl/maven-impl/src/main/java/module-info.java‎

Lines changed: 84 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -40,21 +40,90 @@
4040
requires java.xml;
4141

4242
exports org.apache.maven.api.services.model;
43-
exports org.apache.maven.impl;
44-
exports org.apache.maven.impl.cache;
45-
exports org.apache.maven.impl.di;
46-
exports org.apache.maven.impl.model;
47-
exports org.apache.maven.impl.model.profile;
48-
exports org.apache.maven.impl.model.reflection;
49-
exports org.apache.maven.impl.model.rootlocator;
50-
exports org.apache.maven.impl.resolver;
51-
exports org.apache.maven.impl.resolver.artifact;
52-
exports org.apache.maven.impl.resolver.relocation;
53-
exports org.apache.maven.impl.resolver.scopes;
54-
exports org.apache.maven.impl.resolver.type;
55-
exports org.apache.maven.impl.resolver.validator;
56-
exports org.apache.maven.impl.standalone;
57-
exports org.apache.maven.impl.util;
43+
exports org.apache.maven.impl to
44+
org.apache.maven.core,
45+
org.apache.maven.cling,
46+
org.apache.maven.testing,
47+
org.apache.maven.compat,
48+
org.apache.maven.embedder;
49+
exports org.apache.maven.impl.cache to
50+
org.apache.maven.core,
51+
org.apache.maven.cling,
52+
org.apache.maven.testing,
53+
org.apache.maven.compat,
54+
org.apache.maven.embedder;
55+
exports org.apache.maven.impl.di to
56+
org.apache.maven.core,
57+
org.apache.maven.cling,
58+
org.apache.maven.testing,
59+
org.apache.maven.compat,
60+
org.apache.maven.embedder;
61+
exports org.apache.maven.impl.model to
62+
org.apache.maven.core,
63+
org.apache.maven.cling,
64+
org.apache.maven.testing,
65+
org.apache.maven.compat,
66+
org.apache.maven.embedder;
67+
exports org.apache.maven.impl.model.reflection to
68+
org.apache.maven.core,
69+
org.apache.maven.cling,
70+
org.apache.maven.testing,
71+
org.apache.maven.compat,
72+
org.apache.maven.embedder;
73+
exports org.apache.maven.impl.model.rootlocator to
74+
org.apache.maven.core,
75+
org.apache.maven.cling,
76+
org.apache.maven.testing,
77+
org.apache.maven.compat,
78+
org.apache.maven.embedder;
79+
exports org.apache.maven.impl.resolver to
80+
org.apache.maven.core,
81+
org.apache.maven.cling,
82+
org.apache.maven.testing,
83+
org.apache.maven.compat,
84+
org.apache.maven.embedder;
85+
exports org.apache.maven.impl.resolver.artifact to
86+
org.apache.maven.core,
87+
org.apache.maven.cling,
88+
org.apache.maven.testing,
89+
org.apache.maven.compat,
90+
org.apache.maven.embedder;
91+
exports org.apache.maven.impl.resolver.relocation to
92+
org.apache.maven.core,
93+
org.apache.maven.cling,
94+
org.apache.maven.testing,
95+
org.apache.maven.compat,
96+
org.apache.maven.embedder;
97+
exports org.apache.maven.impl.resolver.scopes to
98+
org.apache.maven.core,
99+
org.apache.maven.cling,
100+
org.apache.maven.testing,
101+
org.apache.maven.compat,
102+
org.apache.maven.embedder;
103+
exports org.apache.maven.impl.resolver.type to
104+
org.apache.maven.core,
105+
org.apache.maven.cling,
106+
org.apache.maven.testing,
107+
org.apache.maven.compat,
108+
org.apache.maven.embedder;
109+
exports org.apache.maven.impl.resolver.validator to
110+
org.apache.maven.core,
111+
org.apache.maven.cling,
112+
org.apache.maven.testing,
113+
org.apache.maven.compat,
114+
org.apache.maven.embedder;
115+
exports org.apache.maven.impl.standalone to
116+
org.apache.maven.core,
117+
org.apache.maven.cling,
118+
org.apache.maven.testing,
119+
org.apache.maven.compat,
120+
org.apache.maven.embedder;
121+
exports org.apache.maven.impl.util to
122+
org.apache.maven.core,
123+
org.apache.maven.cling,
124+
org.apache.maven.testing,
125+
org.apache.maven.compat,
126+
org.apache.maven.embedder;
58127

59128
uses org.apache.maven.api.services.model.RootDetector;
60129

‎impl/maven-jline/src/main/java/module-info.java‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,5 +30,8 @@
3030
requires org.jline.terminal.jni;
3131
requires org.jline.jansi.core;
3232

33-
exports org.apache.maven.jline;
33+
exports org.apache.maven.jline to
34+
org.apache.maven.logging,
35+
org.apache.maven.cling,
36+
org.apache.maven.embedder;
3437
}

‎impl/maven-logging/src/main/java/module-info.java‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,10 @@
2323
requires org.slf4j;
2424

2525
exports org.apache.maven.logging.api;
26-
exports org.apache.maven.slf4j;
26+
exports org.apache.maven.slf4j to
27+
org.apache.maven.cling,
28+
org.apache.maven.embedder,
29+
org.apache.maven.core;
2730

2831
provides org.slf4j.spi.SLF4JServiceProvider with
2932
org.apache.maven.slf4j.MavenServiceProvider;

0 commit comments

Comments
 (0)