From 61e33f77aa516fa51591d20d08068d2ade2a6be2 Mon Sep 17 00:00:00 2001
From: Jack Green <jack.green@hazelcast.com>
Date: Tue, 23 Jan 2024 06:57:04 +0000
Subject: [PATCH] Fix `deploy-snapshot.yml` GitHub action (#304)

[`deploy-snapshot.yml` consistently
fails](https://github.com/hazelcast/hazelcast-jdbc/actions/workflows/deploy-snapshot.yml)
with error:
```
Failed to execute goal
org.apache.maven.plugins:maven-source-plugin:3.3.0:jar-no-fork (default)
on project hazelcast-jdbc-core: Presumably you have configured
maven-source-plugn to execute twice times in your build. You have to
configure a classifier for at least on of them.
```

This is because two different `maven-source-plugin` are configured with
different `id`s and `goal`s.

Changes:
- Unify `maven-source-plugin` configurations
   - `id`s
   - `goal`s
- Specify `license-maven-plugin` version
   - resolves `'build.plugins.plugin.version' for
org.codehaus.mojo:license-maven-plugin is missing.`
---
 hazelcast-jdbc-core/pom.xml |  1 -
 pom.xml                     | 10 ++++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/hazelcast-jdbc-core/pom.xml b/hazelcast-jdbc-core/pom.xml
index f910e2d8..9297f67b 100644
--- a/hazelcast-jdbc-core/pom.xml
+++ b/hazelcast-jdbc-core/pom.xml
@@ -63,7 +63,6 @@
         <dependency>
             <groupId>org.assertj</groupId>
             <artifactId>assertj-core</artifactId>
-            <version>3.25.1</version>
             <scope>test</scope>
         </dependency>
         <dependency>
diff --git a/pom.xml b/pom.xml
index ac848ef8..3a592094 100644
--- a/pom.xml
+++ b/pom.xml
@@ -115,7 +115,7 @@
             <dependency>
                 <groupId>org.assertj</groupId>
                 <artifactId>assertj-core</artifactId>
-                <version>${assertj.version}</version>
+                <version>3.25.1</version>
                 <scope>test</scope>
             </dependency>
         </dependencies>
@@ -194,7 +194,7 @@
                         <execution>
                             <id>attach-sources</id>
                             <goals>
-                                <goal>jar</goal>
+                                <goal>jar-no-fork</goal>
                             </goals>
                         </execution>
                     </executions>
@@ -232,6 +232,11 @@
                         </execution>
                     </executions>
                 </plugin>
+                <plugin>
+                    <groupId>org.codehaus.mojo</groupId>
+                    <artifactId>license-maven-plugin</artifactId>
+                    <version>2.3.0</version>
+                </plugin>
             </plugins>
         </pluginManagement>
     </build>
@@ -474,6 +479,7 @@
                         <artifactId>maven-source-plugin</artifactId>
                         <executions>
                             <execution>
+                                <id>attach-sources</id>
                                 <goals>
                                     <goal>jar-no-fork</goal>
                                 </goals>