mark the target as internal

Author: aignas

python/uv/private/lock.bzl

@@ -313,6 +313,7 @@ def lock(*, name, srcs, out, args = [], env = None, **kwargs):
     """
     update_target = "{}.update".format(name)
     locker_target = "{}.run".format(name)
+    template_target = "_{}.update_template".format(name)
 
     # Check if the output file already exists, if yes, first copy it to the
     # output file location in order to make `uv` not change the requirements if
@@ -353,16 +354,20 @@ def lock(*, name, srcs, out, args = [], env = None, **kwargs):
     # requirements file
     pkg = native.package_name()
     expand_template(
-        name = update_target + "_gen",
+        name = template_target,
         out = update_target + ".py",
         template = "//python/uv/private:lock_copier.py",
         substitutions = {
             "{{dst}}": "{}/{}".format(pkg, out),
             "{{src}}": "{}/{}.out".format(pkg, name),
             "{{update_target}}": "//{}/{}".format(pkg, update_target),
         },
+        tags = ["manual"],
     )
 
+    # TODO @aignas 2025-03-17: use a custom `py_binary` rule and
+    # `ctx.actions.expand_template` instead of the bazel skylib rule. This
+    # would make it possible to avoid the extra target.
     py_binary(
         name = update_target,
         srcs = [update_target + ".py"],

python/uv/private/lock.sh

@@ -1,7 +1,6 @@
 #!/bin/bash
 set -euo pipefail
 
-sleep 5
 out="{{src_out}}"
 if [[ -n "${BUILD_WORKSPACE_DIRECTORY:-}" ]]; then
     out="${BUILD_WORKSPACE_DIRECTORY}/$out"