Give a default version if the version is dynamic in setup.cfg

news/1101.bugfix.md

@@ -0,0 +1 @@
+Give a default version if the version is dynamic in `setup.cfg`.

pdm/models/setup.py

@@ -92,7 +92,7 @@ def read_setup_py(cls, file: Path) -> Setup:
 
         return Setup(
             name=cls._find_single_string(setup_call, body, "name"),
-            version=cls._find_single_string(setup_call, body, "version"),
+            version=cls._find_single_string(setup_call, body, "version") or "0.0.0",
             install_requires=cls._find_install_requires(setup_call, body),
             extras_require=cls._find_extras_require(setup_call, body),
             python_requires=cls._find_single_string(
@@ -107,12 +107,14 @@ def read_setup_cfg(file: Path) -> Setup:
         parser.read(str(file))
 
         name = None
-        version = None
+        version = "0.0.0"
         if parser.has_option("metadata", "name"):
             name = parser.get("metadata", "name")
 
         if parser.has_option("metadata", "version"):
-            version = parser.get("metadata", "version")
+            meta_version = parser.get("metadata", "version")
+            if not meta_version.startswith("attr:"):
+                version = meta_version
 
         install_requires = []
         extras_require: Dict[str, List[str]] = {}

tests/models/test_setup_parsing.py

@@ -0,0 +1,114 @@
+import pytest
+
+from pdm.models.setup import Setup
+
+
+@pytest.mark.parametrize(
+    "content, result",
+    [
+        (
+            """[metadata]
+name = foo
+version = 0.1.0
+""",
+            Setup("foo", "0.1.0"),
+        ),
+        (
+            """[metadata]
+name = foo
+version = attr:foo.__version__
+""",
+            Setup("foo", "0.0.0"),
+        ),
+        (
+            """[metadata]
+name = foo
+version = 0.1.0
+
+[options]
+python_requires = >=3.6
+install_requires =
+    click
+    requests
+[options.extras_require]
+tui =
+    rich
+""",
+            Setup("foo", "0.1.0", ["click", "requests"], {"tui": ["rich"]}, ">=3.6"),
+        ),
+    ],
+)
+def test_parse_setup_cfg(content, result, tmp_path):
+    tmp_path.joinpath("setup.cfg").write_text(content)
+    assert Setup.from_directory(tmp_path) == result
+
+
+@pytest.mark.parametrize(
+    "content,result",
+    [
+        (
+            """from setuptools import setup
+
+setup(name="foo", version="0.1.0")
+""",
+            Setup("foo", "0.1.0"),
+        ),
+        (
+            """import setuptools
+
+setuptools.setup(name="foo", version="0.1.0")
+""",
+            Setup("foo", "0.1.0"),
+        ),
+        (
+            """from setuptools import setup
+
+kwargs = {"name": "foo", "version": "0.1.0"}
+setup(**kwargs)
+""",
+            Setup("foo", "0.1.0"),
+        ),
+        (
+            """from setuptools import setup
+name = 'foo'
+setup(name=name, version="0.1.0")
+""",
+            Setup("foo", "0.1.0"),
+        ),
+        (
+            """from setuptools import setup
+
+setup(name="foo", version="0.1.0", install_requires=['click', 'requests'],
+      python_requires='>=3.6', extras_require={'tui': ['rich']})
+""",
+            Setup("foo", "0.1.0", ["click", "requests"], {"tui": ["rich"]}, ">=3.6"),
+        ),
+        (
+            """from setuptools import setup
+
+version = open('__version__.py').read().strip()
+
+setup(name="foo", version=version)
+""",
+            Setup("foo", "0.0.0"),
+        ),
+    ],
+)
+def test_parse_setup_py(content, result, tmp_path):
+    tmp_path.joinpath("setup.py").write_text(content)
+    assert Setup.from_directory(tmp_path) == result
+
+
+def test_parse_pyproject_toml(tmp_path):
+    content = """[project]
+name = "foo"
+version = "0.1.0"
+requires-python = ">=3.6"
+dependencies = ["click", "requests"]
+
+[project.optional-dependencies]
+tui = ["rich"]
+"""
+    tmp_path.joinpath("pyproject.toml").write_text(content)
+    result = Setup("foo", "0.1.0", ["click", "requests"], {"tui": ["rich"]}, ">=3.6")
+    assert Setup.from_directory(tmp_path) == result