From 8c62620dedc054930ff63d618c687e5b7c41b6e5 Mon Sep 17 00:00:00 2001
From: oxsean <oxsean@gmail.com>
Date: Tue, 7 Nov 2023 08:09:09 +0000
Subject: [PATCH] fix(pom): Merge metadata module into autoconfigure module

---
 .artifacts                                    |   1 -
 dubbo-distribution/dubbo-bom/pom.xml          |   5 -
 .../{metadata => autoconfigure}/README.md     |   6 +-
 .../autoconfigure/pom.xml                     | 105 ++++++++--
 .../DubboConfigurationProperties.java         |   4 +-
 ...ubboMetadataGenerateAutoConfiguration.java |   0
 ...itional-spring-configuration-metadata.json |   0
 .../metadata/pom.xml                          | 187 ------------------
 .../dubbo-spring-boot-compatible/pom.xml      |   1 -
 dubbo-test/dubbo-dependencies-all/pom.xml     |   5 -
 10 files changed, 94 insertions(+), 220 deletions(-)
 rename dubbo-spring-boot/dubbo-spring-boot-compatible/{metadata => autoconfigure}/README.md (79%)
 rename dubbo-spring-boot/dubbo-spring-boot-compatible/{metadata => autoconfigure}/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboMetadataGenerateAutoConfiguration.java (100%)
 rename dubbo-spring-boot/dubbo-spring-boot-compatible/{metadata => autoconfigure}/src/main/resources/META-INF/additional-spring-configuration-metadata.json (100%)
 delete mode 100644 dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/pom.xml

diff --git a/.artifacts b/.artifacts
index db1bbb11533f..5a8fb62cfc09 100644
--- a/.artifacts
+++ b/.artifacts
@@ -107,7 +107,6 @@ dubbo-spring-boot-actuator
 dubbo-spring-boot-actuator-compatible
 dubbo-spring-boot-autoconfigure
 dubbo-spring-boot-autoconfigure-compatible
-dubbo-spring-boot-configuration-metadata-compatible
 dubbo-spring-boot-compatible
 dubbo-observability-spring-boot-starters
 dubbo-observability-spring-boot-autoconfigure
diff --git a/dubbo-distribution/dubbo-bom/pom.xml b/dubbo-distribution/dubbo-bom/pom.xml
index fcb99025dc2c..f0e941af742c 100644
--- a/dubbo-distribution/dubbo-bom/pom.xml
+++ b/dubbo-distribution/dubbo-bom/pom.xml
@@ -643,11 +643,6 @@
                 <artifactId>dubbo-spring-boot-autoconfigure-compatible</artifactId>
                 <version>${project.version}</version>
             </dependency>
-            <dependency>
-                <groupId>org.apache.dubbo</groupId>
-                <artifactId>dubbo-spring-boot-configuration-metadata-compatible</artifactId>
-                <version>${project.version}</version>
-            </dependency>
             <dependency>
                 <groupId>org.apache.dubbo</groupId>
                 <artifactId>dubbo-spring-boot-starter</artifactId>
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/README.md b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/README.md
similarity index 79%
rename from dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/README.md
rename to dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/README.md
index d2119d0d8c1c..99008e743737 100644
--- a/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/README.md
+++ b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/README.md
@@ -1,4 +1,4 @@
-# About dubbo-spring-boot-configuration-metadata-compatible module
+# About spring-configuration-metadata
 
 ## Why
 
@@ -10,9 +10,7 @@ This leads to missing comments and a lack of nested configuration options. There
 
 1. Copy classes under `org/apache/dubbo/config` from `dubbo-common` to the `generated-sources` directory.
 2. Replace `@Nest` with `@NestedConfigurationProperty`.
