Move BuildProperties to the client

apache · May 2, 2022 · 70e2bc0 · 70e2bc0
1 parent d377916
commit 70e2bc0
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 31 deletions.
diff --git a/client/pom.xml b/client/pom.xml
@@ -74,6 +74,21 @@
             </resource>
         </resources>
         <plugins>
+            <plugin>
+                <groupId>org.codehaus.mojo</groupId>
+                <artifactId>buildnumber-maven-plugin</artifactId>
+                <configuration>
+                    <locale>en_US</locale>
+                </configuration>
+                <executions>
+                    <execution>
+                        <phase>validate</phase>
+                        <goals>
+                            <goal>create</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
             <plugin>
                 <groupId>io.takari.maven.plugins</groupId>
                 <artifactId>takari-lifecycle-plugin</artifactId>

diff --git a/...vndaemon/mvnd/common/BuildProperties.java → ...vndaemon/mvnd/client/BuildProperties.java b/...vndaemon/mvnd/common/BuildProperties.java → ...vndaemon/mvnd/client/BuildProperties.java
@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.mvndaemon.mvnd.common;
+package org.mvndaemon.mvnd.client;
 
 import java.io.IOException;
 import java.io.InputStream;

diff --git a/client/src/main/java/org/mvndaemon/mvnd/client/DaemonParameters.java b/client/src/main/java/org/mvndaemon/mvnd/client/DaemonParameters.java
@@ -41,7 +41,6 @@
 import org.apache.maven.cli.internal.extension.model.io.xpp3.CoreExtensionsXpp3Reader;
 import org.codehaus.plexus.util.StringUtils;
 import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
-import org.mvndaemon.mvnd.common.BuildProperties;
 import org.mvndaemon.mvnd.common.Environment;
 import org.mvndaemon.mvnd.common.Os;
 import org.mvndaemon.mvnd.common.SocketFamily;

diff --git a/client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java b/client/src/main/java/org/mvndaemon/mvnd/client/DefaultClient.java
@@ -39,7 +39,6 @@
 import org.fusesource.jansi.internal.CLibrary;
 import org.jline.utils.AttributedString;
 import org.jline.utils.AttributedStyle;
-import org.mvndaemon.mvnd.common.BuildProperties;
 import org.mvndaemon.mvnd.common.DaemonException;
 import org.mvndaemon.mvnd.common.DaemonInfo;
 import org.mvndaemon.mvnd.common.DaemonRegistry;
@@ -187,11 +186,10 @@ public ExecutionResult execute(ClientOutput output, List<String> argv) {
         if (version || showVersion || debug) {
             // Print mvnd version
             BuildProperties buildProperties = BuildProperties.getInstance();
-            final String mvndVersionString = "mvnd "
-                    + (Environment.isNative() ? "native client " : "JVM client ")
-                    + buildProperties.getVersion()
-                    + "-" + buildProperties.getOsName()
-                    + "-" + buildProperties.getOsArch()
+            final String mvndVersionString = "mvnd " + buildProperties.getVersion()
+                    + (Environment.isNative()
+                            ? buildProperties.getOsName() + "-" + buildProperties.getOsArch() + " native client"
+                            : "JVM client")
                     + " (" + buildProperties.getRevision() + ")";
 
             boolean isColored = !"never".equals(Environment.MAVEN_COLOR.getCommandLineOption(args));

diff --git a/...rg/mvndaemon/mvnd/common/build.properties → ...rg/mvndaemon/mvnd/client/build.properties b/...rg/mvndaemon/mvnd/common/build.properties → ...rg/mvndaemon/mvnd/client/build.properties
@@ -16,7 +16,6 @@
 # under the License.
 
 revision=${buildNumber}
-timestamp=${timestamp}
 version=${project.version}
 os.detected.name=${os.detected.name}
 os.detected.arch=${os.detected.arch}
diff --git a/common/pom.xml b/common/pom.xml
@@ -59,29 +59,7 @@
     </dependencies>
 
     <build>
-        <resources>
-            <resource>
-                <directory>${basedir}/src/main/resources</directory>
-                <filtering>true</filtering>
-            </resource>
-        </resources>
         <plugins>
-            <plugin>
-                <groupId>org.codehaus.mojo</groupId>
-                <artifactId>buildnumber-maven-plugin</artifactId>
-                <configuration>
-                    <locale>en_US</locale>
-                    <timestampFormat>{0,date,yyyy-MM-dd'T'HH:mm:ssX}</timestampFormat>
-                </configuration>
-                <executions>
-                    <execution>
-                        <phase>validate</phase>
-                        <goals>
-                            <goal>create</goal>
-                        </goals>
-                    </execution>
-                </executions>
-            </plugin>
             <plugin>
                 <groupId>org.mvndaemon.mvnd</groupId>
                 <artifactId>mvnd-build-maven-plugin</artifactId>