D17154: Go back to SCSS
David Edmundson
noreply at phabricator.kde.org
Sat Jan 12 08:19:42 GMT 2019
davidedmundson added inline comments.
INLINE COMMENTS
> build_theme.sh:89-98
> + COLOR_SCHEME="/usr/share/color-schemes/Breeze.colors"
> + fi
> +else
> + THEME_NAME="${COLOR_SCHEME}"
> + if [ -f "/usr/share/color-schemes/${COLOR_SCHEME}.colors" ]; then
> + COLOR_SCHEME="/usr/share/color-schemes/${COLOR_SCHEME}.colors"
> + elif [ -f "${HOME}/.local/share/color-schemes/${COLOR_SCHEME}.colors" ]; then
We can't assume '/usr/share'/
Nor can we use the install prefix for where this is installed as that breaks distro packaging.
Breeze needs to export this location in it's installed .cmake file, then we should reference that.
I would also scrap all the stuff about ~/.local.
REPOSITORY
R98 Breeze for Gtk
REVISION DETAIL
https://phabricator.kde.org/D17154
To: gepardo, #plasma, jackg, #breeze, #vdg, ngraham, dirrukd, davidedmundson
Cc: lbeltrame, bshah, cgiboudeaux, davidedmundson, bcooksley, ngraham, jackg, plasma-devel, kvanton, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190112/d9014387/attachment.html>
More information about the Plasma-devel
mailing list