From 208d3bb14fb1d74b89ee7678903e70f0c560b4be Mon Sep 17 00:00:00 2001 From: Ramakrishna Prabhu Date: Fri, 10 Apr 2026 14:28:59 -0500 Subject: [PATCH] Fix duplicate libmps_parser.so in libcuopt wheel The mps_parser install in libmps_parser/CMakeLists.txt hardcoded `DESTINATION lib` while the parent cuopt CMakeLists.txt installs to the platform-correct lib directory (lib64 on x86_64) via rapids_cmake_install_lib_dir(). This caused libmps_parser.so to appear in both lib/ and lib64/ in the wheel, wasting ~350 KB. Use rapids_cmake_install_lib_dir() to resolve the correct library directory, eliminating the duplicate. --- cpp/libmps_parser/CMakeLists.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cpp/libmps_parser/CMakeLists.txt b/cpp/libmps_parser/CMakeLists.txt index 427d4ac17b..ea6b50cff2 100644 --- a/cpp/libmps_parser/CMakeLists.txt +++ b/cpp/libmps_parser/CMakeLists.txt @@ -136,8 +136,9 @@ endif(BUILD_TESTS) # ################################################################################################## # * mps_parser Install ---------------------------------------------------------------------------- +rapids_cmake_install_lib_dir(mps_parser_lib_dir) install(TARGETS mps_parser - DESTINATION lib + DESTINATION ${mps_parser_lib_dir} EXPORT mps-parser-exports) install(DIRECTORY include/mps_parser/