<table><tr><td style="">awilcox created this revision.<br />awilcox added a reviewer: aacid.<br />awilcox added a project: KDE Edu.<br />Restricted Application added a subscriber: KDE Edu.<br />awilcox requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11700">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Right now, Blinken does not require Phonon in CMake:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">-- The following OPTIONAL packages have not been found:

 * Phonon4Qt5

-- Configuring done
-- Generating done
-- Build files have been written to: /usr/src/packages/user/blinken/src/blinken-17.12.2</pre></div>

<p>However, this results in a build failure:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">[ 76%] Building CXX object src/CMakeFiles/blinken.dir/main.cpp.o
In file included from /usr/src/packages/user/blinken/src/blinken-17.12.2/src/soundsplayer.cpp:10:0:
/usr/src/packages/user/blinken/src/blinken-17.12.2/src/soundsplayer.h:13:30: fatal error: phonon/MediaObject: No such file or directory
 #include <phonon/MediaObject>
                              ^
compilation terminated.
make[2]: *** [src/CMakeFiles/blinken.dir/build.make:119: src/CMakeFiles/blinken.dir/soundsplayer.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
[ 76%] Built target doc-index-cache-bz2
In file included from /usr/src/packages/user/blinken/src/blinken-17.12.2/src/blinkengame.cpp:17:0:
/usr/src/packages/user/blinken/src/blinken-17.12.2/src/soundsplayer.h:13:30: fatal error: phonon/MediaObject: No such file or directory
 #include <phonon/MediaObject>
                              ^
compilation terminated.
make[2]: *** [src/CMakeFiles/blinken.dir/build.make:143: src/CMakeFiles/blinken.dir/blinkengame.cpp.o] Error 1</pre></div>

<p>This patch forces Phonon to be required.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tested on git master and 17.12.2 release tarballs on Linux.  Since this is an unconditional header I cannot see this requiring testing on other platforms; the behaviour should be the same.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R324 Blinken</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11700">https://phabricator.kde.org/D11700</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>awilcox, aacid<br /><strong>Cc: </strong>kde-edu, KDE Edu, narvaez, apol<br /></div>