[CI/Build] fix setuptools-scm usage (vllm-project#8771)

opendatahub-io · Sep 27, 2024 · ebdf009 · ebdf009
1 parent dae5712
commit ebdf009
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 10 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,5 +1,5 @@
-# vllm commit id, generated by setup.py
-vllm/commit_id.py
+# version file generated by setuptools-scm
+/vllm/_version.py
 
 # vllm-flash-attn built from source
 vllm/vllm_flash_attn/
@@ -196,8 +196,5 @@ _build/
 *_hip*
 hip_compat.h
 
-# version file generated by setuptools-scm
-/vllm/_version.py
-
 # Benchmark dataset
 benchmarks/*.json
diff --git a/pyproject.toml b/pyproject.toml
@@ -51,9 +51,6 @@ ignore = [
     "UP032",
 ]
 
-[tool.setuptools_scm]
-version_file = "vllm/_version.py"
-
 [tool.mypy]
 python_version = "3.8"
 

diff --git a/setup.py b/setup.py
@@ -354,12 +354,15 @@ def get_path(*filepath) -> str:
 
 
 def get_vllm_version() -> str:
-    version = get_version()
+    version = get_version(
+        write_to="vllm/_version.py",  # TODO: move this to pyproject.toml
+    )
+
     sep = "+" if "+" not in version else "."  # dev versions might contain +
 
     if _no_device():
         if envs.VLLM_TARGET_DEVICE == "empty":
-            version += "+empty"
+            version += f"{sep}empty"
     elif _is_cuda():
         cuda_version = str(get_nvcc_cuda_version())
         if cuda_version != MAIN_CUDA_VERSION: