-
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
Preserve --experimental_allow_unresolved_symlinks in exec cfg #15702
Conversation
@lberki Could you take a look? |
/cc @katre I can't imagine you not wanting this change, so I'll go ahead and approve it. |
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.
Thanks! You even added a test case :)
@lberki Will you merge, or shall I? |
@katre I thought that that's what the "awaiting-PR-merge" label is for and that gTech is taking care of it now. That said, if it's urgent I can merge it myself (tomorrow) |
I had no idea that label existed. The future is now! |
@bazel-io flag |
@bazel-io fork 5.3.0 |
See bazelbuild#10298 (comment) Work towards bazelbuild#10298 Closes bazelbuild#15702. PiperOrigin-RevId: 456480627 Change-Id: Ib1237ce9e2c0bab0d7eabd8743f08ec60b8b7ab7
See #10298 (comment) Work towards #10298 Closes #15702. PiperOrigin-RevId: 456480627 Change-Id: Ib1237ce9e2c0bab0d7eabd8743f08ec60b8b7ab7 Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
See #10298 (comment)
Work towards #10298