D15565: Fix bug 389060 (Heaptrack analysis keeps firing /usr/bin/plasmoidviewer)

[Friedrich W. H. Kossebau]

kossebau added a comment.


  In D15565#333758 <https://phabricator.kde.org/D15565#333758>, @kossebau wrote:
  
  > In D15565#333756 <https://phabricator.kde.org/D15565#333756>, @antonanikin wrote:
  >
  > > > Is there a chance those aspects could be split out into separate patches?
  > > >  Also still hoping for a variant of the bug fix patch which does not need a string freeze break, if possible.
  > >
  > > 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.
  > >
  > > 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?
  >
  >
  > 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.
  >
  > Cmp. e.g. https://marc.info/?l=kde-i18n-doc&m=151092571015821&w=2 . 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).
  >
  > So if you see no way around, ask the translators now by emailing such a request to kde-i18n-doc at kde.org Will see to start reviewing in parallel then this WE.
  
  
  So to get this patch moving again I just sent an email asking for string freeze exception. Should be a formality, and hopefully in a day we might have green light from the translators at least. Will see to give the actual code some review as well in the upcoming week.

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D15565

To: antonanikin, #kdevelop
Cc: kfunk, kossebau, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20181028/cf3ae4d9/attachment-0001.html>