cook-build 0.7.1__py3-none-any.whl → 0.7.3__py3-none-any.whl

cook/__main__.py

@@ -342,12 +342,10 @@ def __main__(cli_args: list[str] | None = None) -> None:
                 ]
             )
             if args.module:
-                # Temporarily add the current working directory to the path.
-                try:
-                    sys.path.append(os.getcwd())
-                    importlib.import_module(args.module)
-                finally:
-                    sys.path.pop()
+                # Add the current working directory to the path so local modules can be
+                # imported.
+                sys.path.append(os.getcwd())
+                importlib.import_module(args.module)
             elif args.recipe.is_file():
                 # Parse the recipe.
                 spec = importlib.util.spec_from_file_location("recipe", args.recipe)

cook/actions.py

@@ -36,6 +36,7 @@ executor with a deprecation warning.
 """
 
 import asyncio
+import functools
 import hashlib
 import logging
 import os
@@ -94,7 +95,9 @@ class FunctionAction(Action):
         else:
             # Run sync function in executor
             loop = asyncio.get_running_loop()
-            await loop.run_in_executor(None, self.func, task, *self.args, **self.kwargs)
+            await loop.run_in_executor(
+                None, functools.partial(self.func, task, *self.args, **self.kwargs)
+            )
 
 
 class SubprocessAction(Action):
@@ -119,21 +122,22 @@ class SubprocessAction(Action):
             True
     """
 
-    def __init__(self, *args, **kwargs) -> None:
+    def __init__(self, args: str | list[str], **kwargs) -> None:
         # Validate shell argument early
-        if kwargs.get("shell", False) and args and not isinstance(args[0], str):
+        if kwargs.get("shell", False) and not isinstance(args, str):
             raise ValueError("shell=True requires string args")
         self.args = args
         self.kwargs = kwargs
 
     async def execute(self, task: "Task") -> None:
         # Get the command arguments
-        (args,) = self.args
+        args = self.args
         shell = self.kwargs.get("shell", False)
         other_kwargs = {k: v for k, v in self.kwargs.items() if k != "shell"}
 
         # Create the subprocess
         if shell:
+            assert isinstance(args, str)
             process = await asyncio.create_subprocess_shell(args, **other_kwargs)
         else:
             # Exec mode: args can be a string (single command) or list
@@ -164,7 +168,7 @@ class SubprocessAction(Action):
     @property
     def hexdigest(self) -> str:
         hasher = hashlib.sha1()
-        (args,) = self.args
+        args = self.args
         if isinstance(args, str):
             hasher.update(args.encode())
         else:
@@ -173,7 +177,7 @@ class SubprocessAction(Action):
         return hasher.hexdigest()
 
     def __repr__(self) -> str:
-        args, *_ = self.args
+        args = self.args
         if not isinstance(args, str):
             args = " ".join(map(shlex.quote, args))
         return f"{self.__class__.__name__}({repr(args)})"
@@ -196,13 +200,13 @@ class CompositeAction(Action):
 
     @property
     def hexdigest(self) -> str | None:
-        parts = []
+        hasher = hashlib.sha1()
         for action in self.actions:
             hexdigest = action.hexdigest
             if hexdigest is None:
                 return None
-            parts.append(hexdigest)
-        return "".join(parts)
+            hasher.update(bytearray.fromhex(hexdigest))
+        return hasher.hexdigest()
 
 
 class ModuleAction(SubprocessAction):

cook/contexts.py

@@ -152,7 +152,7 @@ class create_target_directories(Context):
                 create = self.manager.create_task(
                     name,
                     action=actions.FunctionAction(
-                        lambda _: target.parent.mkdir(parents=True, exist_ok=True)
+                        lambda _, p=target.parent: p.mkdir(parents=True, exist_ok=True)
                     ),
                 )
             task.task_dependencies.append(create)

cook/controller.py

@@ -39,11 +39,6 @@ QUERIES = {
         "last_digested" TIMESTAMP NOT NULL
     );
     """,
-    "select_task": """
-        SELECT "digest"
-        FROM "files"
-        WHERE "name" = :name
-    """,
     "upsert_task_completed": """
         INSERT INTO "tasks" ("name", "digest", "last_completed")
         VALUES (:name, :digest, :last_completed)
@@ -80,7 +75,6 @@ class Controller:
     def __init__(self, dependencies: nx.DiGraph, connection: Connection) -> None:
         self.dependencies = dependencies
         self.connection = connection
-        self._digest_cache: dict[Path, tuple[float, bytes]] = {}
 
     def resolve_stale_tasks(self, tasks: list["Task"] | None = None) -> set["Task"]:
         self.is_stale(tasks or list(self.dependencies))
