bazelbuild / bazelbuild/bazel-skylib
run_binary rule should support $(execpath ...) etc.
- Dominant language
- Starlark
- Stars
- 444
- Forks
- 202
- PR merge metrics
- No merged PRs in 30d
Description
Right now `run_binary` only supports `$(location …)`, cf. https://github.com/bazelbuild/bazel-skylib/blob/5bfcb1a684550626ce138fe0fe8f5f702b3764c3/rules/run_binary.bzl#L37. It should also support `$(execpath …)` and possible (if it makes sense) `$(rootpath …)` and `$(rlocationpath …)`. (Same with the pluralized forms, of course.)
Contributor guide
Research direction
Read rules/run_binary.bzl around the linked line to understand the existing $(location …) handling. Check the surrounding rule tests, if present, and determine the expected behavior for execpath, rootpath, rlocationpath, and their plural forms. Done means the supported substitutions work consistently without changing existing location behavior.
Written by the indexing model from the issue text.
Assessment
- Domain
- build-system
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100