arturobernalg commented on code in PR #13727:
URL: https://github.com/apache/maven/pull/13727#discussion_r4388237892
##########
maven-core/src/test/java/org/apache/maven/project/ExtensionDescriptorBuilderTest.java:
##########
@@ -76,4 +76,28 @@ public void testCompleteDescriptor() throws Exception {
assertEquals(Arrays.asList("a", "b", "c"), ed.getExportedPackages());
assertEquals(Arrays.asList("x", "y", "z"), ed.getExportedArtifacts());
}
+
+ /**
+ * maven-core's own {@code META-INF/maven/extension.xml} must not export
an artifact or package that
+ * maven-core no longer actually bundles: a stale export causes {@link
org.apache.maven.ArtifactFilterManager}
+ * to filter that dependency out of a plugin's classpath even though Maven
core does not provide it, breaking
+ * any plugin that declares it directly (see MNG-8724, jansi left exported
after the JAnsi-to-JLine migration).
Review Comment:
Good catch, fixed in 1de5952.
--
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]
