Skip to content

Ensure runfiles.{,root_}symlinks makes it into tarballs#1025

Merged
tonyaiuto merged 1 commit intobazelbuild:mainfrom
EdSchouten:eschouten/20260212-symlinks
Feb 19, 2026
Merged

Ensure runfiles.{,root_}symlinks makes it into tarballs#1025
tonyaiuto merged 1 commit intobazelbuild:mainfrom
EdSchouten:eschouten/20260212-symlinks

Conversation

@EdSchouten
Copy link
Contributor

Right now we only add files. This is problematic when creating a tarball of an sh_binary(). Starting with Bazel 9, sh_binary() is no longer provided by Bazel itself. In that case rules_shell will declare its own copy of runfiles.bash, putting a symlink at the old location in the runfiles directory.

Right now we only add files. This is problematic when creating a tarball
of an sh_binary(). Starting with Bazel 9, sh_binary() is no longer
provided by Bazel itself. In that case rules_shell will declare its own
copy of runfiles.bash, putting a symlink at the old location in the
runfiles directory.
@EdSchouten EdSchouten force-pushed the eschouten/20260212-symlinks branch from f24831f to 7190d4b Compare February 12, 2026 10:37
Copy link
Collaborator

@cgrindel cgrindel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @aiuto I will not merge this PR so that you can take a look as well.

@tonyaiuto tonyaiuto merged commit dab5918 into bazelbuild:main Feb 19, 2026
6 checks passed
@EdSchouten EdSchouten deleted the eschouten/20260212-symlinks branch February 20, 2026 05:24
@EdSchouten
Copy link
Contributor Author

Thanks!

rdesgroppes added a commit to DataDog/datadog-agent that referenced this pull request Feb 21, 2026
### What does this PR do?
Bump `rules_pkg` to Feb 21 `main, picking up:
- bazelbuild/rules_pkg#1021
- bazelbuild/rules_pkg#1024
- bazelbuild/rules_pkg#1025

### Motivation
The missing exit code propagation fixed by bazelbuild/rules_pkg#1024 is the main driver of this change because RPM build errors would go unnoticed.
rdesgroppes added a commit to DataDog/datadog-agent that referenced this pull request Mar 20, 2026
### What does this PR do?
Bump `rules_pkg` to Feb 21 `main, picking up:
- bazelbuild/rules_pkg#1021
- bazelbuild/rules_pkg#1024
- bazelbuild/rules_pkg#1025

### Motivation
The missing exit code propagation fixed by bazelbuild/rules_pkg#1024 is the main driver of this change because RPM build errors would go unnoticed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants