[Differential] [Changed Subscribers] D2372: Make powerdevil normal executable instead of kded module

shumski (Hrvoje Senjan) noreply at phabricator.kde.org
Mon Aug 8 14:41:48 UTC 2016


shumski added inline comments.

INLINE COMMENTS

> CMakeLists.txt:135
>  
> -install(TARGETS kded_powerdevil DESTINATION ${PLUGIN_INSTALL_DIR}/kf5/kded)
> +install(TARGETS powerdevil DESTINATION ${CMAKE_INSTALL_FULL_LIBEXECDIR_KF5})
> +install(FILES ${CMAKE_CURRENT_BINARY_DIR}/powerdevil.desktop

PowerDevil isn't a Framework, so this var is wrong.

> powerdevil.desktop.cmake:1
>  [Desktop Entry]
>  Type=Service

Maybe autostart for this should be limited to Plasma only?

REPOSITORY
  rPOWERDEVIL Powerdevil

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: bshah, #plasma, broulik, graesslin
Cc: shumski, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160808/a5cd2252/attachment.html>


More information about the Plasma-devel mailing list