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

The systemd busctl tool chokes on introspection XML #20

Open
nojocodex opened this issue Jan 16, 2021 · 2 comments
Open

The systemd busctl tool chokes on introspection XML #20

nojocodex opened this issue Jan 16, 2021 · 2 comments

Comments

@nojocodex
Copy link
Contributor

The systemd busctl on Ubuntu 20.04 doesn't like the xmlns:doc attribute that dbus-objects adds to the <node> element of the introspection XML. Just leaving out the attribute makes busctl happy.

Reproduction:

$ busctl introspect <interface> <path>
Unexpected <node> attribute xmlns:doc.

Dunno if you want to fix this, as I think the problem is really in busctl (at least, there are official-ish examples which indicate including xmlns:doc is perfectly valid: https://dbus.freedesktop.org/doc/dbus-api-design.html). Also, I don't know whether anyone besides me actually uses busctl, so it may not be worth doing anything about.

OTOH, the xmlns:doc attribute seems optional (at least, there are official-ish examples without it: https://dbus.freedesktop.org/doc/dbus-specification.html#introspection-format), so a fix is easy and AFAICT harmless.

@FFY00
Copy link
Owner

FFY00 commented Jan 16, 2021

If this is causing issues it can be temporarily removed, but I'd like to see it fixed in busctl. Could you open an issue there? Please ping me if you do.

nojocodex added a commit to nojocodex/dbus-objects that referenced this issue Jan 16, 2021
nojocodex added a commit to nojocodex/dbus-objects that referenced this issue Jan 16, 2021
nojocodex added a commit to nojocodex/dbus-objects that referenced this issue Jan 16, 2021
@FFY00
Copy link
Owner

FFY00 commented Jan 16, 2021

Mitigated in #21, but I will keep this open to track the busctl progress.

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

No branches or pull requests

2 participants