[Bf-blender-cvs] [8b6b880356b] quadriflow: Updated build files based on feedback
Sebastian Parborg
noreply at git.blender.org
Tue Sep 10 18:36:10 CEST 2019
Commit: 8b6b880356b0cc4aca26389b61bf9003a4ad5494
Author: Sebastian Parborg
Date: Wed Sep 4 17:31:49 2019 +0200
Branches: quadriflow
https://developer.blender.org/rB8b6b880356b0cc4aca26389b61bf9003a4ad5494
Updated build files based on feedback
===================================================================
M extern/quadriflow/CMakeLists.txt
D extern/quadriflow/blender_config.cmake
M intern/quadriflow/CMakeLists.txt
M source/blender/editors/object/object_remesh.c
===================================================================
diff --git a/extern/quadriflow/CMakeLists.txt b/extern/quadriflow/CMakeLists.txt
index 01de5191a20..6960dc378f5 100644
--- a/extern/quadriflow/CMakeLists.txt
+++ b/extern/quadriflow/CMakeLists.txt
@@ -1,14 +1,10 @@
-cmake_minimum_required(VERSION 3.1)
-project(QuadriFlow LANGUAGES CXX)
-
-if(DEFINED QUADRIFLOW_CMAKE_CFG)
- include(${QUADRIFLOW_CMAKE_CFG})
-endif()
-
-set(CMAKE_INCLUDE_CURRENT_DIR ON)
set(CMAKE_CXX_STANDARD 14)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
+if (WIN32)
+ add_definitions(-D_USE_MATH_DEFINES)
+endif()
+
set(LEMON_3RD_PATH 3rd/lemon-1.3.1)
set(LEMON_SOURCE_ROOT_DIR ${CMAKE_CURRENT_SOURCE_DIR}/${LEMON_3RD_PATH})
@@ -30,49 +26,19 @@ set(LEMON_SOURCES
${LEMON_SRC}/bits/windows.cc
)
-include_directories(src)
-include_directories(3rd/pcg32)
-include_directories(3rd/pss)
-include_directories(${Boost_INCLUDE_DIRS})
-include_directories(${EIGEN_INCLUDE_DIRS})
-include_directories(${GLM_INCLUDE_DIRS})
-include_directories(${GLUT_INCLUDE_DIRS})
-include_directories(${LEMON_INCLUDE_DIRS})
-include_directories(${TBB_INCLUDE_DIRS})
-include_directories(${GUROBI_INCLUDE_DIRS})
-
-if (BUILD_PERFORMANCE_TEST)
- add_definitions(-DPERFORMANCE_TEST)
-endif(BUILD_PERFORMANCE_TEST)
-
-if (BUILD_OPENMP)
- add_definitions(-DWITH_OMP)
-endif(BUILD_OPENMP)
-
-if (BUILD_LOG)
- add_definitions(-DLOG_OUTPUT)
-endif(BUILD_LOG)
-
-if (BUILD_GUROBI)
- add_definitions(-DWITH_GUROBI)
-endif(BUILD_GUROBI)
-
-if (BUILD_TBB)
- add_definitions(-DWITH_TBB)
-endif(BUILD_TBB)
-
-if (BUILD_FREE_LICENSE)
- add_definitions(-DEIGEN_MPL2_ONLY)
-endif(BUILD_FREE_LICENSE)
+set(INC
+ src
+ 3rd/pcg32
+ 3rd/pss
+ ${BOOST_INCLUDE_DIR}
+ ${EIGEN3_INCLUDE_DIRS}
+ ${LEMON_INCLUDE_DIRS}
+)
-if(BUILD_STATIC_LIB)
- set(LIB_TYPE STATIC)
-else()
- set(LIB_TYPE SHARED)
-endif()
+set(INC_SYS
+)
-set(
- quadriflow_SRC
+set(SRC
src/adjacent-matrix.cpp
src/adjacent-matrix.hpp
src/compare-key.hpp
@@ -103,15 +69,10 @@ set(
src/serialize.hpp
src/subdivide.cpp
src/subdivide.hpp
+ ${LEMON_SOURCES}
)
-add_library( qflow ${LIB_TYPE}
- ${quadriflow_SRC}
- ${LEMON_SRC}
-)
-target_link_libraries(
- qflow
- ${TBB_LIBRARIES}
- ${LEMON_LIBRARIES}
- ${GUROBI_LIBRARIES}
+set(LIB
)
+
+blender_add_lib(extern_quadriflow "${SRC}" "${INC}" "${INC_SYS}" "${LIB}")
diff --git a/extern/quadriflow/blender_config.cmake b/extern/quadriflow/blender_config.cmake
deleted file mode 100644
index 19500d67774..00000000000
--- a/extern/quadriflow/blender_config.cmake
+++ /dev/null
@@ -1,14 +0,0 @@
-# Do NOT use CACHE behavior here, as this has the same effect as defining an option,
-# it exposes the setting in main CMake config (which we do not want here).
-
-set(QUADRIFLOW_STANDALONE FALSE)
-set(BUILD_PERFORMANCE_TEST FALSE)
-set(BUILD_LOG TRUE)
-set(BUILD_GUROBI FALSE)
-set(BUILD_OPENMP ${WITH_OPENMP})
-set(BUILD_TBB FALSE)
-set(BUILD_FREE_LICENSE FALSE) #If TRUE, remove LGPL code
-set(BUILD_STATIC_LIB TRUE)
-
-set(Boost_INCLUDE_DIRS ${Boost_INCLUDE_DIR})
-set(EIGEN_INCLUDE_DIRS ${EIGEN3_INCLUDE_DIRS})
diff --git a/intern/quadriflow/CMakeLists.txt b/intern/quadriflow/CMakeLists.txt
index 7fad274551c..01b0eb15f36 100644
--- a/intern/quadriflow/CMakeLists.txt
+++ b/intern/quadriflow/CMakeLists.txt
@@ -25,7 +25,7 @@ set(INC
set(INC_SYS
../../extern/quadriflow/src
- ${Boost_INCLUDE_DIR}
+ ${BOOST_INCLUDE_DIR}
${EIGEN3_INCLUDE_DIRS}
)
@@ -35,7 +35,11 @@ set(SRC
)
set(LIB
- qflow
+ extern_quadriflow
)
+if (WIN32)
+ add_definitions(-D_USE_MATH_DEFINES)
+endif()
+
blender_add_lib(bf_intern_quadriflow "${SRC}" "${INC}" "${INC_SYS}" "${LIB}")
diff --git a/source/blender/editors/object/object_remesh.c b/source/blender/editors/object/object_remesh.c
index f647ecda2d5..29d249647f5 100644
--- a/source/blender/editors/object/object_remesh.c
+++ b/source/blender/editors/object/object_remesh.c
@@ -69,8 +69,6 @@
#include "object_intern.h" // own include
-#define WITH_SHARP
-
static bool object_remesh_poll(bContext *C)
{
Object *ob = CTX_data_active_object(C);
@@ -181,14 +179,10 @@ static int quadriflow_remesh_exec(bContext *C, wmOperator *op)
}
const int target_faces = RNA_int_get(op->ptr, "target_faces");
const int seed = RNA_int_get(op->ptr, "seed");
-#ifdef WITH_SHARP
+
const bool use_preserve_sharp = RNA_boolean_get(op->ptr, "use_preserve_sharp");
const bool use_preserve_border = RNA_boolean_get(op->ptr, "use_preserve_border");
-#else
- /* The preserve sharp feature is currently unstable and will crash in certain scenarios */
- const bool use_preserve_sharp = false;
- const bool use_preserve_border = false;
-#endif
+
const bool use_adaptive_scale = RNA_boolean_get(op->ptr, "use_adaptive_scale");
const bool preserve_paint_mask = RNA_boolean_get(op->ptr, "preserve_paint_mask");
@@ -219,7 +213,7 @@ static int quadriflow_remesh_exec(bContext *C, wmOperator *op)
}
if (smooth_normals) {
- BKE_mesh_smooth_flag_set(ob, true);
+ BKE_mesh_smooth_flag_set(ob->data, true);
}
if (ob->mode == OB_MODE_SCULPT) {
@@ -251,7 +245,6 @@ void OBJECT_OT_quadriflow_remesh(wmOperatorType *ot)
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
/* properties */
-#ifdef WITH_SHARP
RNA_def_boolean(ot->srna,
"use_preserve_sharp",
false,
@@ -263,7 +256,7 @@ void OBJECT_OT_quadriflow_remesh(wmOperatorType *ot)
false,
"Preserve mesh borders",
"Try to preserve mesh borders on the mesh");
-#endif
+
RNA_def_boolean(ot->srna,
"use_adaptive_scale",
false,
More information about the Bf-blender-cvs
mailing list