Skip to content

[bazel] Replace git_repository with http_archive. #99422

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

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

rupprecht
Copy link
Collaborator

The current git_repository usage points to tags, which leads to warnings that the build may not be reproducable due to not using a git sha.

The docs for git_repository recommend using http_archive, so switch to that instead. Also bump to newer versions for these two repos.

The current git_repository usage points to tags, which leads to warnings that the build may not be reproducable due to not using a git sha.

The docs for [git_repository](https://bazel.build/rules/lib/repo/git#git_repository) recommend using `http_archive`, so switch to that instead. Also bump to newer versions for these two repos.
@rupprecht rupprecht requested a review from keith as a code owner July 18, 2024 03:05
@llvmbot llvmbot added the bazel "Peripheral" support tier build system: utils/bazel label Jul 18, 2024
@rupprecht rupprecht merged commit e1d0913 into llvm:main Jul 19, 2024
7 checks passed
yuxuanchen1997 pushed a commit that referenced this pull request Jul 25, 2024
Summary:
The current git_repository usage points to tags, which leads to warnings
that the build may not be reproducable due to not using a git sha.

The docs for
[git_repository](https://bazel.build/rules/lib/repo/git#git_repository)
recommend using `http_archive`, so switch to that instead. Also bump to
newer versions for these two repos.

Test Plan: 

Reviewers: 

Subscribers: 

Tasks: 

Tags: 


Differential Revision: https://phabricator.intern.facebook.com/D60251492
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bazel "Peripheral" support tier build system: utils/bazel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants