You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2021/03/16 12:54:25 UTC
[GitHub] [activemq-artemis] gemmellr opened a new pull request #3494: ARTEMIS-3183: resolve various immediate warnings at build start
gemmellr opened a new pull request #3494:
URL: https://github.com/apache/activemq-artemis/pull/3494
One of the first things seen when running the build is a bunch of warnings, some which could even break the build in future, so this looks to resolve them:
```
Warning:
Warning: Some problems were encountered while building the effective model for org.apache.activemq:artemis-amqp-protocol:bundle:2.18.0-SNAPSHOT
Warning: 'dependencies.dependency.(groupId:artifactId:type:classifier)' must be unique: org.apache.activemq:artemis-commons:jar -> duplicate declaration of version ${project.version} @ line 83, column 19
Warning:
Warning: Some problems were encountered while building the effective model for org.apache.activemq.tests:smoke-tests:jar:2.18.0-SNAPSHOT
Warning: 'dependencies.dependency.version' for org.eclipse.paho:org.eclipse.paho.client.mqttv3:jar is either LATEST or RELEASE (both of them are being deprecated) @ line 121, column 19
Warning:
Warning: It is highly recommended to fix these problems because they threaten the stability of your build.
Warning:
Warning: For this reason, future Maven versions might no longer support building such malformed projects.
Warning:
Warning: The project org.apache.activemq:artemis-pom:pom:2.18.0-SNAPSHOT uses prerequisites which is only intended for maven-plugin projects but not for non maven-plugin projects. For such purposes you should use the maven-enforcer-plugin. See https://maven.apache.org/enforcer/enforcer-rules/requireMavenVersion.html
```
The first is due to a dependency entry being duplicated in https://github.com/apache/activemq-artemis/commit/e7e3c71511770442cbb206cefcae25001566d75b. Fix is simply to remove one.
The second is due to use of the deprecated 'RELEASE' floating version in the smoke tests module, fix is jsut to stop using the deprecated name. It turns out the build currently uses 3 different versions of this client for tests in different places: 1.2.0, 1.2.2, and the floating RELEASE being 1.2.5 for several months. I introduced a dependency management entry and fixed them all to the 1.2.5 version, as I see no particular reason for it being a floating version and none of the others appear to be.
The last warning is about using the prerequisites element, and suggested fix is to use the enforcer plugin. That is actually already in place, so fix is simply removal.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [activemq-artemis] asfgit merged pull request #3494: ARTEMIS-3183: resolve various immediate warnings at build start
Posted by GitBox <gi...@apache.org>.
asfgit merged pull request #3494:
URL: https://github.com/apache/activemq-artemis/pull/3494
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org