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

Reply via email to