-
Notifications
You must be signed in to change notification settings - Fork 12.1k
ci: fix cross-compile sync issues #12804
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
@slaren I am going to submit a separate PR tomorrow to (for now) remove the cross-compiling from the CI. I don't currently have time to get this one working, so that will be a workaround. |
…sync' into ci-build-cross-fix-sync
@slaren It looks like the solution was hiding in plain sight! The One thing I changed is how the ports repositories are added. Instead of mangling the default azure repositories (which only applies to Github runners), I opted to create a more realistic cross-compile example by directly adding the Ubuntu ports repos alongside the existing ones. This way folks could perform the same steps on a local Ubuntu system without issue. |
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.
Nice! Minor nit, the fetch-depth: 0
in the checkout actions shouldn't be necessary here, that's only used in the releases so that the build number can be calculated correctly.
Fixed the fetch depth will merge once build finishes. 😊 |
During the Linux cross-compile CI setup there seem to be issues syncing dependencies, causing unnecessary build failures and delays in CI success. This PR is an attempt to address that issue.