Skip to content

Commit 4413f6a

Browse files
vishesh92DaanHoogland
authored and
Locharla, Sandeep
committed
Fix mvn warnings (apache#10909)
* Replace maven-jgit-buildnumber-plugin with thread safe buildnumber-maven-plugin * Fix mysql-connector-java warning * Fix thread safe warning for properties-maven-plugin * Fix mvn build - marvin warnings * Update tools/marvin/README.md * Update tools/marvin/README.md Co-authored-by: dahn <daan.hoogland@gmail.com> --------- Co-authored-by: dahn <daan.hoogland@gmail.com>
1 parent 45dd567 commit 4413f6a

File tree

9 files changed

+56
-27
lines changed

9 files changed

+56
-27
lines changed

‎client/pom.xml‎

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -671,16 +671,22 @@
671671
<build>
672672
<plugins>
673673
<plugin>
674-
<groupId>ru.concerteza.buildnumber</groupId>
675-
<artifactId>maven-jgit-buildnumber-plugin</artifactId>
676-
<version>1.2.6</version>
674+
<groupId>org.codehaus.mojo</groupId>
675+
<artifactId>buildnumber-maven-plugin</artifactId>
676+
<version>3.2.0</version>
677677
<executions>
678678
<execution>
679679
<id>git-buildnumber</id>
680680
<goals>
681-
<goal>extract-buildnumber</goal>
681+
<goal>create</goal>
682682
</goals>
683683
<phase>prepare-package</phase>
684+
<configuration>
685+
<doCheck>false</doCheck>
686+
<doUpdate>false</doUpdate>
687+
<getRevisionOnlyOnce>true</getRevisionOnlyOnce>
688+
<revisionOnScmFailure>unknown</revisionOnScmFailure>
689+
</configuration>
684690
</execution>
685691
</executions>
686692
</plugin>
@@ -693,11 +699,11 @@
693699
<mainClass>org.apache.cloudstack.ServerDaemon</mainClass>
694700
</manifest>
695701
<manifestEntries>
696-
<X-Git-Branch>${git.branch}</X-Git-Branch>
697-
<X-Git-Tag>${git.tag}</X-Git-Tag>
698-
<X-Git-Revision>${git.revision}</X-Git-Revision>
699-
<Implementation-Revision>${git.revision}</Implementation-Revision>
700-
<Implementation-Branch>${git.branch}</Implementation-Branch>
702+
<X-Git-Branch>${scmBranch}</X-Git-Branch>
703+
<X-Git-Tag>${project.version}</X-Git-Tag>
704+
<X-Git-Revision>${buildNumber}</X-Git-Revision>
705+
<Implementation-Revision>${buildNumber}</Implementation-Revision>
706+
<Implementation-Branch>${scmBranch}</Implementation-Branch>
701707
</manifestEntries>
702708
</archive>
703709
</configuration>

‎engine/storage/snapshot/pom.xml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@
5757
<scope>compile</scope>
5858
</dependency>
5959
<dependency>
60-
<groupId>mysql</groupId>
61-
<artifactId>mysql-connector-java</artifactId>
60+
<groupId>com.mysql</groupId>
61+
<artifactId>mysql-connector-j</artifactId>
6262
<scope>test</scope>
6363
</dependency>
6464
</dependencies>

‎framework/db/pom.xml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,8 @@
5353
<artifactId>commons-pool2</artifactId>
5454
</dependency>
5555
<dependency>
56-
<groupId>mysql</groupId>
57-
<artifactId>mysql-connector-java</artifactId>
56+
<groupId>com.mysql</groupId>
57+
<artifactId>mysql-connector-j</artifactId>
5858
</dependency>
5959
<dependency>
6060
<groupId>org.apache.cloudstack</groupId>

‎plugins/network-elements/globodns/pom.xml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@
3333
<artifactId>globodns-client</artifactId>
3434
</dependency>
3535
<dependency>
36-
<groupId>mysql</groupId>
37-
<artifactId>mysql-connector-java</artifactId>
36+
<groupId>com.mysql</groupId>
37+
<artifactId>mysql-connector-j</artifactId>
3838
<scope>test</scope>
3939
</dependency>
4040
</dependencies>

‎plugins/network-elements/tungsten/pom.xml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@
4141
<artifactId>reload4j</artifactId>
4242
</dependency>
4343
<dependency>
44-
<groupId>mysql</groupId>
45-
<artifactId>mysql-connector-java</artifactId>
44+
<groupId>com.mysql</groupId>
45+
<artifactId>mysql-connector-j</artifactId>
4646
<scope>test</scope>
4747
</dependency>
4848
</dependencies>

‎pom.xml‎

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -470,8 +470,8 @@
470470
<version>${cs.reload4j.version}</version>
471471
</dependency>
472472
<dependency>
473-
<groupId>mysql</groupId>
474-
<artifactId>mysql-connector-java</artifactId>
473+
<groupId>com.mysql</groupId>
474+
<artifactId>mysql-connector-j</artifactId>
475475
<version>${cs.mysql.version}</version>
476476
<scope>test</scope>
477477
</dependency>
@@ -486,12 +486,6 @@
486486
</exclusion>
487487
</exclusions>
488488
</dependency>
489-
<dependency>
490-
<groupId>com.mysql</groupId>
491-
<artifactId>mysql-connector-j</artifactId>
492-
<version>${cs.mysql.version}</version>
493-
<scope>test</scope>
494-
</dependency>
495489
<dependency>
496490
<groupId>net.sf.ehcache</groupId>
497491
<artifactId>ehcache-core</artifactId>

‎tools/marvin/README.md‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
<!--
2+
Licensed to the Apache Software Foundation (ASF) under one
3+
or more contributor license agreements. See the NOTICE file
4+
distributed with this work for additional information
5+
regarding copyright ownership. The ASF licenses this file
6+
to you under the Apache License, Version 2.0 (the
7+
"License"); you may not use this file except in compliance
8+
with the License. You may obtain a copy of the License at
9+
10+
http://www.apache.org/licenses/LICENSE-2.0
11+
12+
Unless required by applicable law or agreed to in writing,
13+
software distributed under the License is distributed on an
14+
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
KIND, either express or implied. See the License for the
16+
specific language governing permissions and limitations
17+
under the License.
18+
-->
19+
# Marvin
20+
21+
Marvin is the Apache CloudStack Python client written for running smoke tests.
22+
23+
## Overview
24+
25+
Marvin provides a Python-based client for Apache CloudStack. It offers utilities for testing and interacting with CloudStack deployments.
26+
27+
## License
28+
29+
Licensed under the Apache License, Version 2.0. See the LICENSE.txt file for details.

‎tools/marvin/mvn-setup.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ def replaceVersion(fname, version):
3333
"""replace VERSION in setup.py"""
3434
with open(fname, 'r') as f:
3535
content = f.read()
36-
needle = '\nVERSION\s*=\s*[\'"][^\'"]*[\'"]'
36+
needle = r'\nVERSION\s*=\s*[\'"][^\'"]*[\'"]'
3737
# Ensure the version is PEP440 compliant
3838
version = version.replace('-', '+', 1)
3939
replacement = '\nVERSION = "%s"' % version

‎tools/marvin/setup.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
maintainer_email="dev@cloudstack.apache.org",
3939
long_description="Marvin is the Apache CloudStack python "
4040
"client written around the unittest framework",
41-
platforms=("Any",),
41+
platforms=["Any"],
4242
url="https://builds.apache.org/job/cloudstack-marvin/",
4343
packages=["marvin", "marvin.cloudstackAPI",
4444
"marvin.lib", "marvin.config", "marvin.sandbox",

0 commit comments

Comments
 (0)