@@ -88,7 +82,7 @@ class Controller:
             node for node, data in self.dependencies.nodes(True) if data.get("is_stale")
         }
 
-    def _evaluate_task_hexdigest(self, task: "Task") -> str | None:
+    def _evaluate_task_hexdigest(self, task: "Task") -> str:
         """
         Evaluate the digest of a task by combining the digest of all its dependencies.
         """
@@ -102,8 +96,7 @@ class Controller:
             )
             dependency = Path(dependency).resolve()
             if not dependency.is_file():
-                LOGGER.debug("dependency %s of %s is missing", dependency, task)
-                return None
+                raise FileNotFoundError(f"dependency {dependency} of {task} is missing")
             dependencies.append(dependency)
 
         hasher = hashlib.sha1()
@@ -198,9 +191,11 @@ class Controller:
             return True
 
         # If one of the dependencies is missing, the task is stale.
-        current_digest = self._evaluate_task_hexdigest(task)
-        if current_digest is None:
+        try:
+            current_digest = self._evaluate_task_hexdigest(task)
+        except FileNotFoundError:
             LOGGER.debug("%s is stale because one of its dependencies is missing", task)
+            return True
 
         # If the digest has changed, the task is stale.
         (cached_digest,) = cached_digest
@@ -290,8 +285,10 @@ class Controller:
             dep_futures = [task_futures[dep] for dep in dep_tasks]
             await asyncio.gather(*dep_futures)
 
-        start = datetime.now()
-        digest = self._evaluate_task_hexdigest(task)
+        digest: str | None = None
+        if not dry_run:
+            digest = self._evaluate_task_hexdigest(task)
+        start: datetime | None = None
 
         try:
             # Log what we're doing
@@ -307,22 +304,19 @@ class Controller:
                         "  action: %s",
                         task.action,
                     )
-            else:
-                LOGGER.log(
-                    logging.DEBUG if task.name.startswith("_") else logging.INFO,
-                    "executing %s ...",
-                    task,
-                )
-
-            # Update DB for start
-            if not dry_run:
-                params = {"name": task.name, "last_started": start}
-                self.connection.execute(QUERIES["upsert_task_started"], params)
-                self.connection.commit()
 
             # Execute the task
             if not dry_run:
                 async with semaphore:
+                    start = datetime.now()
+                    LOGGER.log(
+                        logging.DEBUG if task.name.startswith("_") else logging.INFO,
+                        "executing %s ...",
+                        task,
+                    )
+                    params = {"name": task.name, "last_started": start}
+                    self.connection.execute(QUERIES["upsert_task_started"], params)
+                    self.connection.commit()
                     await task.execute()
 
                 # Check that all targets were created
@@ -335,6 +329,7 @@ class Controller:
 
             # Update DB for completion
             if not dry_run:
