From 1293012385a828a5267156f92301e6c513dd06a5 Mon Sep 17 00:00:00 2001
From: Hongze Zhang <hongze.zhang@intel.com>
Date: Wed, 11 Dec 2024 09:31:10 +0800
Subject: [PATCH] fixup

---
 backends-velox/pom.xml   | 27 +++++++++++++++++++++------
 gluten-substrait/pom.xml | 31 ++++++++++++++++++++++++-------
 2 files changed, 45 insertions(+), 13 deletions(-)

diff --git a/backends-velox/pom.xml b/backends-velox/pom.xml
index 9821682cc796..70274be4b8d0 100755
--- a/backends-velox/pom.xml
+++ b/backends-velox/pom.xml
@@ -215,12 +215,6 @@
   </dependencies>
 
   <build>
-    <resources>
-      <resource>
-        <directory>${cpp.releases.dir}</directory>
-        <targetPath>${platform}/${arch}</targetPath>
-      </resource>
-    </resources>
     <outputDirectory>target/scala-${scala.binary.version}/classes</outputDirectory>
     <testOutputDirectory>target/scala-${scala.binary.version}/test-classes</testOutputDirectory>
     <plugins>
@@ -274,6 +268,27 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>org.codehaus.mojo</groupId>
+        <artifactId>build-helper-maven-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>add-cpp-libraries</id>
+            <phase>generate-resources</phase>
+            <goals>
+              <goal>add-resource</goal>
+            </goals>
+            <configuration>
+              <resources>
+                <resource>
+                  <directory>${cpp.releases.dir}</directory>
+                  <targetPath>${platform}/${arch}</targetPath>
+                </resource>
+              </resources>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
     </plugins>
   </build>
 </project>
diff --git a/gluten-substrait/pom.xml b/gluten-substrait/pom.xml
index e8eb6a57e6c4..2183f9b15982 100644
--- a/gluten-substrait/pom.xml
+++ b/gluten-substrait/pom.xml
@@ -181,13 +181,6 @@
   <build>
     <outputDirectory>target/scala-${scala.binary.version}/classes</outputDirectory>
     <testOutputDirectory>target/scala-${scala.binary.version}/test-classes</testOutputDirectory>
-    <resources>
-      <resource>
-        <filtering>true</filtering>
-        <!-- Include the properties file to provide the build information. -->
-        <directory>${project.build.directory}/extra-resources</directory>
-      </resource>
-    </resources>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
@@ -217,6 +210,30 @@
           </execution>
         </executions>
       </plugin>
+      <!-- Should always use build-helper-maven-plugin to assign additional resource folders to
+       make sure the default resource folders, namely src/main/resource is not overridden. -->
+      <plugin>
+        <groupId>org.codehaus.mojo</groupId>
+        <artifactId>build-helper-maven-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>add-generated-resources</id>
+            <phase>generate-resources</phase>
+            <goals>
+              <goal>add-resource</goal>
+            </goals>
+            <configuration>
+              <resources>
+                <resource>
+                  <filtering>true</filtering>
+                  <!-- Include the properties file to provide the build information. -->
+                  <directory>${project.build.directory}/extra-resources</directory>
+                </resource>
+              </resources>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
       <!-- compile proto buffer files using copied protoc binary -->
       <plugin>
         <groupId>org.xolstice.maven.plugins</groupId>