-3. Copy the class `DubboConfigurationProperties.java` from `autoconfigure` to the `generated-sources` directory.
-4. Use an annotation-only option to compile and generate `spring-configuration-metadata.json`.
-5. During `autoconfigure` module compilation, will read `spring-configuration-metadata.json` from this module.
+3. Use an annotation-only option to compile and generate `spring-configuration-metadata.json`.
 
 ## How to add a new configuration option
 
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/pom.xml b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/pom.xml
index 98c6f80fc3ae..1d9f92a5a90e 100644
--- a/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/pom.xml
+++ b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/pom.xml
@@ -29,6 +29,10 @@
     <artifactId>dubbo-spring-boot-autoconfigure-compatible</artifactId>
     <description>Apache Dubbo Spring Boot Compatible for Spring Boot 1.x Auto-Configure</description>
 
+    <properties>
+        <sources_directory>${project.build.directory}/generated-sources/java</sources_directory>
+    </properties>
+
     <dependencies>
         <!-- Spring Boot dependencies -->
         <dependency>
@@ -70,10 +74,9 @@
         </dependency>
 
         <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-spring-boot-configuration-metadata-compatible</artifactId>
-            <version>${project.version}</version>
-            <scope>provided</scope>
+            <groupId>org.springframework.boot</groupId>
+            <artifactId>spring-boot-configuration-processor</artifactId>
+            <optional>true</optional>
         </dependency>
 
         <!-- Test Dependencies -->
@@ -85,17 +88,87 @@
     </dependencies>
 
     <build>
-        <resources>
-            <resource>
-                <directory>src/main/resources</directory>
-            </resource>
-            <resource>
-                <targetPath>META-INF</targetPath>
-                <directory>../metadata/target/classes/META-INF</directory>
-                <includes>
-                    <include>spring-configuration-metadata.json</include>
-                </includes>
-            </resource>
-        </resources>
+        <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-antrun-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>copy-sources</id>
+                        <phase>generate-sources</phase>
+                        <goals>
+                            <goal>run</goal>
+                        </goals>
+                        <configuration>
+                            <target>
+                                <delete dir="${sources_directory}" quiet="true"/>
+                                <copy todir="${sources_directory}">
+                                    <fileset dir="../../../dubbo-common/src/main/java">
+                                        <include name="org/apache/dubbo/config/**/*.java"/>
+                                    </fileset>
+                                </copy>
+                                <replace token="@Nested" value="@org.springframework.boot.context.properties.NestedConfigurationProperty">
+                                    <fileset dir="${sources_directory}">
+                                        <include name="**/*.java"/>
+                                    </fileset>
+                                </replace>
+                            </target>
+                        </configuration>
+                    </execution>
+                    <execution>
+                        <id>clean-sources</id>
+                        <phase>process-resources</phase>
+                        <goals>
+                            <goal>run</goal>
+                        </goals>
+                        <configuration>
+                            <target>
+                                <delete dir="${sources_directory}" quiet="true"/>
+                            </target>
+                        </configuration>
+                    </execution>
+                </executions>
+            </plugin>
+            <plugin>
+                <groupId>org.codehaus.mojo</groupId>
+                <artifactId>build-helper-maven-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>add-sources</id>
+                        <phase>generate-sources</phase>
+                        <goals>
+                            <goal>add-source</goal>
+                        </goals>
+                        <configuration>
+                            <sources>
+                                <source>${sources_directory}</source>
+                            </sources>
+                        </configuration>
+                    </execution>
+                </executions>
+            </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-compiler-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>gen-metadata</id>
+                        <phase>process-resources</phase>
+                        <goals>
+                            <goal>compile</goal>
+                        </goals>
+                        <configuration combine.self="override">
+                            <proc>only</proc>
+                            <fork>true</fork>
+                        </configuration>
+                    </execution>
+                </executions>
+                <configuration>
+                    <excludes>
+                        <exclude>**/DubboMetadataGenerateAutoConfiguration.java</exclude>
+                    </excludes>
+                </configuration>
+            </plugin>
+        </plugins>
     </build>
 </project>
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboConfigurationProperties.java b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboConfigurationProperties.java
index efdcf9bc65f1..7186970c1927 100644
--- a/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboConfigurationProperties.java
+++ b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboConfigurationProperties.java
@@ -36,13 +36,15 @@
 import java.util.LinkedHashMap;
 import java.util.Map;
 
