Drop dependency on security/libgnome-keyring in some gnome3 ports

[rm]

Summary

According to libsecret page libsecret replaces libgnome-keyring, so this is a trend in gnome community, that didn't reflected in our ports tree.

This ports mistakenly depends on deprecated security/libgnome-keyring, while they didn't actually required it or required security/libsecret instead:

• databases/evolution-data-server
• net/gnome-online-accounts
• net/vinagre
• net/vino
• www/epiphany
• security/gnome-keyring
• x11/gnome-shell
• x11/gdm

Patch in review resolves all the occurrences of libgnome-keyring. For some ports there are corresponding NEWS entry. For example:

databases/evolution-data-server
./NEWS: * Replaced gnome-keyring-1 with libsecret-1 (since 3.7.1)

net/vinagre
./NEWS: Migrate from libgnome-keyring to libsecret

net/vino
./NEWS: Migrate from libgnome-keyring to libsecret

This was checked both with ldd and manual source code grepping. Now testing in poudriere.

[kwm]

x11/gdm/Makefile

96 ↗

(On Diff #14457)

Since we always depend on gnome-keyring, how about axing the whole KEYRING options completly.

so the SUB_LIST+= PAM_KEYRING lines and the %%PAM_KEYRING%% keywords, not the lines, can be removed from files/gdm-password.in

[rm]

pam_gnome_keyring.so is installed by gnome-keyring, so it's always available, but user may not want to use it. This is why I decided to retain the option.

x11/gdm/Makefile

96 ↗

(On Diff #14457)

pam_gnome_keyring.so is installed by gnome-keyring, so it's always available, but user may not want to use it. This is why I decided to retain the option.

[rm]

Drop KEYRING option altogether.

[mat]

Could you use devel/arcanist to upload the diff, or if you have some allergies to php or something, update the diff using:

svn diff --diff-cmd=diff -x -U9999

So that we get context for the review.

[rm]

Reuploadded the diff with svn diff --diff-cmd=diff -x -U9999

[mat]

x11/gdm/files/gdm-password.in

6 ↗

(On Diff #14460)

Should be LOCALBASE or PREFIX here.

[kwm]

x11/gdm/files/gdm-password.in

6 ↗

(On Diff #14460)

Good catch. It should be LOCALBASE, since that file is installed by the consolekit port.