Skip to content

Commit

Permalink
Fix MODL_Coclustering double size in windows
Browse files Browse the repository at this point in the history
The MODL_Coclustering.exe executable in windows had doubled in size
because the MSVC link option `/INCREMENTAL` was activated.

This wasn't the case for the original build with VS projects.
  • Loading branch information
folmos-at-orange committed Nov 24, 2023
1 parent c3c41e3 commit 8de3c74
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions src/Learning/MODL_Coclustering/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,15 +1,13 @@
file(GLOB cppfiles ${CMAKE_CURRENT_SOURCE_DIR}/*.cpp)

# on fedora, binaries built with mpi must ended by _mpich suffix
# Add MODL_Coclustering executable
#
# On Fedora: binaries built with MPI must suffixed by _mpich (stored in ${MPI_SUFFIX})
add_executable(MODL_Coclustering${MPI_SUFFIX} ${cppfiles} MODL_Coclustering.rc)
set_khiops_options(MODL_Coclustering${MPI_SUFFIX})

target_link_libraries(MODL_Coclustering${MPI_SUFFIX} PUBLIC KMDRRuleLibrary KWLearningProblem)
set_khiops_options(MODL_Coclustering${MPI_SUFFIX})

if(MSVC)
target_link_options(MODL_Coclustering${MPI_SUFFIX} PUBLIC "/INCREMENTAL")
endif()

# Add MODL_Coclustering_SO shared library
add_library(MODL_Coclustering_SO SHARED ${cppfiles})
target_link_libraries(MODL_Coclustering_SO PUBLIC KMDRRuleLibrary KWLearningProblem)
set_target_properties(
Expand Down

0 comments on commit 8de3c74

Please sign in to comment.