D27360: Go back to saving to sddm.conf

Filip Fila noreply at phabricator.kde.org
Thu Feb 13 22:27:05 GMT 2020


filipf added a comment.


  In D27360#611155 <https://phabricator.kde.org/D27360#611155>, @davidedmundson wrote:
  
  > > The SDDM KCM is broken right now because the kde_settings.conf file is never created with the right permissions. This means that the KCM can't read the settings file.
  >
  > that doesn't match the bug report.
  >
  > The bug report shows it does have the right permissions. I'm not quite sure what he is trying to say the bug is.
  
  
  The bug report is a  bit all over the place, and it started out as something different so disregard most of the posts there.
  
  I've been at this for some time and the core issue is that `kde_settings.conf` is not world-readable like `sddm.conf` is. The settings are being written but only SDDM can read them, no one else.
  
  By all accounts the `openConfig` variable in `sddmauthelper.cpp` should set `kde_settings.conf`'s permissions like it does for `sddm.conf`.
  
  I can get it to make `kde_settings.conf` world-readable if I remove the `!file.exists` check but what's mind-boggling is this still doesn't work when `advancedconfig.cpp` i.e. the Advanced tab is in charge of generating `kde_settings.conf` so even adding some code that would set the correct permissions if it detects wrong one wouldn't solve the issue 100%.
  
  I've also tried defining `kde_settings.conf` in `config.h` and CMake but that didn't help.

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

To: filipf, davidedmundson, #plasma
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, 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/20200213/f1748387/attachment.html>


More information about the Plasma-devel mailing list