-
Notifications
You must be signed in to change notification settings - Fork 5
Add ability to override the download URL manually #9
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
|
how does this work for multiple platforms? since a toolchain can target N platforms but this is only 1 url? |
|
that would need a little more work than I have here. worst case you can do a |
|
you can't select in the module.bazel, maybe what this use case wants instead is a |
|
ah right. yea i guess this just can't do multiplatform stuff then in its current state. |
|
it gets a little weird because you'd need to plumb through a way to put the platform name in the url, and i don't know if there's a clear cut way to do that. |
|
modeling after toolchains_llvm the solution would be to make |
|
yea open to suggestions here. the only use case we really have is testing with a local package for our nightly qualifications, so any other way we can do that is fine with me. |
|
i did have this one open |
|
i wouldn't object to replacing this with that. i saw it at the time but not until i had already merged this one lol. |
No description provided.