move oryx:: namespace

enhanced tests

added proper install

updated readme

added automatic test pipeline

Author: Philemon Benner

.clangd

@@ -0,0 +1,37 @@
+CompileFlags:
+  CompilationDatabase: build
+  Add:
+    - -Wno-unknown-warning-option
+    - -std=gnu++23
+    - -Iinclude
+  Remove: [-m*, -f*]
+  Compiler: /usr/bin/gcc-14
+
+Index:
+  Background: Build
+  StandardLibrary: true
+
+Hover:
+  ShowAKA: false
+
+Completion:
+  AllScopes: Yes
+  
+InlayHints:
+  BlockEnd: false
+  Designators: false
+  Enabled: true
+  ParameterNames: false
+  DeducedTypes: false
+  TypeNameLimit: 24
+
+Diagnostics:
+  UnusedIncludes: Strict
+  ClangTidy:
+    FastCheckFilter: Strict
+    Add: 
+      - modernize-*
+      - performance-*
+      - bugprone-use-after-move
+    Remove:
+      - modernize-avoid-c-arrays

.github/workflows/linux.yaml

@@ -0,0 +1,64 @@
+name: linux
+
+on: [ push, pull_request ]
+
+jobs:
+  linux:
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - compiler: llvm
+            compiler-version: 18
+            cxx: 23
+          - compiler: llvm
+            compiler-version: 19
+            cxx: 23
+          - compiler: gcc
+            compiler-version: 11
+            cxx: 20
+          - compiler: gcc
+            compiler-version: 12
+            cxx: 20
+          - compiler: gcc
+            compiler-version: 13
+            cxx: 20 
+          - compiler: gcc
+            compiler-version: 14
+            cxx: 23
+    name: "${{ github.job }} (C++${{ matrix.cxx }}-${{ matrix.compiler }}-${{ matrix.compiler-version }})"
+    runs-on: ubuntu-24.04
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          submodules: recursive
+          fetch-depth: 0
+      - name: Export GitHub Actions cache environment variables
+        uses: actions/github-script@v7
+        with:
+          script: |
+            core.exportVariable('ACTIONS_CACHE_URL', process.env.ACTIONS_CACHE_URL || '');
+            core.exportVariable('ACTIONS_RUNTIME_TOKEN', process.env.ACTIONS_RUNTIME_TOKEN || '');
+      - name: Setup ccache
+        uses: hendrikmuhs/ccache-action@v1
+        with:
+          key: "${{ github.job }}-${{ matrix.compiler }}-${{ matrix.compiler-version }}"
+          max-size: "2G"
+      - name: Compile
+        run: |
+          if [[ "${{ matrix.compiler }}" == "llvm" ]]; then
+            export CC=clang-${{ matrix.compiler-version }}
+            export CXX=clang++-${{ matrix.compiler-version }}
+          elif [[ "${{ matrix.compiler }}" == "gcc" ]]; then
+            export CC=gcc-${{ matrix.compiler-version }}
+            export CXX=g++-${{ matrix.compiler-version }}
+          fi
+          sudo ln -s $(which ccache) /usr/local/bin/$CC
+          sudo ln -s $(which ccache) /usr/local/bin/$CXX
+          $CXX --version
+          cmake -B build -G Ninja -DCMAKE_CXX_STANDARD=${{ matrix.cxx }} -DORYX_CRT_BUILD_TESTS=ON -DCMAKE_BUILD_TYPE=Release
+          cmake --build build
+      - name: Run tests
+        run: |
+          ./build/kvdb-cpp_tests --success

.vscode/c_cpp_properties.json

