Remove redundancy

Hind-M · Hind-M · commit ffe5718878e7 · 2025-09-18T15:05:24.000+02:00
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -101,7 +101,7 @@ set(SPARROW_IPC_HEADERS
     ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/arrow_interface/arrow_schema.hpp
     ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/arrow_interface/arrow_schema/private_data.hpp
     ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/config/config.hpp
-    ${SPARROW_IPC_INCLUDE_DIR}/config/sparrow_ipc_version.hpp
+    ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/config/sparrow_ipc_version.hpp
     ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/deserialize_variable_size_binary_array.hpp
     ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/deserialize_fixedsizebinary_array.hpp
     ${SPARROW_IPC_INCLUDE_DIR}/sparrow_ipc/deserialize_primitive_array.hpp
diff --git a/cmake/sparrow-ipcConfig.cmake.in b/cmake/sparrow-ipcConfig.cmake.in
@@ -3,7 +3,7 @@
 #
 #   sparrow-ipc_FOUND - true if sparrow-ipc found on the system
 #   sparrow-ipc_INCLUDE_DIRS - the directory containing sparrow-ipc headers
-#   sparrow-ipc_LIBRARY - empty
+#   sparrow-ipc_LIBRARY - the location of the sparrow-ipc library (or empty if not applicable)
 
 @PACKAGE_INIT@
 
diff --git a/conanfile.py b/conanfile.py
@@ -18,7 +18,7 @@ class SparrowIPCRecipe(ConanFile):
     topics = ("arrow", "apache arrow", "columnar format", "dataframe", "ipc", "serialization", "deserialization", "flatbuffers")
     package_type = "library"
     settings = "os", "arch", "compiler", "build_type"
-    exports_sources = "include/*", "src/*", "cmake/*", "docs/*", "CMakeLists.txt", "sparrow-ipcConfig.cmake.in", "LICENSE"
+    exports_sources = "include/*", "src/*", "cmake/*", "docs/*", "CMakeLists.txt", "LICENSE"
     options = {
         "shared": [True, False],
         "fPIC": [True, False],
diff --git a/include/config/sparrow_ipc_version.hpp b/include/config/sparrow_ipc_version.hpp
diff --git a/include/sparrow_ipc/config/sparrow_ipc_version.hpp b/include/sparrow_ipc/config/sparrow_ipc_version.hpp
@@ -9,4 +9,6 @@ namespace sparrow_ipc
     constexpr int SPARROW_IPC_BINARY_CURRENT = 1;
     constexpr int SPARROW_IPC_BINARY_REVISION = 0;
     constexpr int SPARROW_IPC_BINARY_AGE = 0;
+
+    static_assert(SPARROW_IPC_BINARY_AGE <= SPARROW_IPC_BINARY_CURRENT, "SPARROW_IPC_BINARY_AGE cannot be greater than SPARROW_IPC_BINARY_CURRENT");
 }
diff --git a/sparrow-ipcConfig.cmake.in b/sparrow-ipcConfig.cmake.in
diff --git a/tests/main.cpp b/tests/main.cpp
@@ -4,7 +4,7 @@
 
 #include "doctest/doctest.h"
 
-#include "config/sparrow_ipc_version.hpp"
+#include "sparrow_ipc/config/sparrow_ipc_version.hpp"
 
 TEST_CASE("versions exist and are readable")
 {

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`#`
`4`	`4`	`# sparrow-ipc_FOUND - true if sparrow-ipc found on the system`
`5`	`5`	`# sparrow-ipc_INCLUDE_DIRS - the directory containing sparrow-ipc headers`
`6`		`-# sparrow-ipc_LIBRARY - empty`
	`6`	`+# sparrow-ipc_LIBRARY - the location of the sparrow-ipc library (or empty if not applicable)`
`7`	`7`
`8`	`8`	`@PACKAGE_INIT@`
`9`	`9`
Original file line number	Diff line number	Diff line change
`@@ -9,4 +9,6 @@ namespace sparrow_ipc`
`9`	`9`	`constexpr int SPARROW_IPC_BINARY_CURRENT = 1;`
`10`	`10`	`constexpr int SPARROW_IPC_BINARY_REVISION = 0;`
`11`	`11`	`constexpr int SPARROW_IPC_BINARY_AGE = 0;`
	`12`	`+`
	`13`	`+ static_assert(SPARROW_IPC_BINARY_AGE <= SPARROW_IPC_BINARY_CURRENT, "SPARROW_IPC_BINARY_AGE cannot be greater than SPARROW_IPC_BINARY_CURRENT");`
`12`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`#include "doctest/doctest.h"`
`6`	`6`
`7`		`-#include "config/sparrow_ipc_version.hpp"`
	`7`	`+#include "sparrow_ipc/config/sparrow_ipc_version.hpp"`
`8`	`8`
`9`	`9`	`TEST_CASE("versions exist and are readable")`
`10`	`10`	`{`