bazelbuild / bazelbuild/bazel-skylib

run_binary rule should support $(execpath ...) etc.

Open
#425 5 comments 2 reactions 0 assignees View on GitHub
good first issue P2 type: bug
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.