[M] Change in bazel/bazel[master]: Fix ArtifactFactory.getPathFromSourceExecPath for external repos (htt...

[Copybara Service (Gerrit)]

Copybara Service uploaded new patchset

Copybara Service uploaded patch set #6 to this change.

Open in Gerrit

Related details

Attention set is empty

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

Gerrit-MessageType: newpatchset

Gerrit-Project: bazel

Gerrit-Branch: master

Gerrit-Change-Id: Ifadb17823a3247b98cbeea13c98e31b05de8e19e

Gerrit-Change-Number: 328550

Gerrit-PatchSet: 6

Gerrit-Owner: Copybara Service <copybara-wor...@google.com>

Gerrit-CC: Keith Smiley <keithb...@gmail.com>

[Copybara Service (Gerrit)]

Copybara Service submitted the change

Change information

Commit message:

Fix ArtifactFactory.getPathFromSourceExecPath for external repos (https://github.com/bazelbuild/bazel/pull/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 #29657.

PiperOrigin-RevId: 924751755

Change-Id: Ifadb17823a3247b98cbeea13c98e31b05de8e19e

Files:

• M src/main/java/com/google/devtools/build/lib/actions/ArtifactFactory.java
• M src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java

Change size: M

Delta: 2 files changed, 53 insertions(+), 3 deletions(-)

Branch: refs/heads/master

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

Gerrit-MessageType: merged