+                assert digest is not None
                 params = {
                     "name": task.name,
                     "digest": digest,
@@ -344,6 +339,7 @@ class Controller:
                 self.connection.commit()
 
                 # Log completion
+                assert start is not None
                 delta = util.format_timedelta(datetime.now() - start)
                 LOGGER.log(
                     logging.DEBUG if task.name.startswith("_") else logging.INFO,
@@ -362,7 +358,7 @@ class Controller:
                 self.connection.execute(QUERIES["upsert_task_failed"], params)
                 self.connection.commit()
 
-            delta = util.format_timedelta(datetime.now() - start)
+            delta = util.format_timedelta(datetime.now() - start) if start else "?"
             LOGGER.exception("failed to execute %s after %s", task, delta)
             raise util.FailedTaskError(ex, task=task) from ex
 

cook/manager.py

@@ -63,7 +63,7 @@ class Manager:
                     raise ValueError(f"{context} did not return a task")
             self.tasks[task.name] = task
             return task
-        except:
+        except Exception:
             filename, lineno = util.get_location()
             LOGGER.exception(
                 "failed to create task with name '%s' at %s:%d", name, filename, lineno
@@ -150,7 +150,7 @@ def create_task(
 
     Args:
         name: Name of the new task. Defaults to the string representation of the first
-            dependency if not provided.
+            target if not provided.
         action: Action to execute or a string for shell commands.
         targets: Paths for files to be generated.
         dependencies: Paths to files on which this task depends.

cook/util.py

@@ -32,6 +32,7 @@ def evaluate_hexdigest(path: PathOrStr, size=2**16, hasher: str = "sha1") -> str
 class Timer:
     def __init__(self):
         self.start = None
+        self.end = None
 
     def __enter__(self) -> Timer:
         self.start = time()

{cook_build-0.7.1.dist-info → cook_build-0.7.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cook-build
-Version: 0.7.1
+Version: 0.7.3
 Summary: A task-centric build system with simple declarative recipes specified in Python
 Author: Till Hoffmann
 License: BSD-3-Clause

cook_build-0.7.3.dist-info/RECORD

@@ -0,0 +1,14 @@
+cook/__init__.py,sha256=SCa9_i6B84IzSAwq0wnSQqvycyL4dvTO7dRIysJXZj4,179
+cook/__main__.py,sha256=3vnDZOJLTMJGRQzPolqurVzKJ7KCUjosBuuUkW22tKY,13192
+cook/actions.py,sha256=yxo_LQ42iejq0K92OnYs7x2GaTuyNK3ZmqnRikQUgWs,7537
+cook/contexts.py,sha256=-L4o_b_XPNZ_MBuYAgR1LYbptWPankDyj-00Pji7EVg,10710
+cook/controller.py,sha256=vs9wBfvdQzE5RGZxZtKef6hb8rUOKKMp_kuPnmVAf9s,13432
+cook/manager.py,sha256=sLlqPjsi81UTSHNhNj-ZTDilCdVmEF992N94yxWPOlU,6247
+cook/task.py,sha256=-LNMwHdFlTUG45DF_QyWlIHw55_n_u2Xf5beKB7LrdY,2308
+cook/util.py,sha256=Ssk_79cB7tvxEk_I9VAjjzmEp1bJ95ELZHCAx6sybyo,2459
+cook_build-0.7.3.dist-info/licenses/LICENSE,sha256=3Nuj_WTTcz7JDg4-9EzNf6vHlKRWpdLUccg-pvoZ3WE,1500
+cook_build-0.7.3.dist-info/METADATA,sha256=M83L-Uar5tKKLB2OyqQ1IS3wxe-idBZIvH2foLTXxAE,4586
+cook_build-0.7.3.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+cook_build-0.7.3.dist-info/entry_points.txt,sha256=5UP0ZmmxSNKevTVISUJxmdXEQsKrI4n54OQYkjrdX2c,48
+cook_build-0.7.3.dist-info/top_level.txt,sha256=ewNQIn2oRSYV98vAsUnw88u2Q8XHKhAz70ed2PEdR2c,5
+cook_build-0.7.3.dist-info/RECORD,,

{cook_build-0.7.1.dist-info → cook_build-0.7.3.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: setuptools (80.10.1)
+Generator: setuptools (80.10.2)
 Root-Is-Purelib: true
 Tag: py3-none-any
 

cook_build-0.7.1.dist-info/RECORD

@@ -1,14 +0,0 @@
-cook/__init__.py,sha256=SCa9_i6B84IzSAwq0wnSQqvycyL4dvTO7dRIysJXZj4,179
-cook/__main__.py,sha256=4sO22TsNTt3oirT71dJjZJwmynsRyUGT-CHJNlDgCkk,13242
-cook/actions.py,sha256=1VLyWL-pACuWpAjIyNWPBeu6wlKkoqi6t3lHr6hV0EQ,7399
-cook/contexts.py,sha256=AMKO7Uz-nI52OsdPQ_zCLXjOf77V4pgzvDyj6-N8Ods,10705
-cook/controller.py,sha256=vpF3QhiM3HImaI0rHJ58T8i5AQi5_P4Z2p42qFvG1po,13426
-cook/manager.py,sha256=Y-QGVw9x8ZpSBMRALJIHgcMmIZulAV9KxwtBJz35Gpw,6241
-cook/task.py,sha256=-LNMwHdFlTUG45DF_QyWlIHw55_n_u2Xf5beKB7LrdY,2308
-cook/util.py,sha256=15MMG07CYZZ-YdFE_2jzRRTaqHMsw83UFg0s7e72MhI,2435
-cook_build-0.7.1.dist-info/licenses/LICENSE,sha256=3Nuj_WTTcz7JDg4-9EzNf6vHlKRWpdLUccg-pvoZ3WE,1500
-cook_build-0.7.1.dist-info/METADATA,sha256=8CK_rxir23FBjEo5H3IhxB1tkzXlMrolpAuhwqYLP4g,4586
-cook_build-0.7.1.dist-info/WHEEL,sha256=qELbo2s1Yzl39ZmrAibXA2jjPLUYfnVhUNTlyF1rq0Y,92
-cook_build-0.7.1.dist-info/entry_points.txt,sha256=5UP0ZmmxSNKevTVISUJxmdXEQsKrI4n54OQYkjrdX2c,48
-cook_build-0.7.1.dist-info/top_level.txt,sha256=ewNQIn2oRSYV98vAsUnw88u2Q8XHKhAz70ed2PEdR2c,5
-cook_build-0.7.1.dist-info/RECORD,,