Bug 2412451
| Summary: | [virt-xml] fails with python exception | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Jelle van der Waa <jvanderwaa> |
| Component: | virt-manager | Assignee: | Cole Robinson <crobinso> |
| Status: | NEW --- | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | high | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | berrange, crobinso, ngompa13, nixuser, phrdina, suraj.ghimire7 |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Linux | ||
| URL: | https://artifacts.dev.testing-farm.io/de8ebb17-8433-4848-a9d1-c3ee2b44fb06/work-basicj1prldf9/plans/all/basic/execute/data/guest/default-0/test/browser/basic-1/output.txt | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | --- | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | Type: | --- | |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Jelle van der Waa
2025-11-05 09:57:15 UTC
@ngompa13 I see you dropped the `Requires: gobject-introspection` from python3-gobject. Can you advise, what's the expectation here? Should libraries that provide gobject-introspection bindings pull it in, or should applications that use those bindings add the dep? Maybe we should avoid this for f43 since this could trickle out dependency bugs (In reply to Cole Robinson from comment #1) > @ngompa13 I see you dropped the `Requires: gobject-introspection` > from python3-gobject. > > Can you advise, what's the expectation here? Should libraries that provide > gobject-introspection bindings pull it in, or should applications that use > those bindings add the dep? > > Maybe we should avoid this for f43 since this could trickle out dependency > bugs gobject-introspection was replaced with a glib2-internal implementation in v2.80, so applications and libraries need to switch to that. pygobject switched in 3.52 to this, so you need to migrate as well. Migration documentation is provided here: https://docs.gtk.org/girepository/migrating-gi.html (In reply to Neal Gompa from comment #2) > (In reply to Cole Robinson from comment #1) > > @ngompa13 I see you dropped the `Requires: gobject-introspection` > > from python3-gobject. > > > > Can you advise, what's the expectation here? Should libraries that provide > > gobject-introspection bindings pull it in, or should applications that use > > those bindings add the dep? > > > > Maybe we should avoid this for f43 since this could trickle out dependency > > bugs > > gobject-introspection was replaced with a glib2-internal implementation in > v2.80, so applications and libraries need to switch to that. pygobject > switched in 3.52 to this, so you need to migrate as well. > > Migration documentation is provided here: > https://docs.gtk.org/girepository/migrating-gi.html That's talking about the C api, but apps using pygobject aren't directly exposed to that, so it doesn't seem like code changes should be needed. AFAICT all that's required is to add the missing "Requires: gobject-introspection" dep to virt-xml which should always have been present, as we shouldn't rely on indirect deps for things we directly need. I don't see how this is virt-xml issue.
From the original error message we try to import libosinfo using `from gi.repository import Libosinfo`.
If I try this in fedora rawhide/43 container where I install only python3-gobject and libosinfo and run the following in python:
import gi
from gi.repository import Libosinfo
I'll get the same error.
For some reason `gobject-introspection` provides `/usr/lib64/girepository-1.0/libxml2-2.0.typelib` and without this file present importing libosinfo fails.
Based on all of that `gobject-introspection` needs to be added to libosinfo package as dependency.
Sigh. We really need a typelib dependency generator like every other RPM distro... :( |