Bugfix: call to rules_apko target from another repository was failing (…

…#39)
chainguard-dev · Nov 30, 2023 · d8f33ae · d8f33ae
1 parent 207ff7a
commit d8f33ae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/apko/private/BUILD.bazel b/apko/private/BUILD.bazel
@@ -14,7 +14,10 @@ bzl_library(
     name = "apko_image",
     srcs = ["apko_image.bzl"],
     visibility = ["//apko:__subpackages__"],
-    deps = ["apko_run"],
+    deps = [
+        "apko_run",
+        "@bazel_skylib//:lib",
+    ],
 )
 
 bzl_library(

diff --git a/apko/private/apko_image.bzl b/apko/private/apko_image.bzl
@@ -1,6 +1,7 @@
 "A rule for running apko with prepopulated cache"
 
 load("//apko/private:apko_run.bzl", "apko_run")
+load("@bazel_skylib//lib:paths.bzl", "paths")
 
 _ATTRS = {
     "contents": attr.label(doc = "Label to the contents repository generated by translate_lock. See [apko-cache](./apko-cache.md) documentation.", mandatory = True),
@@ -31,7 +32,7 @@ def _impl(ctx):
 
     args.add_all(ctx.attr.args)
 
-    args.add("--cache-dir={}/{}/{}".format(ctx.bin_dir.path, ctx.label.package, cache_name))
+    args.add("--cache-dir={}".format(paths.join(ctx.bin_dir.path, ctx.label.workspace_root, ctx.label.package, cache_name)))
     args.add("--offline")
 
     if ctx.attr.architecture: