Skip to content

[AOSP-pick] Add/Update getLocalFiles method to support both BlazeArtifact and OutputArtifact type artifacts #7508

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

Conversation

LeFrosch
Copy link
Collaborator

Cherry pick AOSP commit ed8535e5178e2f3f0459e7c01de1902fb319d4c5.

STAT (diff to AOSP): 0 insertions(+), 0 deletion(-)

  • Add a method getLocalFilesForLegacySync to support processing
    BlazeArtifact artifact types.
  • Update existing getLocalFiles method to fetch local files for OutputArtifact type artifacts.
  • This will help migrate LocalFileArtifact#getLocalFiles to using RuntimeArtifactCache, which
    only supports OutputArtifact (as it expects artifacts to contain a
    digest).
  • Invokers of the method has been updated based on whether the flow is
    used in LegacySync or QuerySync (Native debugging).

Bug: 385469770
Test: Existing tests for testing old and new methods as their
implementation will only be updated in next CLs.
Change-Id: I6b7ecfd522f086b712c99e90fe30f1b3b3cb7abe

Change-Id: I523e59d72a3c239b234e0112c76b128e2bc19fb7

AOSP: ed8535e5178e2f3f0459e7c01de1902fb319d4c5

…putArtifact type artifacts

- Add a method getLocalFilesForLegacySync to support processing
  BlazeArtifact artifact types.
- Update existing getLocalFiles method to fetch local files for OutputArtifact type artifacts.
- This will help migrate LocalFileArtifact#getLocalFiles to using RuntimeArtifactCache, which
only supports OutputArtifact (as it expects artifacts to contain a
digest).
- Invokers of the method has been updated based on whether the flow is
  used in LegacySync or QuerySync (Native debugging).

Bug: 385469770
Test: Existing tests for testing old and new methods as their
implementation will only be updated in next CLs.
Change-Id: I6b7ecfd522f086b712c99e90fe30f1b3b3cb7abe

Change-Id: I523e59d72a3c239b234e0112c76b128e2bc19fb7

AOSP: ed8535e5178e2f3f0459e7c01de1902fb319d4c5
@LeFrosch LeFrosch force-pushed the AOSP/ed8535e5178e2f3f0459e7c01de1902fb319d4c5 branch from eb1be36 to 10bcaa5 Compare March 27, 2025 19:17
@sellophane sellophane merged commit 2852f1f into bazelbuild:master Mar 27, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Bazel IntelliJ Plugin Mar 27, 2025
@github-actions github-actions bot removed the awaiting-review Awaiting review from Bazel team on PRs label Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product: CLion CLion plugin product: GoLand GoLand plugin product: IntelliJ IntelliJ plugin
Projects
Development

Successfully merging this pull request may close these issues.

4 participants