Skip to content

[9.1.0] Treat RUNFILES_TREE_MARKER as a directory in `BuildEvent.forArtifac…#29011

Merged
iancha1992 merged 1 commit intobazelbuild:release-9.1.0from
iancha1992:cp28612
Mar 17, 2026
Merged

[9.1.0] Treat RUNFILES_TREE_MARKER as a directory in `BuildEvent.forArtifac…#29011
iancha1992 merged 1 commit intobazelbuild:release-9.1.0from
iancha1992:cp28612

Conversation

@iancha1992
Copy link
Member

…t()`.

952aabb didn't fix #28330 because BuildEvent.forArtifact() (sometimes?) sees a FileArtifactValue.RUNFILES_TREE_MARKER, not a RunfilesProxyArtifactValue.

Fixes #28330.

PiperOrigin-RevId: 884388938
Change-Id: Ic5be6a83479d5afefd4a25a5b83049a2fe7b0244

Description

Motivation

Build API Changes

No

Checklist

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

Release Notes

RELNOTES: None

Commit 7bf8f2b

…t()`.

bazelbuild@952aabb didn't fix bazelbuild#28330 because `BuildEvent.forArtifact()` (sometimes?) sees a `FileArtifactValue.RUNFILES_TREE_MARKER`, not a `RunfilesProxyArtifactValue`.

Fixes bazelbuild#28330.

PiperOrigin-RevId: 884388938
Change-Id: Ic5be6a83479d5afefd4a25a5b83049a2fe7b0244
@iancha1992 iancha1992 requested a review from tjgq March 16, 2026 18:16
@iancha1992 iancha1992 requested a review from a team as a code owner March 16, 2026 18:16
@iancha1992 iancha1992 added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Mar 16, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Mar 17, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 17, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Mar 17, 2026
Merged via the queue into bazelbuild:release-9.1.0 with commit e2cf975 Mar 17, 2026
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants