maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Olaf Flebbe <>
Subject Trying to fix integration tests of mvn-site-plugin
Date Sun, 18 Nov 2018 15:09:52 GMT

I looked into fixing the Hudson build of maven-site-plugin <>

There are two "it" failing, one of them is the fullreporting. This fullreporting test seems
to run something like

mvn -Preporting site 

[ERROR] ProjectBuildingException for commons-logging:commons-logging:jar:1.1:compile: Some
problems were encountered while processing the POMs:
[ERROR] 'distributionManagement.status' must not be specified. @ line 189, column 13

In fact commons-logging pom includes an <status> tag within <distributionManagement>
which seems to be not so valid. And there are a couple of other poms with the same "problem".

I do know nothing about the history of this tag and do find a commit in spring-gradle-plugins
to work around exactly the same problem

and a discussion that this tag may be obsolete: <>

I am asking myself if maven should relax the DefaultModelValidator. At least from the Maven
Model <>
it looks like to be valid.

diff --git a/maven-model-builder/src/main/java/org/apache/maven/model/validation/
index 1c8477678..b50b0f2e7 100644
--- a/maven-model-builder/src/main/java/org/apache/maven/model/validation/
+++ b/maven-model-builder/src/main/java/org/apache/maven/model/validation/
@@ -444,12 +444,6 @@ public void validateEffectiveModel( Model m, ModelBuildingRequest request,
             DistributionManagement distMgmt = m.getDistributionManagement();
             if ( distMgmt != null )
-                if ( distMgmt.getStatus() != null )
-                {
-                    addViolation( problems, Severity.ERROR, Version.V20, "distributionManagement.status",
-                                  "must not be specified.", distMgmt );
-                }
                 validate20EffectiveRepository( problems, distMgmt.getRepository(), "distributionManagement.repository",
                                                request );
                 validate20EffectiveRepository( problems, distMgmt.getSnapshotRepository(),


  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message