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

Update dependencies #55

Closed
wants to merge 1 commit into from
Closed

Conversation

Victor-Savu
Copy link

@Victor-Savu Victor-Savu commented Jan 20, 2021

In particular, updating the dependency on secret-service-rs means that
keyring-rs no longer depends on the system dbus library on linux.

I know secret-service-rs is still in pre-release, but I thought I'd open the PR just so that it is ready when the time comes.

In particular, updating the dependency on secret-service means that
keyring-rs no longer depends on the system dbus library on linux.
@hwchen
Copy link
Owner

hwchen commented Jan 21, 2021

Hey, thanks for this. Sorry I’ve been a little slow on getting key ring updated, work has been pretty busy. I’ll try to get things rolling again.

@Victor-Savu
Copy link
Author

@hwchen Thank you so much for your work on keyring-rs and on the underlying platform-specific integrations for Windows, Linux and MacOS!
I initially did the update only to serve myself because I was just eager to get rid of the native dbus dependency, but when I saw how small the change was, I decided to open a PR and share back 🙂

@adobeDan
Copy link
Collaborator

Hi @Victor-Savu, thanks so much for leading the way here! We have now integrated the dependency updates in #66 so I closing this PR.

@adobeDan adobeDan closed this Nov 13, 2021
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.

3 participants