ervin requested changes to this revision.
ervin added a comment.
This revision now requires changes to proceed.


  Just a last nitpick.

INLINE COMMENTS

> CMakeLists.txt:24
> +kconfig_add_kcfg_files(kcm_cursortheme_PART_SRCS cursorthemesettings.kcfgc 
> GENERATE_MOC)
> +install(FILES cursorthemesettings.kcfg DESTINATION ${KDE_INSTALL_KCFGDIR})
>  add_library(kcm_cursortheme MODULE ${kcm_cursortheme_PART_SRCS})

Could that be moved in the install file section of the file? Everything related 
to file installs (but not target installs...) seems to be at the end of that 
file.

REPOSITORY
  R119 Plasma Desktop

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

To: bport, #plasma, mart, ervin, davidedmundson
Cc: davidedmundson, 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