Skip to content

Commit 10711d3

Browse files
acozzetteclaude
andcommitted
refactor: simplify resolve_execroot_bin_path condition
The exec_path == target_path guard was redundant: when the two paths are identical (same exec/target platform), [ -e "$exec_path" ] produces the same result as [ -e "$target_path" ], so the caller gets the correct path either way. Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
1 parent b8f486e commit 10711d3

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

js/private/js_binary.sh.tpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ function resolve_execroot_bin_path {
105105
fi
106106
local exec_path="$JS_BINARY__EXECROOT/$JS_BINARY__BINDIR/$suffix"
107107
local target_path="$JS_BINARY__EXECROOT/${BAZEL_BINDIR:-$JS_BINARY__BINDIR}/$suffix"
108-
if [ "$exec_path" = "$target_path" ] || [ -e "$exec_path" ]; then
108+
if [ -e "$exec_path" ]; then
109109
echo "$exec_path"
110110
else
111111
echo "$target_path"

js/private/test/snapshots/launcher.sh

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)