<table><tr><td style="">kossebau 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/D15565">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/D15565#333756" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15565#333756</a>, <a href="https://phabricator.kde.org/p/antonanikin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@antonanikin</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>Is there a chance those aspects could be split out into separate patches?<br />
 Also still hoping for a variant of the bug fix patch which does not need a string freeze break, if possible.</p></blockquote>

<p>Hi, Friedrich, sorry for delay. I think we should push it as is. Since current (wrong) logic of plugin controller we can have execute plugin unloaded with heaptrack loaded in same time. But execute plugin is really needed for plugin work so to don't break strings freeze we can only silently stop heaptrack analysis which is wrong and ugly. I think user should receive normal feedback.</p>

<p>So I suggest wait for 5.3 release and push this after. The patch will be available at 5.3.1 correcting release. Your opinion?</p></div>
</blockquote>

<p>String freeze holds for the full time in the release branch. What usually is done in such a case where a bug fix really needs to break string freeze, is to ask translators for an exception. Usually a formality, but done at least to show respect to translators and their work.</p>

<p>Cmp. e.g. <a href="https://marc.info/?l=kde-i18n-doc&m=151092571015821&w=2" class="remarkup-link" target="_blank" rel="noreferrer">https://marc.info/?l=kde-i18n-doc&m=151092571015821&w=2</a> . Once two or more representatives of language teams have okayed (and no-one objected), exception is granted (that is documented somewhere, but could not find that quickly).</p>

<p>So if you see no way around, ask the translators now by emailing such a request to kde-i18n-doc@kde.org Will see to start reviewing in parallel then this WE.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15565">https://phabricator.kde.org/D15565</a></div></div><br /><div><strong>To: </strong>antonanikin, KDevelop<br /><strong>Cc: </strong>kossebau, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>