Fixes in the CMake install setup for EXPORT

- lib64 is not consistently used; usually it's lib, and CMAKE_INSTALL_LIBDIR is the right way to get the name
- the link options for regex_lookahead are already exported; don't need to do them again in our manual config.cmake
- had to unbreak the macros we use to set the link options so that they work when exported, though

Author: swolchok

CMakeLists.txt

@@ -61,6 +61,7 @@ file(GLOB unicode_source_files
 add_library(
   tokenizers STATIC ${tokenizers_source_files} ${unicode_source_files}
 )
+add_library(tokenizers::tokenizers ALIAS tokenizers)
 
 # Using abseil from sentencepiece/third_party
 target_include_directories(
@@ -101,6 +102,7 @@ if(SUPPORT_REGEX_LOOKAHEAD)
     ${CMAKE_CURRENT_SOURCE_DIR}/src/regex_lookahead.cpp
     ${CMAKE_CURRENT_SOURCE_DIR}/src/std_regex.cpp
   )
+  add_library(tokenizers::regex_lookahead ALIAS regex_lookahead)
   target_link_libraries(regex_lookahead PUBLIC pcre2-8-static)
   target_include_directories(
     regex_lookahead

Utils.cmake

@@ -20,21 +20,21 @@ function(kernel_link_options target_name)
   # "$<LINK_LIBRARY:WHOLE_ARCHIVE,target_name>")
   target_link_options(
     ${target_name} INTERFACE "SHELL:LINKER:--whole-archive \
-    $<TARGET_FILE:${target_name}> \
+    $<TARGET_FILE:tokenizers::${target_name}> \
     LINKER:--no-whole-archive")
 endfunction()
 
 # Same as kernel_link_options but it's for MacOS linker
 function(macos_kernel_link_options target_name)
   target_link_options(${target_name} INTERFACE
-                      "SHELL:LINKER:-force_load,$<TARGET_FILE:${target_name}>")
+                      "SHELL:LINKER:-force_load,$<TARGET_FILE:tokenizers::${target_name}>")
 endfunction()
 
 # Same as kernel_link_options but it's for MSVC linker
 function(msvc_kernel_link_options target_name)
   target_link_options(
     ${target_name} INTERFACE
-    "SHELL:LINKER:/WHOLEARCHIVE:$<TARGET_FILE:${target_name}>")
+    "SHELL:LINKER:/WHOLEARCHIVE:$<TARGET_FILE:tokenizers::${target_name}>")
 endfunction()
 
 # Ensure that the load-time constructor functions run. By default, the linker

cmake/tokenizers-config.cmake.in

@@ -7,12 +7,12 @@
 @PACKAGE_INIT@
 
 include(CMakeFindDependencyMacro)
-
+include(GNUInstallDirs)
 # Find dependencies
 find_dependency(re2 REQUIRED)
 find_dependency(absl REQUIRED)
 # Directly include sentencepiece library
-set(SENTENCEPIECE_LIBRARY "${CMAKE_INSTALL_PREFIX}/lib64/libsentencepiece.a")
+set(SENTENCEPIECE_LIBRARY "${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR}/libsentencepiece.a")
 if(NOT EXISTS "${SENTENCEPIECE_LIBRARY}")
   message(
     FATAL_ERROR
@@ -26,21 +26,6 @@ include("${CMAKE_CURRENT_LIST_DIR}/tokenizers-targets.cmake")
 # Set the include directories
 set_and_check(TOKENIZERS_INCLUDE_DIRS "@PACKAGE_CMAKE_INSTALL_INCLUDEDIR@")
 
-# Add --whole-archive linker flag for tokenizers library
-if(APPLE)
-  set(TOKENIZERS_LINK_OPTIONS
-      "SHELL:-force_load,$<TARGET_FILE:tokenizers::regex_lookahead>"
-  )
-elseif(MSVC)
-  set(TOKENIZERS_LINK_OPTIONS
-      "SHELL:/WHOLEARCHIVE:$<TARGET_FILE:tokenizers::regex_lookahead>"
-  )
-else()
-  set(TOKENIZERS_LINK_OPTIONS
-      "SHELL:LINKER:--whole-archive $<TARGET_FILE:tokenizers::regex_lookahead> LINKER:--no-whole-archive"
-  )
-endif()
-
 # Set the libraries and link options
 set(TOKENIZERS_LIBRARIES tokenizers::tokenizers)
 set_property(