-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
rust: update to 1.68.0 #16128
rust: update to 1.68.0 #16128
Conversation
|
aarch64 failed with
|
Not sure I can help here... |
Seems like on aarch64 the previous rust is not downloaded and extracted . A local mingw64 builds shows:
While the failed aarch64 build shows:
|
(I had to revert it to unblock the queue, I hope you understand) |
@lazka, no problems. It happened with 1.67.0 too (also because of aarch64..). Why is aarch64 not part of CI ? |
IIUC, the reason is because there are no GitHub-hosted runners for that platform. And @jeremyd2019's runner is busy building the packages in staging. |
It would also be a security issue since there is no job isolation; if we start building PRs the runner is tainted for staging builds. |
I'll create a meta issue for rust... |
Correct, AFAIK there is no stage0 for that triple (quad?) so we have to build rust with the previous rust package rather than have it download a stage0. To start us off, @mati865 cross-compiled a stage0.
Also a security issue that people could run arbitrary code on my runner by opening a PR, potentially maliciously. With the hosted github runner github takes on that risk. |
And the related patch got removed for rust 1.67.0 ? How did 1.67.0 got built ? |
Can you give me link to whole log from the build?
Correct, AArch64 should build with toolchain that already exists in the repo without the need to download things. |
The necessary fixes were upstreamed in 1.67.0. The other part of that patch had the 'injected' cross-compiled stage0, which was out of date and no longer used anyway, due to MINGW-packages/mingw-w64-rust/PKGBUILD Lines 7 to 10 in 4544b4c
We could maybe try |
I'll give it a try. |
I cannot test locally for some time as my HDD seems to be dying. |
That could be avoided by requiring an approval to run CI. https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks |
_bootstrapping=no for other envs fails with the same error |
Reverting rust-lang/rust@6f0fc2f fixes the issue. |
I created an issue upstream: rust-lang/rust#109103 |
See #16189 |
OK, I was thinking maybe |
No description provided.