Skip to content
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

Update Qemu version to v6.1.0 if possible. #807

Merged
merged 2 commits into from
Jun 16, 2022

Conversation

Alexhuszagh
Copy link
Contributor

@Alexhuszagh Alexhuszagh commented Jun 16, 2022

The latest Qemu release contains numerous bug fixes, and other improvements, and the existing patches various issues reported in older Qemu versions. The only changes required are incrementing the Qemu version and adding ninja-build as a temporary dependency, since Qemu now uses it for the build system. However, Ubuntu 16.04, the base image for numerous dependencies we use, still ships with Python3.5, and Qemu 6.1.0 requires Python3.6+, as well as the ninja-build system. We therefore add a check to see if we have a compatible Python version, and if we do, build the last Qemu version, so our code can be fully backwards-compatible.

Closes #587.

@Alexhuszagh Alexhuszagh added enhancement A-qemu Area: qemu runners labels Jun 16, 2022
@Alexhuszagh Alexhuszagh requested a review from a team as a code owner June 16, 2022 07:33
@Alexhuszagh
Copy link
Contributor Author

If this looks good, and passes the tests, please don't merge it (yet) since I still have to update the README.

@Alexhuszagh
Copy link
Contributor Author

There's so many compatible targets it makes no sense to do a partial test.

bors try

bors bot added a commit that referenced this pull request Jun 16, 2022
@Alexhuszagh
Copy link
Contributor Author

Failed due to an unrelated error (TLS handshake timeout while fetching from the registry), but all the MUSL images and a few others seem to work, which means it should work for everything.

CHANGELOG.md Outdated Show resolved Hide resolved
@Emilgardis
Copy link
Member

r=me when readme updated, feel free to steal the ci and use it to parallel build and grab the versions to make sure it's correct, we should maybe even do that unconditionally for no other reason than curiosity now that we have a great way to do it

@bors
Copy link
Contributor

bors bot commented Jun 16, 2022

try

Build failed:

The latest Qemu release contains numerous bug fixes, and other improvements, and the existing patches various issues reported in older Qemu versions. The only changes required are incrementing the Qemu version and adding ninja-build as a temporary dependency, since Qemu now uses it for the build system. However, Ubuntu 16.04, the base image for numerous dependencies we use, still ships with Python3.5, and Qemu 6.1.0 requires Python3.6+, as well as the ninja-build system. We therefore add a check to see if we have a compatible Python version, and if we do, build the last Qemu version, so our code can be fully backwards-compatible.

Fixes cross-rs#557.
Closes cross-rs#587.
@Alexhuszagh
Copy link
Contributor Author

bors r=Emilgardis

@bors
Copy link
Contributor

bors bot commented Jun 16, 2022

Build succeeded:

@bors bors bot merged commit a4c444b into cross-rs:main Jun 16, 2022
@Emilgardis Emilgardis added this to the v0.2.2 milestone Jun 18, 2022
@Alexhuszagh Alexhuszagh deleted the qemu_rolling branch June 23, 2022 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-qemu Area: qemu runners enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants