bazel: resolve runfiles root without hardcoding the workspace name

Gemini: the lint test scripts derived RUNFILES_DIR with ${PWD%/_main},
which strips nothing when the main workspace is not named "_main" (e.g.
when OpenROAD is consumed as a dependency), leaving RUNFILES_DIR pointing
one level too deep. Strip the last path component with ${PWD%/*} instead
— the workspace subdirectory we start in — so the .runfiles root is found
regardless of the workspace name.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>

Author: oharboe

bazel/bzl_lint_test.sh

@@ -11,9 +11,12 @@ GIT="$(realpath "$2")"
 
 # With rules_python's script bootstrap (bootstrap_impl=script), a py_binary
 # invoked via its realpath cannot locate its runfiles once we cd away.
-# Export the runfiles root we start in (<target>.runfiles/_main) so nested
-# tools resolve it from the environment instead.
-export RUNFILES_DIR="${RUNFILES_DIR:-${PWD%/_main}}"
+# Export the runfiles root (the .runfiles directory) so nested tools
+# resolve it from the environment instead. We start in the workspace
+# subdirectory of it (<target>.runfiles/<workspace>); strip the last path
+# component rather than a hardcoded workspace name so this also holds when
+# the workspace is not "_main" (e.g. consumed as a dependency).
+export RUNFILES_DIR="${RUNFILES_DIR:-${PWD%/*}}"
 
 # MODULE.bazel must be in the sh_test `data` deps so it appears as a
 # runfiles symlink pointing at the real workspace. `readlink` (no -f,

bazel/tcl_lint_test.sh

@@ -11,9 +11,12 @@ GIT="$(realpath "$2")"
 
 # With rules_python's script bootstrap (bootstrap_impl=script), a py_binary
 # invoked via its realpath cannot locate its runfiles once we cd away.
-# Export the runfiles root we start in (<target>.runfiles/_main) so nested
-# tools resolve it from the environment instead.
-export RUNFILES_DIR="${RUNFILES_DIR:-${PWD%/_main}}"
+# Export the runfiles root (the .runfiles directory) so nested tools
+# resolve it from the environment instead. We start in the workspace
+# subdirectory of it (<target>.runfiles/<workspace>); strip the last path
+# component rather than a hardcoded workspace name so this also holds when
+# the workspace is not "_main" (e.g. consumed as a dependency).
+export RUNFILES_DIR="${RUNFILES_DIR:-${PWD%/*}}"
 
 WORKSPACE="$(dirname "$(readlink -f tclint.toml)")"
 cd "$WORKSPACE"