+import static org.apache.dubbo.spring.boot.util.DubboUtils.DUBBO_PREFIX;
+
 /**
  * Dubbo {@link ConfigurationProperties Config Properties} only used to generate JSON metadata (non-public class)
  *
  * @see ConfigKeys
  * @since 2.7.1
  */
-@ConfigurationProperties("dubbo")
+@ConfigurationProperties(DUBBO_PREFIX)
 public class DubboConfigurationProperties {
 
     /**
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboMetadataGenerateAutoConfiguration.java b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboMetadataGenerateAutoConfiguration.java
similarity index 100%
rename from dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboMetadataGenerateAutoConfiguration.java
rename to dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboMetadataGenerateAutoConfiguration.java
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/src/main/resources/META-INF/additional-spring-configuration-metadata.json b/dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/resources/META-INF/additional-spring-configuration-metadata.json
similarity index 100%
rename from dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/src/main/resources/META-INF/additional-spring-configuration-metadata.json
rename to dubbo-spring-boot/dubbo-spring-boot-compatible/autoconfigure/src/main/resources/META-INF/additional-spring-configuration-metadata.json
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/pom.xml b/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/pom.xml
deleted file mode 100644
index ff56b943b440..000000000000
--- a/dubbo-spring-boot/dubbo-spring-boot-compatible/metadata/pom.xml
+++ /dev/null
@@ -1,187 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
-  Licensed to the Apache Software Foundation (ASF) under one or more
-  contributor license agreements.  See the NOTICE file distributed with
-  this work for additional information regarding copyright ownership.
-  The ASF licenses this file to You under the Apache License, Version 2.0
-  (the "License"); you may not use this file except in compliance with
-  the License.  You may obtain a copy of the License at
-
-      http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-  -->
-<project xmlns="http://maven.apache.org/POM/4.0.0"
-         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-    <parent>
-        <groupId>org.apache.dubbo</groupId>
-        <artifactId>dubbo-spring-boot-compatible</artifactId>
-        <version>${revision}</version>
-        <relativePath>../pom.xml</relativePath>
-    </parent>
-    <modelVersion>4.0.0</modelVersion>
-
-    <artifactId>dubbo-spring-boot-configuration-metadata-compatible</artifactId>
-
-    <properties>
-        <skip_maven_deploy>true</skip_maven_deploy>
-        <sources_directory>${project.build.directory}/generated-sources/java</sources_directory>
-    </properties>
-
-    <dependencies>
-        <dependency>
-            <groupId>org.slf4j</groupId>
-            <artifactId>slf4j-api</artifactId>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>log4j</groupId>
-            <artifactId>log4j</artifactId>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-api</artifactId>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.logging.log4j</groupId>
-            <artifactId>log4j-core</artifactId>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>org.springframework.boot</groupId>
-            <artifactId>spring-boot-autoconfigure</artifactId>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-common</artifactId>
-            <version>${project.version}</version>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>org.springframework.boot</groupId>
-            <artifactId>spring-boot-configuration-processor</artifactId>
-            <optional>true</optional>
-        </dependency>
-    </dependencies>
-
-    <build>
-        <plugins>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-antrun-plugin</artifactId>
-                <executions>
-                    <execution>
-                    <id>get-version-infos</id>
-                    <phase>none</phase>
-                    </execution>
-                    <execution>
-                        <id>copy-sources</id>
-                        <phase>process-sources</phase>
-                        <goals>
-                            <goal>run</goal>
-                        </goals>
-                        <configuration>
-                            <target>
-                                <delete dir="${sources_directory}" quiet="true"/>
-                                <copy todir="${sources_directory}">
-                                    <fileset dir="../../../dubbo-common/src/main/java">
-                                        <include name="org/apache/dubbo/config/**/*.java"/>
-                                    </fileset>
-                                </copy>
-                                <replace token="@Nested" value="@org.springframework.boot.context.properties.NestedConfigurationProperty">
-                                    <fileset dir="${sources_directory}">
-                                        <include name="**/*.java"/>
-                                    </fileset>
-                                </replace>
-                                <copy todir="${sources_directory}">
-                                    <fileset dir="../autoconfigure/src/main/java">
-                                        <include name="**/DubboConfigurationProperties.java"/>
-                                    </fileset>
-                                </copy>
-                            </target>
-                        </configuration>
-                    </execution>
-                    <execution>
-                        <id>clean-sources</id>
-                        <phase>compile</phase>
-                        <goals>
-                            <goal>run</goal>
-                        </goals>
-                        <configuration>
-                            <target>
-                                <delete dir="${sources_directory}" quiet="true"/>
-                            </target>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
-            <plugin>
-                <groupId>org.codehaus.mojo</groupId>
-                <artifactId>build-helper-maven-plugin</artifactId>
-                <executions>
-                    <execution>
-                        <id>add-source</id>
-                        <phase>generate-sources</phase>
-                        <goals>
-                            <goal>add-source</goal>
-                        </goals>
-                        <configuration>
-                            <sources>
-                                <source>${sources_directory}</source>
-                            </sources>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-compiler-plugin</artifactId>
-                <configuration>
-                    <proc>only</proc>
-                </configuration>
-            </plugin>
-            <plugin>
-                <artifactId>maven-surefire-plugin</artifactId>
-                <configuration>
-                    <skip>true</skip>
-                </configuration>
-            </plugin>
-            <plugin>
-                <artifactId>maven-javadoc-plugin</artifactId>
-                <configuration>
-                    <skip>true</skip>
-                </configuration>
-            </plugin>
-            <plugin>
-                <artifactId>maven-jar-plugin</artifactId>
-                <executions>
-                    <execution>
-                        <id>default-jar</id>
-                        <phase>none</phase>
-                    </execution>
-                </executions>
-            </plugin>
-            <plugin>
-                <artifactId>maven-install-plugin</artifactId>
-                <configuration>
-                    <skip>true</skip>
-                </configuration>
-            </plugin>
-            <plugin>
-                <artifactId>maven-source-plugin</artifactId>
-                <configuration>
-                    <skipSource>true</skipSource>
-                </configuration>
-            </plugin>
-        </plugins>
-    </build>
-</project>
diff --git a/dubbo-spring-boot/dubbo-spring-boot-compatible/pom.xml b/dubbo-spring-boot/dubbo-spring-boot-compatible/pom.xml
index 926e7b27b030..1526635a7f73 100644
--- a/dubbo-spring-boot/dubbo-spring-boot-compatible/pom.xml
+++ b/dubbo-spring-boot/dubbo-spring-boot-compatible/pom.xml
@@ -37,7 +37,6 @@
     <modules>
         <module>autoconfigure</module>
         <module>actuator</module>
-        <module>metadata</module>
     </modules>
 
     <profiles>
diff --git a/dubbo-test/dubbo-dependencies-all/pom.xml b/dubbo-test/dubbo-dependencies-all/pom.xml
index a9834cd0edc6..ab5635e36e6f 100644
--- a/dubbo-test/dubbo-dependencies-all/pom.xml
+++ b/dubbo-test/dubbo-dependencies-all/pom.xml
@@ -517,11 +517,6 @@
             <artifactId>dubbo-spring-boot-autoconfigure-compatible</artifactId>
             <version>${project.version}</version>
         </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-spring-boot-configuration-metadata-compatible</artifactId>
-            <version>${project.version}</version>
-        </dependency>
         <dependency>
             <groupId>org.apache.dubbo</groupId>
             <artifactId>dubbo-spring-boot-starter</artifactId>