-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Download pre-compiled bootstrap from CI #115076
Conversation
|
Some changes occurred in src/tools/cargo cc @ehuss |
This comment has been minimized.
This comment has been minimized.
Sorry for the wrong cc. I accidentally forgot to reset the stdarch submodule. Fixed now |
For the last issue for modification change detection I am open to suggestion. I don't really comprehend my current take on the issue and it is really hard to test since there is no easy way to mock git rev-parse. |
The job Click to see the possible cause of the failure (guessed by this bot)
|
There also seems to be changes on the component download url format. I am not sure how the bootstrap artifacts fit into those |
I am quite busy and don't have time for mentoring tasks these days. Overally, the flow should include the steps I mentioned on the previous PR #112598 (comment) As for the catching the modification changes, you can find several references in the codebase(I see I even left a suggestion related to that on #112598 (comment)). @rustbot author |
☔ The latest upstream changes (presumably #116260) made this pull request unmergeable. Please resolve the merge conflicts. |
Sorry for the lack of progress on this. I've been quite busy for a while and couldn't find any time to work on this I am closing this PR in case someone wants to continue feel free to base from my changes. |
because of a silly rebase mistake i accidently nuked my branch which in turn closed the pr #112598. I recovered the branch using the github repo events and creating a new branch using their API.
r? @ozkanonur
Currently with no changes on the bootstrap directory while doing a pre-push hook. I am getting
Other than this issue (it triggers a download from ci even if it is not available or trying to extract the wrong file)