-
Notifications
You must be signed in to change notification settings - Fork 80
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
Add nixpkgs_http_repository #356
Conversation
A more general version of nixpkgs_git_repository that is not limited to the specific archives offered by Github, but can be applied to other HTTP provided repositories in general. This is a pre-requisite for some of the tags for the new Bazel module extension designed in design/bzlmod/README.md, in particular the http and github tags.
|
||
A unique name for this repository. | ||
|
||
url: String |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of this docstring is taken and adapted from http_archive
.
}, | ||
) | ||
|
||
def nixpkgs_http_repository( | ||
*, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hadn't seen this syntax before. It makes everything after *
be a keyword argument?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's correct.
This adds
nixpkgs_http_repository
a more general version ofnixpkgs_git_repository
that is not limited to the specific archives offered by Github, but can be applied to other HTTP provided repositories in general.This new rule will be used to implement the
http
andgithub
tags in the Nix repository module extension as described indesign/bzlmod/README.md
.nixpkgs_git_repository
is modified to callnixpkgs_http_repository
itself.