Skip to content

Commit

Permalink
Fix couple of small issues on rebase
Browse files Browse the repository at this point in the history
  • Loading branch information
Jovasa committed Jan 3, 2024
1 parent 925f9cf commit 0ccff5f
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
9 changes: 9 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,14 @@ option(BUILD_SHARED_LIBS "Build using shared kvazaar library" ON)

option(BUILD_TESTS "Build tests" ON)


include(GNUInstallDirs) #Helps to define correct distro specific install directories

set(KVAZAAR_INSTALL_LIBDIR "${CMAKE_INSTALL_LIBDIR}" CACHE PATH "kvazaar library install path")
set(KVAZAAR_INSTALL_BINDIR "${CMAKE_INSTALL_BINDIR}" CACHE PATH "kvazaar binary install path")
set(KVAZAAR_INSTALL_INCLUDEDIR "${CMAKE_INSTALL_INCLUDEDIR}" CACHE PATH "kvazaar include install path")
set(KVAZAAR_INSTALL_MANDIR "${CMAKE_INSTALL_MANDIR}/man1" CACHE PATH "kvazaar manual page file install path")

# https://www.kitware.com/cmake-and-the-default-build-type/
# Set a default build type if none was specified
set(KVZ_DEFAULT_BUILD_TYPE "RelWithDebInfo")
Expand Down Expand Up @@ -156,6 +164,7 @@ if(MSVC)
target_include_directories(kvazaar PUBLIC src/threadwrapper/include)
set_property( SOURCE ${LIB_SOURCES_STRATEGIES_AVX2} APPEND PROPERTY COMPILE_FLAGS "/arch:AVX2" )
else()
set_target_properties(kvazaar-bin PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/src)
list(APPEND ALLOW_AVX2 "x86_64" "AMD64")
if(${CMAKE_SYSTEM_PROCESSOR} IN_LIST ALLOW_AVX2)
set_property( SOURCE ${LIB_SOURCES_STRATEGIES_AVX2} APPEND PROPERTY COMPILE_FLAGS "-mavx2 -mbmi -mpopcnt -mlzcnt -mbmi2" )
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -446,7 +446,7 @@ It is recommended to use Clang instead of GCC in MinGW environments. GCC also wo
to build Kvazaar using Clang.

### CMake
Depending on the platform, some additional tools are required for compiling Kvazzar with CMake.
Depending on the platform, some additional tools are required for compiling Kvazaar with CMake.
For Ubuntu, the required packages are `build-essential cmake`.


Expand Down
2 changes: 2 additions & 0 deletions src/version.h.in
Original file line number Diff line number Diff line change
Expand Up @@ -34,4 +34,6 @@
#ifndef KVZ_VERSION
#define KVZ_VERSION @PROJECT_VERSION@
#endif
#define KVZ_COMPILER_STRING "@KVZ_COMPILER_STRING@"
#define KVZ_COMPILE_DATE "@CMAKE_BUILD_DATE@"
#define VERSION_STRING QUOTE_EXPAND(KVZ_VERSION)

0 comments on commit 0ccff5f

Please sign in to comment.