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

qemu: force --with kvmonly as default behavior #263

Merged

Conversation

mopsfelder
Copy link
Member

Since CentOS 7.5, sgabios-bin is not provided in the os/ repository. This arose
an issue in our qemu.spec, which mistakenly builds qemu-system-x86 on ppc64
architecture.

This patch assumes --with kvmonly as the default behavior, which won't build
qemu-system-x86 anymore; and also fixes some non-x86 files packaging.

Even though we're forcing --with kvmonly, we still provide qemu-system-ppc
package for compatibility reasons.

@farosas
Copy link
Member

farosas commented May 10, 2018

LGTM, but let's make sure the build passes

Since CentOS 7.5, sgabios-bin is not provided in the os/ repository. This arose
an issue in our qemu.spec, which mistakenly builds qemu-system-x86 on ppc64
architecture.

This patch assumes `--with kvmonly` as the default behavior, which won't build
qemu-system-x86 anymore; and also fixes some non-x86 files packaging.

Even though we're forcing `--with kvmonly`, we still provide qemu-system-ppc
package for compatibility reasons.
@mopsfelder
Copy link
Member Author

The reason why CI is failing is not related to the changes of this pull.
Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants