D27142: Have authhelper define config file paths on its own

David Edmundson noreply at phabricator.kde.org
Mon Feb 3 21:05:14 GMT 2020


davidedmundson added inline comments.

INLINE COMMENTS

> sddmkcm.cpp:55
>  
> -    mSddmConfig = KSharedConfig::openConfig(QStringLiteral(SDDM_CONFIG_FILE), KConfig::CascadeConfig);
> +    mSddmConfig = KSharedConfig::openConfig(QString {QLatin1String(SDDM_CONFIG_DIR "/") + QStringLiteral("kde_settings.conf")}, KConfig::CascadeConfig);
>  

I don't think this is right.

This file is added below:

  systemConfigFiles = QDir(QLatin1String(SDDM_SYSTEM_CONFIG_DIR)).entryList(QDir::Files | QDir::NoDotAndDotDot, QDir::LocaleAware);

loops through every file in /etc/sddm.conf.d

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

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


More information about the Plasma-devel mailing list