Fix packaging version (#276)

* Fix packaging version

* Also print tiledb-vector-search version in CI/CD

* Remove .git from sdist

* Rework config file regeneration

Author: dudoslav

.github/workflows/ci-python.yml

@@ -40,3 +40,6 @@ jobs:
         env:
           TILEDB_REST_TOKEN: ${{ secrets.TILEDB_CLOUD_HELPER_VAR }}
         shell: bash -el {0}
+      - name: Check tiledb-vector-search version
+        run: |
+          python -c "from tiledb.vector_search.version import version; print(version)"

pyproject.toml

@@ -41,14 +41,19 @@ wheel.expand-macos-universal-tags = true
 metadata.version.provider = "scikit_build_core.metadata.setuptools_scm"
 cmake.source-dir = "src"
 cmake.targets = ["skbuild-export"]
-# TODO: This should be removed, but currently can't bacause: 'src/src/CMakeLists.txt'
 sdist.include = [
-    ".git"
+    "apis/python/src/tiledb/vector_search/version.py",
+    "src/config.h"
+]
+sdist.exclude = [
+    ".github"
 ]
 wheel.packages = ["apis/python/src/tiledb"]
+# We need to run cmake during packaging step in order to generate src/config.h and
+# thus not including .git to generate it later
+sdist.cmake = true
 
 [tool.scikit-build.cmake.define]
-#GIT_REPO_NAME = "placeholder"
 TILEDB_VS_PYTHON = "ON"
 
 [tool.setuptools_scm]

src/CMakeLists.txt

@@ -175,7 +175,9 @@ include(nlohmann_json)
 include (logging)
 
 # Write config file
-configure_file(${CMAKE_SOURCE_DIR}/include/config.h.in config.h)
+if(LOGGING_INFO_QUERIED)
+    configure_file(${CMAKE_SOURCE_DIR}/include/config.h.in ${CMAKE_SOURCE_DIR}/config.h)
+endif()
 
 # -----------------------------------------------------------------------------
 # Interface libraries

src/cmake/logging.cmake

@@ -1,4 +1,4 @@
-
+if(EXISTS ${CMAKE_SOURCE_DIR}/../.git)
 
 # Get the current date and time
 string(TIMESTAMP CURRENT_DATETIME "%Y-%m-%d %H:%M:%S")
@@ -58,3 +58,9 @@ execute_process(
 )
 
 get_filename_component(IVF_HACK_CXX_COMPILER ${CMAKE_CXX_COMPILER} NAME)
+
+set(LOGGING_INFO_QUERIED True)
+
+elseif(NOT EXISTS ${CMAKE_SOURCE_DIR}/config.h)
+    message(FATAL_ERROR ".git or pre generated config.h is required")
+endif()