Skip to content

Conversation

@sheepman4267
Copy link
Member

Summary
New builddep of python-keyring

Test Plan

  • Built. Installed. Built the new version of python-keyring which now requires this. Worked.

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

**Summary**
New builddep of python-keyring
Copy link
Member

@HarveyDevel HarveyDevel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@HarveyDevel HarveyDevel merged commit 0c20b91 into main Dec 16, 2025
1 check passed
@HarveyDevel HarveyDevel deleted the add-coherent-licensed branch December 16, 2025 08:22
@github-project-automation github-project-automation bot moved this from Triage to Done in Solus Dec 16, 2025
HarveyDevel added a commit that referenced this pull request Dec 16, 2025
**Summary**
Release notes can be found
[here](https://github.com/jaraco/keyring/blob/HEAD/NEWS.rst)

Requires #7404 to be merged first.

**Test Plan**
On Solus Plasma:
- Install `python-keyring` from the repo.
- Run `python -m keyring -b keyring.backends.kwallet.DBusKeyringKWallet4
set test test`, then enter a password.
- See that it fails with a BIG UGLY EXCEPTION.
- Build and install `python-keyring` from this PR.
- Try `python -m keyring -b keyring.backends.kwallet.DBusKeyringKWallet4
set test test` again.
- See that it still fails, but with a nice error message this time. This
means that applications using the library internally and auto-selecting
a backend will not fail badly if they try `DBusKeyringKWallet4`.

**Checklist**

- [x] Package was built and tested against unstable
- [ ] This change could gainfully be listed in the weekly sync notes
once merged <!-- Write an appropriate message in the Summary section,
then add the "Topic: Sync Notes" label -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants