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

[Duplicate] Allow repo rules to download multiple things in parallel. #20857

Closed
wants to merge 1 commit into from

Conversation

bazel-io
Copy link
Member

This is accomplished by adding a "block" argument to the "download" call. If set,
the call returns a "pending download" object with one single method that waits for the download to finish.

Appropriate care is taken that downloads don't hang around after the repository function finishes running.

Fixes #19674 .

RELNOTES: None.
Commit 73c1a1e

PiperOrigin-RevId: 588320352
Change-Id: Ib0f48b6c7c2a07e93a4af602b0045120bd418829

This is accomplished by adding a "block" argument to the "download" call. If set,
the call returns a "pending download" object with one single method that waits for the download to finish.

Appropriate care is taken that downloads don't hang around after the repository function finishes running.

Fixes bazelbuild#19674 .

RELNOTES: None.
PiperOrigin-RevId: 588320352
Change-Id: Ib0f48b6c7c2a07e93a4af602b0045120bd418829
@bazel-io bazel-io added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. team-Core Skyframe, bazel query, BEP, options parsing, bazelrc labels Jan 11, 2024
@bazel-io bazel-io requested a review from a team as a code owner January 11, 2024 19:12
@bazel-io bazel-io added the awaiting-review PR is awaiting review from an assigned reviewer label Jan 11, 2024
@iancha1992 iancha1992 requested review from lberki and removed request for a team and lberki January 11, 2024 19:13
@iancha1992
Copy link
Member

Duplicate of #20856

@iancha1992 iancha1992 marked this as a duplicate of #20856 Jan 11, 2024
@iancha1992 iancha1992 closed this Jan 11, 2024
@iancha1992 iancha1992 changed the title [7.1.0] Allow repo rules to download multiple things in parallel. [Duplicate] Allow repo rules to download multiple things in parallel. Jan 11, 2024
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Core Skyframe, bazel query, BEP, options parsing, bazelrc team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants