Change versioning policy to avoid .dev versions.

It is no longer possible to write half-open version constraints
with packages that publish `.dev` versions. Since that was a misuse
of `.dev` versions anyway, stop doing that.

See also pypa/setuptools#3807.

Author: whitequark

README.rst

@@ -7,6 +7,17 @@ The Amaranth HDL Yosys wheels provide a specialized `WebAssembly <https://webass
 
 Although this package is platform-independent, it depends on wasmtime-py wheels, which are currently available only for x86_64 Windows, Linux, and macOS. This is expected to improve in the future.
 
+Versioning
+==========
+
+The version of this package (which is derived from the upstream Yosys package version) is comprised of five parts in a ``X.Y.Z.N.postM`` format:
+
+  1. ``X``: Yosys major version
+  2. ``Y``: Yosys minor version
+  3. ``Z``: Yosys patch version; only present for some Yosys releases, zero if not present
+  4. ``N``: Yosys node version; only present for unreleased Yosys snapshots (where it matches the ``N`` in the ``X.Y+N`` upstream version), zero for releases
+  5. ``postM``: package version, monotonically incrementing from the initial commit
+
 Building
 ========
 

setup.py

@@ -1,23 +1,25 @@
+import re
 import subprocess
-from setuptools import setup, find_packages
+from setuptools import setup
 from setuptools_scm.git import parse as parse_git
 
 
 def version():
-    yosys_makefile_ver = subprocess.check_output([
+    yosys_version_raw = subprocess.check_output([
         "make", "-s", "-C", "yosys-src", "echo-yosys-ver"
-    ], encoding="utf-8").strip().split("+")
-    if len(yosys_makefile_ver) == 1:
-        yosys_version = yosys_makefile_ver[0]
-    elif len(yosys_makefile_ver) == 2:
-        yosys_version = f"{yosys_makefile_ver[0]}.post{yosys_makefile_ver[1]}"
-    else:
-        assert False
+    ], encoding="utf-8").strip()
 
-    package_git = parse_git(".")
-    package_version = package_git.format_with(".dev{distance}")
+    # Yosys can't figure out if it should have a patch version or not.
+    # Match one, and add one below in our version just in case.
+    yosys_version = re.match(r"^(\d+)\.(\d+)(?:\.(\d+))?(?:\+(\d+))?$", yosys_version_raw)
 
-    return yosys_version + package_version
+    yosys_major  = yosys_version[1]
+    yosys_minor  = yosys_version[2]
+    yosys_patch  = yosys_version[3] or "0"
+    yosys_node   = yosys_version[4] or "0"
+    git_distance = parse_git(".").format_with("post{distance}")
+
+    return ".".join([yosys_major, yosys_minor, yosys_patch, yosys_node, git_distance])
 
 
 setup(