D28360: cmake: make xsettingsd check an option

[Mikhail Zolotukhin]

gikari added a comment.


  In D28360#636435 <https://phabricator.kde.org/D28360#636435>, @eszlari wrote:
  
  > In D28360#636418 <https://phabricator.kde.org/D28360#636418>, @gikari wrote:
  >
  > > But the option is enabled by default and needs to be explicitly disabled. This seems to be an overengineered solution. It basically says, that XSettingsd is an optional dependency, which already is.
  >
  >
  > But it make the packager aware, that this dependency exists, instead of at runtime, the end user wondering, why his/her Plasma desktop behaves differently from others.
  >
  > Anecdote: obs-studio checks for pulseaudio at compile time, but fails silently(!) when it can't find it, leading users to wonder, why they can't record audio.
  
  
  Again, why not just make XSettingsd required? It's simpler.

REPOSITORY
  R99 KDE Gtk Configuration Tool

REVISION DETAIL
  https://phabricator.kde.org/D28360

To: eszlari, gikari
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200327/d6a97a40/attachment.html>