-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replaced usage of rev
with awk
in bash runfiles
#20885
Replaced usage of rev
with awk
in bash runfiles
#20885
Conversation
Nixpkgs bazel build restricts the set of tools that are available by default. Using awk instead of rev reduces the need to add additional packages to the PATH
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@meteorcloudy Could you review this as well?
@bazel-io fork 7.1.0 |
Closes bazelbuild#20885. PiperOrigin-RevId: 599442183 Change-Id: I4d546ebebeb9007632e51f1ec000bb2332f19cb6
Closes #20885. Commit 9ae9d1d PiperOrigin-RevId: 599442183 Change-Id: I4d546ebebeb9007632e51f1ec000bb2332f19cb6 Co-authored-by: Nikhil Kalige <[email protected]>
The changes in this PR have been included in Bazel 7.1.0 RC1. Please test out the release candidate and report any issues as soon as possible. If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=last_rc. |
Nixpkgs bazel builds restricts the set of tools that are available by default to
Using
awk
instead ofrev
reduces the need to add additional packages to the PATH