There should be no functional difference here, but it's not
obvious at a glance how qemu handles globally defined CFLAGS + LDFLAGS
with --extra-cflags and --extra-ldflags.
Reproduce the desired behavior with explicit configure options and
RPM variables
Signed-off-by: Cole Robinson <crobinso@redhat.com>
This is the historical intended behavior in the buildroot, but for
local builds, or with clang, qemu would detect a c++ compiler on the
host. So explicitly make the check fail by passing /bin/false
Signed-off-by: Cole Robinson <crobinso@redhat.com>
Commit 7a925b3802 ("spec: More RHEL merging") placed the %description
section right up against the list of Requires. This resulted in RPM
silently ignoring the Requires and adding them to the description
instead. As a result the qemu metapackage was totally broken.
Fixes: commit 7a925b3802
Fedora CI can't handle subpackages with Conflicts. There's ongoing
work to make it happen:
https://pagure.io/fedora-ci/general/issue/184
Once that lands we can re-enable these conflicts.
Signed-off-by: Cole Robinson <crobinso@redhat.com>
There's only one remaining ambiguous python shebang in qemu.git, and
it's not for anything we care about.
Also redhat-rpm-config brp-mangle-shebangs added in 2018 will do
/usr/bin/env python3 -> /usr/bin/python3 automatically for us
Signed-off-by: Cole Robinson <crobinso@redhat.com>
* Add tools_only path to only build qemu-img and qemu-guest-agent
* Add a -tests subpackage
* Install tracetool, simpletrace, dump-guest-memory
* Add vhost modprobe file
* Sync qemu-guest-agent file lists
* Use udevrulesdir from systemd
* Add more seabios and seavgabios roms
* More use of %{name} to make paths more generic
* Group all fedora specific %install content at the end
* Drop the rpath stripping, in my testing it's not needed anymore
* Drop the s390x iotests skippage from last year
Signed-off-by: Cole Robinson <crobinso@redhat.com>
We use `fmf` and `tmt` test config to run qemu-sanity-check,
similar to how its done in the spec file at the moment.
Add gating.yml that requires these tests to pass before updates
can be pushed
Signed-off-by: Cole Robinson <crobinso@redhat.com>
This moves the %build section to be closer to RHEL qemu-kvm content.
It's a bit tedious but it ensures we enable every feature we actually
think we are enabling :)
Add some macros to facilitate sharing this layout with the RHEL spec
The only functional change is now we use gcrypt instead of nettle.
I'm not sure if that was intentional before, but gcrypt is requested
on RHEL so I'm guessing that's what we want for Fedora too
Signed-off-by: Cole Robinson <crobinso@redhat.com>
This should not be any functional change, just some BuildRequires
movement and macro tweaking with the goal of sharing the first block
of deps with the RHEL/centos qemu-kvm spec
Signed-off-by: Cole Robinson <crobinso@redhat.com>
