D7103: Allow to build KSyntaxHighlighter without Qt5Gui

2017-08-04 Thread Volker Krause 
This revision was automatically updated to reflect the committed changes.
Closed by commit R216:703b48665f3d: Allow to build KSyntaxHighlighter without 
Qt5Gui (authored by vkrause).

REPOSITORY
  R216 Syntax Highlighting

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D7103?vs=17646&id=17721

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

AFFECTED FILES
  CMakeLists.txt
  src/CMakeLists.txt

To: vkrause, #frameworks, kfunk
Cc: kfunk

D7103: Allow to build KSyntaxHighlighter without Qt5Gui

2017-08-03 Thread Kevin Funk 
kfunk accepted this revision.
kfunk added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> CMakeLists.txt:3
> +if(TARGET Qt5::Gui)
> +add_subdirectory(lib)
> +add_subdirectory(cli)

Too bad it needs QtGui. Just b/c it uses `QColor` as far as I can see...

REPOSITORY
  R216 Syntax Highlighting

BRANCH
  master

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

To: vkrause, #frameworks, kfunk
Cc: kfunk

D7103: Allow to build KSyntaxHighlighter without Qt5Gui

2017-08-03 Thread Volker Krause 
vkrause created this revision.
Restricted Application added a project: Frameworks.

REVISION SUMMARY
  This actually only builds the indexer, which is useful for
  cross-compilation. With Yocto this avoids pulling in a full
  graphics stack for the host build.

REPOSITORY
  R216 Syntax Highlighting

BRANCH
  master

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

AFFECTED FILES
  CMakeLists.txt
  src/CMakeLists.txt

To: vkrause, #frameworks