Skip to content
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

[8.0.0] Allowlist mac indexing directory #24130

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

bazel-io
Copy link
Member

We were checking every directory in the sandbox to make sure we keep track of
everything that is stored there. On macOS .DS_Store is a valid
file that shouldn't trigger a crash.

Fixes #23963

RELNOTES:none
PiperOrigin-RevId: 691061742
Change-Id: I085a38fa174cd71682bb599ab3cb1ade0dcd350d

Commit e297727

We were checking every directory in the sandbox to make sure we keep track of
everything that is stored there. On macOS .DS_Store is a valid
file that shouldn't trigger a crash.

Fixes bazelbuild#23963

RELNOTES:none
PiperOrigin-RevId: 691061742
Change-Id: I085a38fa174cd71682bb599ab3cb1ade0dcd350d
@bazel-io bazel-io requested a review from a team as a code owner October 29, 2024 17:07
@bazel-io bazel-io added team-Local-Exec Issues and PRs for the Execution (Local) team awaiting-review PR is awaiting review from an assigned reviewer labels Oct 29, 2024
@iancha1992 iancha1992 added this pull request to the merge queue Oct 30, 2024
Merged via the queue into bazelbuild:release-8.0.0 with commit 7f06964 Oct 30, 2024
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Local-Exec Issues and PRs for the Execution (Local) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants