Update to metall 0.28

CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.22)
-project(metall-ffi VERSION 0.2.2)
+project(metall-ffi VERSION 0.2.4)
 
 include(cmake/boilerplate_init.cmake)
 boilerplate_init()

conanfile.py

@@ -16,15 +16,22 @@ class Recipe(ConanFile):
 
     # Binary configuration
     settings = "os", "compiler", "build_type", "arch"
-    options = {"shared": [True, False], "fPIC": [True, False], "with_test_deps": [True, False]}
-    default_options = {"shared": False, "fPIC": True, "with_test_deps": False}
+    options = {
+        "shared": [True, False],
+        "fPIC": [True, False],
+        "with_test_deps": [True, False],
+    }
+    default_options = {
+        "shared": False,
+        "fPIC": True,
+        "with_test_deps": False,
+    }
     exports = "LICENSE",
     exports_sources = "src/*", "CMakeLists.txt", "cmake/*"
     generators = "CMakeDeps", "CMakeToolchain"
 
     def requirements(self):
-        self.requires("metall/0.23.1", transitive_headers=True)
-        self.requires("boost/1.83.0", transitive_headers=True, force=True)
+        self.requires("metall/0.28", transitive_headers=True)
 
         if self.options.with_test_deps:
             self.requires("doctest/2.4.11")
@@ -64,5 +71,4 @@ def package_info(self):
         self.cpp_info.set_property("cmake_target_name", f"{self.name}::{self.name}")
         self.cpp_info.requires = [
             "metall::metall",
-            "boost::headers",
         ]

src/dice/ffi/metall.cpp

@@ -47,6 +47,10 @@ metall_manager *metall_create(char const *path) {
     return reinterpret_cast<metall_manager *>(manager);
 }
 
+bool metall_is_read_only(metall_manager const *manager) {
+    return reinterpret_cast<metall_manager_t const *>(manager)->read_only();
+}
+
 bool metall_snapshot(metall_manager *manager, char const *dst_path) {
     return reinterpret_cast<metall_manager_t *>(manager)->snapshot(dst_path);
 }

src/dice/ffi/metall.h

@@ -4,6 +4,8 @@
 #include <stdbool.h>
 #include <stddef.h>
 
+#include <metall/logger_interface.h>
+
 #ifdef __cplusplus
 extern "C" {
 #endif
@@ -35,6 +37,13 @@ metall_manager *metall_open_read_only(char const *path);
  */
 metall_manager *metall_create(char const *path);
 
+/**
+ * @brief Returns true if the metall manager was opened as read-only
+ * @param manager manager to check
+ * @return true if the given manager was openened as read-only
+ */
+bool metall_is_read_only(metall_manager const *manager);
+
 /**
  * @brief Creates a snapshot of the metall datastore of manager and places it at dst_path
  * @param manager manager to perform snapshot

src/dice/ffi/metall_internal.hpp

@@ -1,14 +1,15 @@
 #ifndef DICE_METALLFFI_METALLINTERNAL_HPP
 #define DICE_METALLFFI_METALLINTERNAL_HPP
 
+#define METALL_LOGGER_EXTERN_C 1
 #include <metall/metall.hpp>
 
 namespace dice::metall_ffi::internal {
     /**
      * @brief The metall manager type used internally.
      *      This object type is whats actually behind the opaque ::metall_manager * in the interface
      */
-    using metall_manager = metall::basic_manager<uint32_t, 1ULL << 28>;
+    using metall_manager = metall::basic_manager<metall::kernel::storage, metall::kernel::segment_storage, uint32_t, 1ULL << 28>;
 } // namespace
 
 #endif//DICE_METALLFFI_METALLINTERNAL_HPP

test_package/example.cpp

@@ -1,5 +1,9 @@
 #include <dice/ffi/metall.h>
 
+extern "C" void metall_log(metall_log_level, char const *, size_t, char const *) {
+	// noop
+}
+
 int main() {
     metall_open("/tmp/test");
 }

tests/tests_Sanity.cpp

@@ -6,6 +6,10 @@
 #include <string>
 #include <random>
 
+extern "C" void metall_log(metall_log_level lvl, char const *file, size_t line, char const *msg) {
+    std::cerr << lvl << " " << file << ":" << line << ": " << msg << std::endl;
+}
+
 TEST_SUITE("metall-ffi") {
     TEST_CASE("sanity check") {
         char const *obj_name = "obj";
@@ -19,7 +23,7 @@ TEST_SUITE("metall-ffi") {
             }
 
             {
-                auto *ptr = static_cast<size_t *>(metall_malloc(manager, obj_name, sizeof(size_t)));
+                auto *ptr = static_cast<size_t *>(metall_malloc(manager, obj_name, sizeof(size_t) * 2));
                 CHECK_NE(ptr, nullptr);
                 CHECK_EQ(reinterpret_cast<uintptr_t>(ptr) % alignof(size_t), 0);