dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> CMakeLists.txt:80
> +                    COMMAND ${CMAKE_COMMAND} -E
> +                        copy 
> ${CMAKE_BINARY_DIR}/bin/libcorebindingsplugin.so 
> ${CMAKE_BINARY_DIR}/bin/org/kde/plasma/core/libcorebindingsplugin.so)
> +add_custom_command(TARGET copy POST_BUILD

The filename won't work on Windows. Here's more portable syntax:

  add_custom_command(TARGET copy POST_BUILD
                      COMMAND ${CMAKE_COMMAND} -E
                      copy $<TARGET_FILE:corebindingsplugin> 
${CMAKE_BINARY_DIR}/bin/org/kde/plasma/core/)

REPOSITORY
  R242 Plasma Framework (Library)

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

To: davidedmundson, #plasma, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to