@@ -1,7 +1,7 @@
 {
     "configurations": [
         {
-            "name": "KeyValueDatabase",
+            "name": "Linux",
             "cppStandard": "c++20",
             "compilerPath": "/usr/bin/g++",
             "intelliSenseMode": "${default}",

.vscode/settings.json

@@ -1,73 +1,8 @@
 {
-    "files.associations": {
-        "typeinfo": "cpp",
-        "text_encoding": "cpp",
-        "optional": "cpp",
-        "array": "cpp",
-        "atomic": "cpp",
-        "bit": "cpp",
-        "*.tcc": "cpp",
-        "bitset": "cpp",
-        "cctype": "cpp",
-        "charconv": "cpp",
-        "chrono": "cpp",
-        "cinttypes": "cpp",
-        "clocale": "cpp",
-        "cmath": "cpp",
-        "compare": "cpp",
-        "concepts": "cpp",
-        "condition_variable": "cpp",
-        "csignal": "cpp",
-        "cstdarg": "cpp",
-        "cstddef": "cpp",
-        "cstdint": "cpp",
-        "cstdio": "cpp",
-        "cstdlib": "cpp",
-        "cstring": "cpp",
-        "ctime": "cpp",
-        "cwchar": "cpp",
-        "cwctype": "cpp",
-        "deque": "cpp",
-        "map": "cpp",
-        "set": "cpp",
-        "string": "cpp",
-        "unordered_map": "cpp",
-        "unordered_set": "cpp",
-        "vector": "cpp",
-        "exception": "cpp",
-        "algorithm": "cpp",
-        "functional": "cpp",
-        "iterator": "cpp",
-        "memory": "cpp",
-        "memory_resource": "cpp",
-        "numeric": "cpp",
-        "random": "cpp",
-        "ratio": "cpp",
-        "regex": "cpp",
-        "string_view": "cpp",
-        "system_error": "cpp",
-        "tuple": "cpp",
-        "type_traits": "cpp",
-        "utility": "cpp",
-        "fstream": "cpp",
-        "initializer_list": "cpp",
-        "iomanip": "cpp",
-        "iosfwd": "cpp",
-        "iostream": "cpp",
-        "istream": "cpp",
-        "limits": "cpp",
-        "mutex": "cpp",
-        "new": "cpp",
-        "numbers": "cpp",
-        "ostream": "cpp",
-        "semaphore": "cpp",
-        "span": "cpp",
-        "sstream": "cpp",
-        "stdexcept": "cpp",
-        "stop_token": "cpp",
-        "streambuf": "cpp",
-        "thread": "cpp",
-        "variant": "cpp",
-        "format": "cpp"
-    }
+    "editor.formatOnSave": true,
+    "cmake.configureOnOpen": false,
+    "cmake.configureOnEdit": false,
+    "clangd.path": "/usr/bin/clangd",
+    "C_Cpp.clang_format_path": "/usr/bin/clang-format-18",
+    "files.associations": {},
 }

CMakeLists.txt

@@ -1,58 +1,115 @@
-cmake_minimum_required(VERSION 3.11...3.31)
+cmake_minimum_required(VERSION 3.24)
 
-project(kvdbcpp VERSION 0.0.0 LANGUAGES CXX)
+project(kvdb-cpp VERSION 0.1.0 LANGUAGES CXX)
 
-option(KVDB_ENABLE_TESTS "Build Tests" ON)
-option(KVDB_BUILD_DEPS "Build Dependencies from source" ON)
+option(ORYX_KVDB_ENABLE_TESTS "Build Tests" ON)
+option(ORYX_KVDB_BUILD_DEPS "Build Dependencies from source" ON)
 
-include(FetchContent)
-include(GNUInstallDirs)
+set(CMAKE_CXX_STANDARD_REQUIRED ON)
+if(NOT DEFINED CMAKE_CXX_STANDARD)
+    set(CMAKE_CXX_STANDARD 20)
+endif()
 
+include(FetchContent)
 
-if(KVDB_BUILD_DEPS)
+if(ORYX_KVDB_BUILD_DEPS)
     message(STATUS "Building deps from source")
 
     FetchContent_Declare(
         reflectcpp
         GIT_REPOSITORY https://github.com/getml/reflect-cpp.git
-        GIT_TAG v0.15.0
+        GIT_TAG v0.19.0
         OVERRIDE_FIND_PACKAGE
-        EXCLUDE_FROM_ALL
     )
 
     FetchContent_Declare(
         leveldb
         GIT_REPOSITORY https://github.com/google/leveldb.git
-        GIT_TAG 1.23
+        GIT_TAG ac691084fdc5546421a55b25e7653d450e5a25fb
         OVERRIDE_FIND_PACKAGE
-        EXCLUDE_FROM_ALL
     )
 
+    set(LEVELDB_BUILD_BENCHMARKS OFF)
+    set(LEVELDB_BUILD_TESTS OFF)
     FetchContent_MakeAvailable(reflectcpp leveldb)
+    
+    add_library(reflectcpp::reflectcpp ALIAS reflectcpp)
+    add_library(leveldb::leveldb ALIAS leveldb)
+    
+    install(TARGETS reflectcpp EXPORT ${PROJECT_NAME}-exports)
+    install(TARGETS leveldb EXPORT ${PROJECT_NAME}-exports)
+else()
+    find_package(reflectcpp CONFIG REQUIRED)
+    find_package(leveldb CONFIG REQUIRED)
 endif()
 
-find_package(reflectcpp)
-find_package(leveldb)
-
 add_library(${PROJECT_NAME} INTERFACE)
 add_library(${PROJECT_NAME}::${PROJECT_NAME} ALIAS ${PROJECT_NAME})
 
 target_link_libraries(${PROJECT_NAME}
     INTERFACE 
-        reflectcpp
-        leveldb
+        reflectcpp::reflectcpp
+        leveldb::leveldb
 )
 
-if(KVDB_ENABLE_TESTS)
-    set(test_exe ${PROJECT_NAME}_test)
-    add_executable(${test_exe} test/main.cpp test/read_write.cpp)
+if(ORYX_KVDB_ENABLE_TESTS)
+    set(test_exe ${PROJECT_NAME}_tests)
+    add_executable(${test_exe} 
+        tests/main.cpp 
+        tests/read_write.cpp
+    )
     target_link_libraries(${test_exe} PRIVATE ${PROJECT_NAME})
 endif()
 
-
-target_include_directories(${PROJECT_NAME}
-    INTERFACE
-        "$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>"
-        "$<INSTALL_INTERFACE:include>" 
+target_include_directories(${PROJECT_NAME} INTERFACE
+    $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
+    $<INSTALL_INTERFACE:include>
 )
 
+target_compile_options(${PROJECT_NAME} INTERFACE $<$<CONFIG:Debug>:-Wall -Wextra>)
+
+if(PROJECT_IS_TOP_LEVEL)
+    include(GNUInstallDirs)
+    include(CMakePackageConfigHelpers)
+
+    configure_package_config_file(
+        cmake/${PROJECT_NAME}-config.cmake.in
+        ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config.cmake
+        INSTALL_DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}
+    )
+
+    write_basic_package_version_file(
+        ${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config-version.cmake
+        VERSION ${PROJECT_VERSION}
+        COMPATIBILITY SameMajorVersion
+    )
+
+    install(
+        FILES 
+            "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config.cmake"
+            "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config-version.cmake"
+        DESTINATION "${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}"
+    )
+
+    file(GLOB_RECURSE KVDB_CPP_HEADERS RELATIVE ${CMAKE_CURRENT_LIST_DIR} "${CMAKE_CURRENT_LIST_DIR}/include/*")
+
+    target_sources(${PROJECT_NAME}
+        PUBLIC
+        FILE_SET kvdb_cpp_headers
+        TYPE HEADERS
+        BASE_DIRS $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include> $<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/include>
+        FILES ${KVDB_CPP_HEADERS}
+    )
+
+    install(
+        TARGETS ${PROJECT_NAME}
+        EXPORT ${PROJECT_NAME}-exports
+        FILE_SET kvdb_cpp_headers DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}
+    )
+
+    install(
+        EXPORT ${PROJECT_NAME}-exports
+        DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}
+        NAMESPACE oryx::
+    )
+endif()

README.md

@@ -18,12 +18,22 @@ kvdb-cpp supports the following types:
 
 - `std::string`
 - `bool`
-- `int`
-- `double`
-- `structs / classes`
+- `integral types`
+- `floating types`
+- `any that reflect cpp can serialize beyond that`
 
-## Roadmap
+## Build locally
 
-- Examples
-- Add cmake support
-- Docs
+```bash
+cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=1 -DCMAKE_BUILD_TYPE=Debug -Bbuild -H.
+      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+      Only needed for clangd   
+```
+
+```bash
+cmake --build build -j32
+```
+
+## Todo
+
+- Fix install for reflect-cpp

cmake/kvdb-cpp-config.cmake.in

@@ -0,0 +1,8 @@
+@PACKAGE_INIT@
+
+include(${CMAKE_CURRENT_LIST_DIR}/kvdb-cpp-exports.cmake)
+
+find_dependency(reflectcpp)
+find_dependency(leveldb)
+
+check_required_components(kvdb-cpp)