driverclient 0.2.30__tar.gz → 0.2.31__tar.gz

{driverclient-0.2.30 → driverclient-0.2.31}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: driverclient
-Version: 0.2.30
+Version: 0.2.31
 Summary: Driver Server client with an embeddable connector for config injection at runtime.
 Project-URL: Homepage, https://example.com/driverclient
 Author-email: Raja Sanaullah <sanaullah@99technologies.com>

{driverclient-0.2.30 → driverclient-0.2.31}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "driverclient"
-version = "0.2.30"
+version = "0.2.31"
 description = "Driver Server client with an embeddable connector for config injection at runtime."
 readme = "README.md"
 requires-python = ">=3.10"

{driverclient-0.2.30 → driverclient-0.2.31}/src/driverclient/__init__.py

@@ -26,7 +26,7 @@ from driverclient.main import run as _run
 
 __all__ = ["ClientEvent", "DriverClient"]
 
-__version__ = "0.2.30"
+__version__ = "0.2.31"
 
 
 class DriverClient:

{driverclient-0.2.30 → driverclient-0.2.31}/src/driverclient/ops/download.py

@@ -12,7 +12,6 @@ Factored out here so the two modes share one download implementation:
   • download_drivers() — fan the above out across a driver list in parallel
 """
 
-import shutil
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from pathlib import Path
 
@@ -21,21 +20,7 @@ from driverclient.core.http import http_download_retry
 
 def download_driver(driver: dict, base: Path, repo_url: str, cfg: dict) -> tuple[Path, bool]:
     """Download all files for one driver into base/<binding_id>/.
-    Returns (dest_dir, success). success is False if ANY file failed.
-
-    On failure, dest is removed before returning. The live install path stages
-    every driver under one shared `base` and installs it with a single batched
-    `pnputil /add-driver base\\*.inf /subdirs` — that command globs the
-    filesystem directly, so a driver this function reports as failed (and the
-    caller correctly excludes from the install list) would otherwise still
-    leave its already-downloaded files (commonly just the small .inf, if a
-    large companion .sys/.dll timed out) sitting under base/<binding_id>/ for
-    pnputil to find anyway. pnputil then tries to add that incomplete package —
-    "cannot find the file specified" for its missing payload — which large
-    graphics drivers hit most often (biggest files, likeliest to time out
-    partway), and can poison the whole batched pass for the OTHER, fully
-    downloaded drivers riding in the same pnputil call.
-    """
+    Returns (dest_dir, success). success is False if ANY file failed."""
     bid = driver.get("binding_id") or driver.get("package_id", "")
     pkg_id = driver["package_id"]
     dest = base / bid
@@ -51,9 +36,6 @@ def download_driver(driver: dict, base: Path, repo_url: str, cfg: dict) -> tuple
         if not http_download_retry(url, path, f["sha256"], timeout=cfg["timeout_long"]):
             ok = False
 
-    if not ok:
-        shutil.rmtree(dest, ignore_errors=True)
-
     return dest, ok