<table><tr><td style="">vkrause 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/D27525">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D27525#614905" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27525#614905</a>, <a href="https://phabricator.kde.org/p/apol/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@apol</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Assets (QML files and icons) are no longer extracted into the file system,<br />
 but now are available in the android_rcc_bundle QRC file.</p></blockquote>

<p>Meaning that we now need to change all applications to put the resources on the rcc file?</p></div>
</blockquote>

<p>No, androiddeployqt does that on its own. So overall the apk still has exactly the same content, just the way to access it changed. It is possible apps make assumption about this, some of the frameworks definitely do.</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>The change upstream seems bonkers to me. Whatever, let's get this in. We'll possibly need to patch quite many apps for this... :(</p>

<p>Also feels quite weird how the commit message is twice the same text but it never says why it's done. <a href="https://codereview.qt-project.org/c/qt/qtbase/+/270573/24" class="remarkup-link" target="_blank" rel="noreferrer">https://codereview.qt-project.org/c/qt/qtbase/+/270573/24</a></p>

<p>¯\_(ツ)_/¯</p></blockquote>

<p>It's avoiding duplicating the assets again in the file system, thus needing less space and significantly speeding up the first start. So while technically going in the right direction I would certainly have preferred to have this as something opt-in to not break all existing non-trivial code out there...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27525">https://phabricator.kde.org/D27525</a></div></div><br /><div><strong>To: </strong>vkrause<br /><strong>Cc: </strong>apol, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, ahiemstra, davidedmundson, mart, hein<br /></div>