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 2, 2023
1 parent 92fabda commit 05397ae
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions src/Learning/MODL_Coclustering/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,15 +1,11 @@
file(GLOB cppfiles ${CMAKE_CURRENT_SOURCE_DIR}/*.cpp)

# on fedora, binaries built with mpi must ended by _mpich suffix
# MODL_Coclustering target 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()

# MODL_SO target (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 05397ae

Please sign in to comment.