Skip to content

Conversation

zeenix
Copy link
Contributor

@zeenix zeenix commented Jun 30, 2025

This fixes a regression that was exposed by 56cbb1f.

Fixes #1417.

@zeenix zeenix enabled auto-merge June 30, 2025 21:56
@zeenix zeenix merged commit f9870cd into dbus2:main Jun 30, 2025
8 checks passed
@zeenix zeenix deleted the fix-zv-regression branch June 30, 2025 22:03
@decathorpe
Copy link
Contributor

decathorpe commented Jul 7, 2025

Do you expect to publish a new release of zvariant with this fix? I'm working on all the Rust dependencies for GNOME 49 and snapshot pinning to zvariant ~5.4 currently prevents us from building it - and just relaxing the dependency would just expose this bug IIUC?

EDIT: If no immediate release is planned, I can just backport this patch to zvariant 5.5 that we have in Fedora, too.

@zeenix
Copy link
Contributor Author

zeenix commented Jul 7, 2025

EDIT: If no immediate release is planned, I can just backport this patch to zvariant 5.5 that we have in Fedora, too.

Usually I don't have a timeline. I'll release soon now that I know it's blocking stuff..

@decathorpe
Copy link
Contributor

Thanks - then there is no urgency, I've included the patch from this PR in the Fedora package for zvariant v5 for now. :)

@zeenix
Copy link
Contributor Author

zeenix commented Jul 7, 2025

@decathorpe zvariant 5.6 published to crates.io. I'll prep the release notes and tags soon..

@decathorpe
Copy link
Contributor

That works too. Thank you 🙌🏼

@zeenix
Copy link
Contributor Author

zeenix commented Jul 7, 2025

That works too. Thank you 🙌🏼

All done.

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.

org.freedesktop.zbus.Error: invalid length 19, expected >= 20
2 participants