From a21a3a6074de9e59bde6d5f10bd3c77697deb13e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=98ystein=20Hovind?= <oystein@ascenium.com>
Date: Fri, 25 Oct 2024 11:48:28 +0200
Subject: [PATCH] openroad.bzl: Hack away prefix

Hack to make `$(location <label>)` expansions
work when configuration files are generated for
executables that are run with `bazelisk run`.
Hacks a `File`'s `.path` into a `.short_path`.
---
 openroad.bzl | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/openroad.bzl b/openroad.bzl
index 50469f9..df4f9f8 100644
--- a/openroad.bzl
+++ b/openroad.bzl
@@ -517,6 +517,9 @@ def _config_content(arguments, paths):
     return ("".join(sorted(["export {}?={}\n".format(*pair) for pair in arguments.items()]) +
                     ["include {}\n".format(path) for path in paths]))
 
+def _hack_away_prefix(arguments, prefix):
+    return {k: v.removeprefix(prefix + "/") for k, v in arguments.items()}
+
 def _data_arguments(ctx):
     return {k: ctx.expand_location(v, ctx.attr.data) for k, v in ctx.attr.arguments.items()}
 
@@ -732,7 +735,10 @@ def _yosys_impl(ctx):
     ctx.actions.write(
         output = config_short,
         content = _config_content(
-            arguments = _data_arguments(ctx) | _required_arguments(ctx) | _orfs_arguments(short = True, *[dep[OrfsInfo] for dep in ctx.attr.deps]) | _verilog_arguments(ctx.files.verilog_files, short = True),
+            arguments = _hack_away_prefix(
+                arguments = _data_arguments(ctx) | _required_arguments(ctx) | _orfs_arguments(short = True, *[dep[OrfsInfo] for dep in ctx.attr.deps]) | _verilog_arguments(ctx.files.verilog_files, short = True),
+                prefix = config_short.root.path,
+            ),
             paths = [file.short_path for file in ctx.files.extra_configs],
         ),
     )
@@ -894,7 +900,10 @@ def _make_impl(ctx, stage, steps, forwarded_names = [], result_names = [], objec
     ctx.actions.write(
         output = config_short,
         content = _config_content(
-            arguments = extra_arguments | _data_arguments(ctx) | _required_arguments(ctx) | _orfs_arguments(ctx.attr.src[OrfsInfo], short = True),
+            arguments = _hack_away_prefix(
+                arguments = extra_arguments | _data_arguments(ctx) | _required_arguments(ctx) | _orfs_arguments(ctx.attr.src[OrfsInfo], short = True),
+                prefix = config_short.root.path,
+            ),
             paths = [file.short_path for file in ctx.files.extra_configs],
         ),
     )