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

Replace build dependency on libgnome-keyring by libsecret #6

Closed
sunweaver opened this issue Jan 15, 2018 · 4 comments
Closed

Replace build dependency on libgnome-keyring by libsecret #6

sunweaver opened this issue Jan 15, 2018 · 4 comments

Comments

@sunweaver
Copy link

sunweaver commented Jan 15, 2018

The GNOME people in Debian are asking for replacement of libgnome-keyring in caja-seahorse and use libsecret instead.

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=886476

They even started raising severity which threatens mate-power-manager to be auto-removed from Debian testing. Any porting work on this in sight?

If you have any questions or are in need of help, please contact the Debian+Ubuntu MATE Packaging Team on OFTC IRC (irc.debian.org, channel #debian-mate) or upstream MATE devs on Freenode IRC (channel: #mate-dev). Thanks!

@darkshram
Copy link
Owner

Ok. I'll start as soon as my work schedule allows it. Will try to do it myself, but guess l'll need help at some point if there is a deadline.

@darkshram
Copy link
Owner

darkshram commented Jan 17, 2018

Seems It's done in my test machine. There are no includes for libgnome-keyring, neither major code modifications were needed. Upstream code for latest stable (as you know seahorse-caja is a fork of seahorse-nautilus) just links to libgnome-keyring, but the only relevant code used to access the keyring is through gcr library. I'm going to do some more testing and review the code (after a quick review, seems nothing is really used to justify libgnome-keyring dependency). For the time being, just replaced the libgnome-keyring dependency with libsecret, but seems does not really use it either, because it works perfectly without any code modification.

Please, do me a favor and help me testing this patch.
seahorse-caja-libsecret.zip

Tomorrow I'll do some extra testing removing libsecret and test if everything works. Recentely seahorse-nautilus migrated from autotools to meson, and current commit with meson config does not include any reference to libgnome-keyring or libsecret. So seems libgnome-keyring dependency is just a left-over form earlier development.

If things work as I expect, will release 1.18.4 this week.

@darkshram
Copy link
Owner

Ok. I just did the test. libgnome-keyring is not used. It was a left-over from earlier development. So, just removed this dependecy. Code works fine. So, I'm releasing 1.18.4.

@sunweaver
Copy link
Author

sunweaver commented Jan 19, 2018 via email

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

No branches or pull requests

2 participants