Skip to content

[9.2.0] Fix ArtifactFactory.getPathFromSourceExecPath for external repos (https://github.com/bazelbuild/bazel/pull/29657)#29722

Open
bazel-io wants to merge 1 commit into
bazelbuild:release-9.2.0from
bazel-io:cp29657-9.2.0-191246
Open

[9.2.0] Fix ArtifactFactory.getPathFromSourceExecPath for external repos (https://github.com/bazelbuild/bazel/pull/29657)#29722
bazel-io wants to merge 1 commit into
bazelbuild:release-9.2.0from
bazel-io:cp29657-9.2.0-191246

Conversation

@bazel-io
Copy link
Copy Markdown
Member

@bazel-io bazel-io commented Jun 2, 2026

This function is only called for include scanning, which isn't used
widely externally today. Previously if a source file was from an
external repo it would not be correctly found by this function, which
assumed all source files are relative to the workspace.

Closes #29657.

PiperOrigin-RevId: 924751755
Change-Id: Ifadb17823a3247b98cbeea13c98e31b05de8e19e

Commit a11ca57

…elbuild#29657)

This function is only called for include scanning, which isn't used
widely externally today. Previously if a source file was from an
external repo it would not be correctly found by this function, which
assumed all source files are relative to the workspace.

Closes bazelbuild#29657.

PiperOrigin-RevId: 924751755
Change-Id: Ifadb17823a3247b98cbeea13c98e31b05de8e19e
@bazel-io bazel-io requested a review from a team as a code owner June 2, 2026 19:13
@bazel-io bazel-io added team-Performance Issues for Performance teams awaiting-review PR is awaiting review from an assigned reviewer labels Jun 2, 2026
@bazel-io bazel-io requested review from Wyverald and fmeum June 2, 2026 19:13
@iancha1992 iancha1992 removed the request for review from fmeum June 2, 2026 19:14
@iancha1992 iancha1992 enabled auto-merge June 2, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-Performance Issues for Performance teams

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants