build: Remove test component of lib icu (facebookincubator#11826)

Summary: The test component of lib icu is not used in Velox. This pr simply removes it, assuming there is no reason for keeping it. Pull Request resolved: facebookincubator#11826 Reviewed By: xiaoxmeng Differential Revision: D67119218 Pulled By: kgpai fbshipit-source-id: a1a91e2e6c07ce699b2c23bf999ef0793070bd38
spershin · Dec 12, 2024 · 63fcb27 · 63fcb27
1 parent 8d0d244
commit 63fcb27
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/CMake/resolve_dependency_modules/icu.cmake b/CMake/resolve_dependency_modules/icu.cmake
@@ -70,8 +70,7 @@ set(icu_components
     i18n
     io
     uc
-    tu
-    test)
+    tu)
 
 foreach(component ${icu_components})
   add_library(ICU::${component} SHARED IMPORTED)

diff --git a/CMake/resolve_dependency_modules/icu/FindICU.cmake b/CMake/resolve_dependency_modules/icu/FindICU.cmake
@@ -13,7 +13,12 @@
 # limitations under the License.
 message("Using ICU - Bundled")
 set(ICU_FOUND TRUE)
-set(icu_components data i18n io uc tu test)
+set(icu_components
+    data
+    i18n
+    io
+    uc
+    tu)
 foreach(component icu_components)
   set(ICU_${component}_FOUND TRUE)
 endforeach()
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -427,8 +427,7 @@ velox_resolve_dependency(
   i18n
   io
   uc
-  tu
-  test)
+  tu)
 
 set(BOOST_INCLUDE_LIBRARIES
     atomic
-Original file line number
+Diff line change
@@ Expand Up / @@ -70,8 +70,7 @@ set(icu_components @@
         i18n
         io
         uc
-        tu
-        test)
+        tu)
     foreach(component ${icu_components})
       add_library(ICU::${component} SHARED IMPORTED)
@@ Expand Down @@