<table><tr><td style="">apol added a comment.
</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/D12251">View Revision</a></tr></table><br /><div><div><p>I don't really see why we need a special path for parts.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I understand you're doing it to be able to just get everything from that folder and then put it on the apk</p>

<p>Now without knowing anything about cmake, wouldn't it make more sense put on the apk every single thing i'm installing?</p>

<p>I mean if i do</p>

<p>add_library(okularpart SHARED ${okularpart_SRCS})<br />
 install(TARGETS okularpart DESTINATION ${KDE_INSTALL_PLUGINDIR})<br />
 it's surely because i'm interesting in having okularpart in my apk, otherwise why would i spend precious time compiling and installing it?</p></blockquote>

<p>That's not a bad point, I think having all plugins in the same subdirectory is a good idea because it simplifies the process of looking for them. Instead of having a complex cache such as sycoca that knows about all the desktop files, we can embed the plugin metadata in them. It's what we do with most plugin systems in KDE in fact, and we've ported some this way. It's not the case for all such as kparts.</p>

<p>Regarding pulling everything, I don't necessarily disagree, I already proposed it on <a href="https://phabricator.kde.org/D12172" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12172</a>. I do agree though that having some kind of control over what goes in can be useful, especially if we can include this information.</p>

<p>In case you are curious, after building kf5 and okular (we are using kde-build-metadata information to bring dependencies which include kf5umbrella) these are the contents of the plugins folder:</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);">/opt/kdeandroid-deps/plugins/kcm_webshortcuts.so
/opt/kdeandroid-deps/plugins/kauth/helper/kauth_helper_plugin.so
/opt/kdeandroid-deps/plugins/kcm_kio.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_comicbook.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_txt.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_ooo.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_dvi.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_fb.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_fax.so
/opt/kdeandroid-deps/plugins/okular/generators/okularGenerator_xps.so
/opt/kdeandroid-deps/plugins/krossmodulekdetranslation.so
/opt/kdeandroid-deps/plugins/krossmoduleforms.so
/opt/kdeandroid-deps/plugins/kf5/urifilters/localdomainurifilter.so
/opt/kdeandroid-deps/plugins/kf5/urifilters/kuriikwsfilter.so
/opt/kdeandroid-deps/plugins/kf5/urifilters/kurisearchfilter.so
/opt/kdeandroid-deps/plugins/kf5/urifilters/fixhosturifilter.so
/opt/kdeandroid-deps/plugins/kf5/urifilters/kshorturifilter.so
/opt/kdeandroid-deps/plugins/kf5/kirigami/org.kde.desktop.so
/opt/kdeandroid-deps/plugins/kf5/ktranscript.so
/opt/kdeandroid-deps/plugins/kf5/parts/katepart.so
/opt/kdeandroid-deps/plugins/kf5/kio/trash.so
/opt/kdeandroid-deps/plugins/kf5/kio/http.so
/opt/kdeandroid-deps/plugins/kf5/kio/ftp.so
/opt/kdeandroid-deps/plugins/kf5/kio/file.so
/opt/kdeandroid-deps/plugins/kf5/kio/remote.so
/opt/kdeandroid-deps/plugins/kf5/emoticonsthemes/kde.so
/opt/kdeandroid-deps/plugins/kf5/emoticonsthemes/xmpp.so
/opt/kdeandroid-deps/plugins/kf5/emoticonsthemes/adium.so
/opt/kdeandroid-deps/plugins/kf5/emoticonsthemes/pidgin.so
/opt/kdeandroid-deps/plugins/kf5/kded/kcookiejar.so
/opt/kdeandroid-deps/plugins/kf5/kded/remotenotifier.so
/opt/kdeandroid-deps/plugins/kf5/org.kde.kwindowsystem.platforms/KF5WindowSystemWaylandPlugin.so
/opt/kdeandroid-deps/plugins/kf5/kiod/kssld.so
/opt/kdeandroid-deps/plugins/kf5/kiod/kpasswdserver.so
/opt/kdeandroid-deps/plugins/kf5/kiod/kioexecd.so
/opt/kdeandroid-deps/plugins/kf5/KEmoticonsIntegrationPlugin.so
/opt/kdeandroid-deps/plugins/kf5/FrameworkIntegrationPlugin.so
/opt/kdeandroid-deps/plugins/kcm_trash.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_ras.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_psd.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_kra.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_xcf.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_eps.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_tga.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_pcx.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_ora.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_rgb.so
/opt/kdeandroid-deps/plugins/imageformats/kimg_pic.so
/opt/kdeandroid-deps/plugins/krossqts.so
/opt/kdeandroid-deps/plugins/script/krossqtsplugin.so
/opt/kdeandroid-deps/plugins/okularpart.so</pre></div>

<p>So yes, it could be an over-kill. Maybe not.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R306 KParts</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12251">https://phabricator.kde.org/D12251</a></div></div><br /><div><strong>To: </strong>apol, Frameworks, vkrause, aacid<br /><strong>Cc: </strong>michaelh, bruns<br /></div>