dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed.
Please make sure the first line of the commit log says CONFIG and not DATA, too (the phab title still says DATA). INLINE COMMENTS > startplasma.cpp:197 > + if (!qEnvironmentVariableIsSet("XDG_CONFIG_DIRS")) { > + qputenv("XDG_CONFIG_DIRS", KDE_INSTALL_FULL_CONFDIR > ":/etc/xdg:/usr/local/etc/xdg"); > + } Wait, where does /usr/local/etc/xdg come from? https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html says the default is only /etc/xdg. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D24810 To: tcberner, #freebsd, dfaure, apol Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart