<table><tr><td style="">rjvbb requested changes to this revision.<br />rjvbb added a comment.<br />This revision now requires changes to proceed.
</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/D17053">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p><a href="https://phabricator.kde.org/p/rjvbb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rjvbb</a> what do you say? makes sense?</p></blockquote>

<p>Yes, this is required. I've got a bunch of patches for Okular I never got around to sub/committing. <br />
The patch as proposed is OK as a packaging patch, but requires some polishing before it's fit for upstreaming. You can take inspiration from my version of this patch:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="diff" 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);"><span style="color: #a00000">--- a/CMakeLists.txt</span>
<span style="color: #00a000">+++ b/CMakeLists.txt</span>
<span style="color: #800080">@@ -216,7 +216,7 @@ generate_export_header(okularcore BASE_NAME okularcore EXPORT_FILE_NAME "${CMAKE</span>
 
 # Special handling for linking okularcore on OSX/Apple
 IF(APPLE)
<span style="color: #a00000">-    SET(OKULAR_IOKIT "-framework IOKit" CACHE STRING "Apple IOKit framework")</span>
<span style="color: #00a000">+    SET(OKULAR_APPLE_FRAMEWORKS "-framework IOKit -framework CoreGraphics -framework CoreFoundation" CACHE STRING "required Apple frameworks")</span>
 ENDIF(APPLE)
 
 # Extra library needed by imported synctex code on Windows
<span style="color: #800080">@@ -226,7 +226,7 @@ endif(WIN32)</span>
 
 target_link_libraries(okularcore
 PRIVATE
<span style="color: #a00000">-    ${OKULAR_IOKIT}</span>
<span style="color: #00a000">+    ${OKULAR_APPLE_FRAMEWORKS}</span>
     ${SHLWAPI}
     KF5::Archive
     KF5::JS</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17053">https://phabricator.kde.org/D17053</a></div></div><br /><div><strong>To: </strong>darcyshen, Okular, rjvbb<br /><strong>Cc: </strong>aacid, rjvbb, okular-devel, ngraham, darcyshen<br /></div>