[Differential] [Accepted] D3551: Fix build system issues

kfunk (Kevin Funk) noreply at phabricator.kde.org
Wed Nov 30 21:02:15 UTC 2016


kfunk accepted this revision.
kfunk added a comment.
This revision is now accepted and ready to land.


  Feel free to push after fixing those style issues

INLINE COMMENTS

> CMakeLists.txt:67
> +                        SOURCES ${KDEVEMBEDDED_PART_SRCS}
> +                        )
> +kdevplatform_add_plugin(kdevembedded-launcher

Move closing paren to front

> CMakeLists.txt:95
> +configure_file(${CMAKE_CURRENT_SOURCE_DIR}/arduinoversion.h.in
> +            ${CMAKE_CURRENT_BINARY_DIR}/arduinoversion.h)
>  

Style: Indent off

> arduinoversion.h.in:1
> +#ifndef _ARDUINO_VERSION_
> +#define _ARDUINO_VERSION_

Style: Naming scheme is usually like this: `ARDUINOVERSION_H`

REPOSITORY
  R185 KDevelop Plugin: Embedded Platforms

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

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

To: obogdan, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel, #kdevelop, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161130/7a76cdcb/attachment-0001.html>


More information about the KDevelop-devel mailing list