Add option to deactivate searching for gnome keyring
ClosedPublic

Authored by steinhauser on Sep 28 2017, 5:32 PM.

Details

Summary

The current cmake files search for gnome keyring unconditionally, which on source-based distros would lead to a hard dependency on the package.
Hence, it would be a good idea to introduce an option to deactivate this if necessary.

Diff Detail

Repository
R875 Falkon
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
steinhauser created this revision.Sep 28 2017, 5:32 PM
steinhauser updated this revision to Diff 20177.Oct 1 2017, 7:49 AM

Use a more commonly used option naming style.

drosca added a subscriber: drosca.Oct 1 2017, 7:50 AM

Why is this needed? It is optional and used only for plugin (meaning if you build with gnome-keyring and run without it will just fail to load the plugin).

Why is this needed? It is optional and used only for plugin (meaning if you build with gnome-keyring and run without it will just fail to load the plugin).

The result of the build becomes unreproduceable, but depends on the availability of gnome-keyring instead.

Symmetry reasons might be another reason. It's possible controlling building of the KWallet plugin via CMAKE_DISABLE_FIND_PACKGE_KF5Wallet, but there's no way to do that for pkg_check_modules.

drosca accepted this revision.Oct 1 2017, 8:21 AM
This revision is now accepted and ready to land.Oct 1 2017, 8:21 AM
This revision was automatically updated to reflect the committed changes.