Skip to content

Download expat as an archive and make it an isolated bazel module - #3176

Open
Databean wants to merge 3 commits into
google:mainfrom
Databean:expat_module
Open

Download expat as an archive and make it an isolated bazel module#3176
Databean wants to merge 3 commits into
google:mainfrom
Databean:expat_module

Conversation

@Databean

Copy link
Copy Markdown
Member

Using http over git is recommended by bazel: https://bazel.build/rules/lib/repo/git

Prefer http_archive to git_repository.

It can no longer depend on build targets from the root module, and has an isolated build. The root module and other modules can still depend on it.

This is a step on the way to isolating the crosvm build, so its custom rust configuration doesn't affect the root module. It is part of the dependency chain of crosvm -> wayland -> expat.

Bug: b/559825860

This is recommended by bazel: https://bazel.build/rules/lib/repo/git

> Prefer http_archive to git_repository.

Bug: b/559825860
@Databean
Databean requested a review from jmacnak September 11, 2026 23:50
@Databean Databean added the kokoro:run Run e2e tests. label Sep 11, 2026
@Databean
Databean enabled auto-merge September 11, 2026 23:50
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Sep 12, 2026
It can no longer depend on build targets from the root module, and has
an isolated build. The root module and other modules can still depend on
it.

This is a step on the way to isolating the crosvm build, so its custom
rust configuration doesn't affect the root module. It is part of the
dependency chain of crosvm -> wayland -> expat.

Bug: b/559825860
@Databean
Databean added this pull request to the merge queue Sep 12, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants