Switch Python bindings to pybind11.

Drop MVD2 support from Python, and keep the interface more in line with
Python conventions.

.gitignore

@@ -0,0 +1,5 @@
+.eggs
+.pytest_cache
+*.egg-info
+__pycache__
+tags

.gitmodules

@@ -0,0 +1,3 @@
+[submodule "python/pybind11"]
+ path = python/pybind11
+ url = [email protected]:pybind/pybind11.git

CMakeLists.txt

@@ -3,9 +3,9 @@ cmake_minimum_required(VERSION 3.0)
 
 include(CTest)
 
-set(MVDTOOL_VERSION_MAJOR 1)
-set(MVDTOOL_VERSION_MINOR 5)
-set(MVDTOOL_VERSION_PATCH 2)
+set(MVDTOOL_VERSION_MAJOR 2)
+set(MVDTOOL_VERSION_MINOR 0)
+set(MVDTOOL_VERSION_PATCH 0)
 
 list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/CMake
  ${PROJECT_SOURCE_DIR}/CMake/portability
@@ -32,10 +32,9 @@ FILE(GLOB MVDTOOL_HEADERS "include/mvd/*.hpp")
 FILE(GLOB MVDTOOLE_BITS_HEADERS "include/mvd/*/*.hpp")
 set(MVDTOOL_INCLUDE_DIRS "${CMAKE_CURRENT_SOURCE_DIR}/include")
 
-add_subdirectory(src)
-
 if(BUILD_PYTHON_BINDINGS)
  add_subdirectory(python)
 endif()
 
+add_subdirectory(src)
 add_subdirectory(tests/unit)

python/CMakeLists.txt

@@ -1,90 +1,5 @@
-##############################################
-# Requirements
-find_package(PythonComplete QUIET)
-find_package(Cython REQUIRED)
-
-
-
-
-
-##############################################
-# Definitions
-set(cymod_src "${CMAKE_CURRENT_SOURCE_DIR}/mvdtool.pyx")
-set(default_cybindings_cpp "${CMAKE_CURRENT_SOURCE_DIR}/mvdtool.cpp")
-set(cybindings_cpp "${CMAKE_CURRENT_BINARY_DIR}/mvdtool.cpp")
-set(local_include_dir "${CMAKE_CURRENT_SOURCE_DIR}/include")
-set(cy_sources
- mvdtool.pyx
- include/boost.pxd
- include/MVD2.pxd
- include/MVD3.pxd
- include/MVD_MVDType.pxd
- include/MVD.pxd
- include/std.pxd)
-
-##############################################
-# Rules
-
-# .pyx -> .cpp
-add_custom_command(
- OUTPUT ${cybindings_cpp}
- COMMAND ${CYTHON_EXECUTABLE} ${CYTHON_FLAGS} --fast-fail --cplus -I${local_include_dir} --output-file ${cybindings_cpp} ${cymod_src}
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- # No dependencies, except own sources
- DEPENDS ${cy_sources}
- COMMENT "Cythonizing ${cymod_src}"
-)
-
-# .cpp -> shared_lib
-add_definitions("-std=c++11" "-Wfatal-errors" "-Wno-strict-aliasing")
-python_add_module(mvdtool ${cybindings_cpp})
-target_include_directories(mvdtool PRIVATE ${PYTHON_INCLUDE_DIRS} ${local_include_dir})
-target_link_libraries(mvdtool MVDTool)
-
-
-##############################################
-# Python packaging
-
-get_target_property(_include_dirs mvd-tool INCLUDE_DIRECTORIES)
-set(_inc_str ${local_include_dir})
-foreach(dir ${_include_dirs})
- set(_inc_str "${_inc_str}:${dir}")
-endforeach()
-
-# Wheel
-add_custom_target(pydist
- COMMAND python setup.py build --build-base=${CMAKE_CURRENT_BINARY_DIR} build_ext "-I${_inc_str}" bdist_wheel --dist-dir ${CMAKE_BINARY_DIR}
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS ${cybindings_cpp}
- COMMENT "Building distributable Python wheel in ${CMAKE_BINARY_DIR}"
-)
-
-# Source dist - still requires include dirs
-add_custom_target(pysrcdist
- COMMAND python setup.py build --build-base=${CMAKE_CURRENT_BINARY_DIR} sdist --dist-dir ${CMAKE_BINARY_DIR}
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS ${cybindings_cpp}
- COMMENT "Building distributable Python wheel in ${CMAKE_BINARY_DIR}"
-)
-
-
-##############################################
-# Install
-#
-install(TARGETS mvdtool
- LIBRARY DESTINATION ${CMAKE_INSTALL_FULL_PYTHON_PACKAGES} )
-
-
-##############################################
-# TESTS
-#
-add_test(NAME python_read_mvd2
- COMMAND bash -c "PYTHONPATH=${CMAKE_CURRENT_BINARY_DIR} python test/read_mvd2.py"
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
-)
-
-add_test(NAME python_read_mvd3
- COMMAND bash -c "PYTHONPATH=${CMAKE_CURRENT_BINARY_DIR} python test/read_mvd3.py"
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
-)
+add_subdirectory(pybind11)
 
+pybind11_add_module(mvdtool mvdtool.cpp)
+target_link_libraries(mvdtool PUBLIC MVDTool HighFive)
+install(TARGETS mvdtool DESTINATION lib/python)