python-discovery 1.3.1__tar.gz → 1.3.2__tar.gz

{python_discovery-1.3.1 → python_discovery-1.3.2}/.pre-commit-config.yaml

@@ -19,7 +19,7 @@ repos:
     hooks:
       - id: pyproject-fmt
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.15.12"
+    rev: "v0.15.14"
     hooks:
       - id: ruff-format
       - id: ruff
@@ -32,7 +32,7 @@ repos:
           - prettier@3.8.1
           - "@prettier/plugin-xml@3.4.2"
   - repo: https://github.com/zizmorcore/zizmor-pre-commit
-    rev: v1.24.1
+    rev: v1.25.2
     hooks:
       - id: zizmor
   - repo: meta

{python_discovery-1.3.1 → python_discovery-1.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-discovery
-Version: 1.3.1
+Version: 1.3.2
 Summary: Python interpreter discovery
 Project-URL: Changelog, https://github.com/tox-dev/python-discovery/releases
 Project-URL: Documentation, https://python-discovery.readthedocs.io

{python_discovery-1.3.1 → python_discovery-1.3.2}/docs/changelog.rst

@@ -6,6 +6,13 @@
 
 .. towncrier release notes start
 
+********************
+ v1.3.2 (2026-05-27)
+********************
+
+No significant changes.
+
+
 ********************
  v1.3.1 (2026-05-12)
 ********************

{python_discovery-1.3.1 → python_discovery-1.3.2}/src/python_discovery/_discovery.py

@@ -314,13 +314,7 @@ class LazyPathDump:
             content += " with =>"
             for file_path in self.path.iterdir():
                 try:
-                    if file_path.is_dir():
-                        continue
-                    if IS_WIN:  # pragma: win32 cover
-                        pathext = self.env.get("PATHEXT", ".COM;.EXE;.BAT;.CMD").split(";")
-                        if not any(file_path.name.upper().endswith(ext) for ext in pathext):
-                            continue
-                    elif not (file_path.stat().st_mode & os.X_OK):
+                    if not self._is_executable(file_path):
                         continue
                 except OSError:
                     pass
@@ -328,6 +322,14 @@ class LazyPathDump:
                 content += file_path.name
         return content
 
+    def _is_executable(self, file_path: Path) -> bool:
+        if file_path.is_dir():
+            return False
+        if IS_WIN:  # pragma: win32 cover
+            pathext = self.env.get("PATHEXT", ".COM;.EXE;.BAT;.CMD").split(";")
+            return any(file_path.name.upper().endswith(ext) for ext in pathext)
+        return bool(file_path.stat().st_mode & os.X_OK)
+
 
 def path_exe_finder(
     spec: PythonSpec, *, all_implementations: bool = False

{python_discovery-1.3.1 → python_discovery-1.3.2}/src/python_discovery/_py_info.py

@@ -16,6 +16,7 @@ from string import digits
 from typing import TYPE_CHECKING, ClassVar, Final, NamedTuple
 
 if TYPE_CHECKING:
+    import tkinter as tk
     from collections.abc import Generator, Mapping
 
     from ._cache import PyInfoCache
@@ -165,42 +166,37 @@ class PythonInfo:  # noqa: PLR0904
             try:
                 tcl = tk.Tcl()
                 tcl_lib = tcl.eval("info library")
-
-                # Try to get TK library path directly first
-                try:
-                    tk_lib = tcl.eval("set tk_library")
-                    if tk_lib and os.path.isdir(tk_lib):
-                        pass  # We found it directly
-                    else:
-                        tk_lib = None  # Reset if invalid
-                except tk.TclError:
-                    tk_lib = None
-
-                # If direct query failed, try constructing the path
-                if tk_lib is None:
-                    tk_version = tcl.eval("package require Tk")
-                    tcl_parent = os.path.dirname(tcl_lib)
-
-                    # Try different version formats
-                    version_variants = [
-                        tk_version,  # Full version like "8.6.12"
-                        ".".join(tk_version.split(".")[:2]),  # Major.minor like "8.6"
-                        tk_version.split(".")[0],  # Just major like "8"
-                    ]
-
-                    for version in version_variants:
-                        tk_lib_path = os.path.join(tcl_parent, f"tk{version}")
-                        if not os.path.isdir(tk_lib_path):
-                            continue
-                        if os.path.exists(os.path.join(tk_lib_path, "tk.tcl")):
-                            tk_lib = tk_lib_path
-                            break
-
+                tk_lib = PythonInfo._resolve_tk_lib(tcl, tcl_lib)
             except tk.TclError:
                 pass
 
         return tcl_lib, tk_lib
 
+    @staticmethod
+    def _query_tk_library(tcl: tk.Tk) -> str | None:  # pragma: no cover
+        """Try to get the TK library path directly from Tcl."""
+        import tkinter as tk  # noqa: PLC0415
+
+        try:
+            if (tk_lib := tcl.eval("set tk_library")) and os.path.isdir(tk_lib):
+                return tk_lib
+        except tk.TclError:
+            pass
+        return None
+
+    @staticmethod
+    def _resolve_tk_lib(tcl: tk.Tk, tcl_lib: str) -> str | None:  # pragma: no cover
+        """Resolve the TK library path by direct query or path construction."""
+        if (tk_lib := PythonInfo._query_tk_library(tcl)) is not None:
+            return tk_lib
+        tk_version = tcl.eval("package require Tk")
+        tcl_parent = os.path.dirname(tcl_lib)
+        for version in (tk_version, ".".join(tk_version.split(".")[:2]), tk_version.split(".")[0]):
+            tk_lib_path = os.path.join(tcl_parent, f"tk{version}")
+            if os.path.isdir(tk_lib_path) and os.path.exists(os.path.join(tk_lib_path, "tk.tcl")):
+                return tk_lib_path
+        return None
+
     def _fast_get_system_executable(self) -> str | None:
         """Try to get the system executable by just looking at properties."""
         # if we're not in a virtual environment, this is already a system python, so return the original executable

{python_discovery-1.3.1 → python_discovery-1.3.2}/src/python_discovery/_py_spec.py

@@ -17,7 +17,7 @@ PATTERN = re.compile(
     (?P<version>[0-9.]+)?           # version (e.g. 3.12, 3.12.1)
     (?P<threaded>t)?                # free-threaded flag
     (?:-(?P<arch>32|64))?           # architecture bitness
-    (?:-(?P<machine>[a-zA-Z0-9_]+))?  # ISA (e.g. arm64, x86_64)
+    (?:-(?P<machine>[a-zA-Z0-9_.]+))?  # ISA (e.g. arm64, x86_64, i86pc.64bit)
     $
     """,
     re.VERBOSE,

{python_discovery-1.3.1 → python_discovery-1.3.2}/src/python_discovery/_windows/_pep514.py

@@ -124,23 +124,23 @@ def load_exe(hive_name: str, company: str, company_key: Any, tag: str) -> tuple[
     key_path = f"{hive_name}/{company}/{tag}"
     try:
         with winreg.OpenKeyEx(company_key, rf"{tag}\InstallPath") as ip_key, ip_key:  # ty: ignore[unresolved-attribute]
-            exe = get_value(ip_key, "ExecutablePath")
-            if exe is None:
-                ip = get_value(ip_key, None)
-                if ip is None:
-                    msg(key_path, "no ExecutablePath or default for it")
-
-                else:
-                    exe = os.path.join(ip, "python.exe")
-            if exe is not None and os.path.exists(exe):
-                args = get_value(ip_key, "ExecutableArguments")
-                return exe, args
+            if (exe := _resolve_exe(ip_key, key_path)) is not None and os.path.exists(exe):
+                return exe, get_value(ip_key, "ExecutableArguments")
             msg(key_path, f"could not load exe with value {exe}")
     except OSError:
         msg(f"{key_path}/InstallPath", "missing")
     return None
 
 
+def _resolve_exe(ip_key: Any, key_path: str) -> str | None:  # noqa: ANN401
+    if (exe := get_value(ip_key, "ExecutablePath")) is not None:
+        return exe
+    if (ip := get_value(ip_key, None)) is None:
+        msg(key_path, "no ExecutablePath or default for it")
+        return None
+    return os.path.join(ip, "python.exe")
+
+
 def load_arch_data(hive_name: str, company: str, tag: str, tag_key: Any, default_arch: int) -> int | None:  # noqa: ANN401
     arch_str = get_value(tag_key, "SysArchitecture")
     if arch_str is not None:

{python_discovery-1.3.1 → python_discovery-1.3.2}/tasks/release.py

@@ -19,29 +19,13 @@ def main(version_str: str) -> None:
         msg = "Current repository is dirty. Please commit any changes and try again."
         raise RuntimeError(msg)
     upstream, release_branch = create_release_branch(repo, version)
+    original_main_sha = upstream.refs.main.commit.hexsha
     main_pushed = False
     tag_pushed = False
     release_created = False
-    original_main_sha = upstream.refs.main.commit.hexsha
     try:
-        release_commit = release_changelog(repo, version)
-        tag = tag_release_commit(release_commit, repo, version)
-        print("push release commit")  # noqa: T201
-        repo.git.push(upstream.name, f"{release_branch}:main", "-f")
-        main_pushed = True
-        print("push release tag")  # noqa: T201
-        repo.git.push(upstream.name, tag, "-f")
-        tag_pushed = True
-        create_github_release(version)
-        release_created = True
-        print("checkout main to new release and delete release branch")  # noqa: T201
-        repo.heads.main.checkout()
-        repo.delete_head(release_branch, force=True)
-        print("delete remote release branch")  # noqa: T201
-        repo.git.push(upstream.name, f":{release_branch}", "--no-verify")
-        upstream.fetch()
-        repo.git.reset("--hard", f"{upstream.name}/main")
-        print("All done!")  # noqa: T201
+        main_pushed, tag_pushed, release_created = push_release(repo, upstream, release_branch, version)
+        finalize_release(repo, upstream, release_branch)
     except Exception:
         cleanup_failed_release(
             repo,
@@ -76,6 +60,17 @@ def get_upstream(repo: Repo) -> Remote:
     raise RuntimeError(msg)
 
 
+def push_release(repo: Repo, upstream: Remote, release_branch: Head, version: Version) -> tuple[bool, bool, bool]:
+    release_commit = release_changelog(repo, version)
+    tag = tag_release_commit(release_commit, repo, version)
+    print("push release commit")  # noqa: T201
+    repo.git.push(upstream.name, f"{release_branch}:main", "-f")
+    print("push release tag")  # noqa: T201
+    repo.git.push(upstream.name, tag, "-f")
+    create_github_release(version)
+    return True, True, True
+
+
 def release_changelog(repo: Repo, version: Version) -> Commit:
     print("generate release commit")  # noqa: T201
     check_call(["towncrier", "build", "--yes", "--version", version.public], cwd=str(ROOT_SRC_DIR))  # noqa: S607
@@ -121,6 +116,17 @@ def create_github_release(version: Version) -> None:
         raise
 
 
+def finalize_release(repo: Repo, upstream: Remote, release_branch: Head) -> None:
+    print("checkout main to new release and delete release branch")  # noqa: T201
+    repo.heads.main.checkout()
+    repo.delete_head(release_branch, force=True)
+    print("delete remote release branch")  # noqa: T201
+    repo.git.push(upstream.name, f":{release_branch}", "--no-verify")
+    upstream.fetch()
+    repo.git.reset("--hard", f"{upstream.name}/main")
+    print("All done!")  # noqa: T201
+
+
 def cleanup_failed_release(  # noqa: PLR0913
     repo: Repo,
     upstream: Remote,

{python_discovery-1.3.1 → python_discovery-1.3.2}/tests/test_py_spec.py

@@ -180,6 +180,7 @@ def test_specifier_satisfies_with_partial_information() -> None:
         pytest.param("cpython3.12-64", None, id="no-machine"),
         pytest.param("cpython3.12", None, id="no-arch-no-machine"),
         pytest.param("python3.12-64-arm64", "arm64", id="python-impl"),
+        pytest.param("cpython3.14-64-i86pc.64bit", "i86pc.64bit", id="dotted-machine"),
     ],
 )
 def test_spec_parse_machine(spec_str: str, expected_machine: str | None) -> None: