Am 15.11.2017 um 15:40 schrieb elextr:<br>
&gt; @kugel- &lt;https://github.com/kugel-&gt; sadly I think we have to have a work <br>
&gt; around in Geany, even if there is a fix in Glib, there are at least 10 <br>
&gt; versions in the wild with the problem by @b4n &lt;https://github.com/b4n&gt;&#39;s <br>
&gt; post above, so either Geany becomes tolerant of those versions, or <br>
&gt; (IIUC) scope is unusable on systems with those versions (and more <br>
&gt; versions until its fixed).<br>
&gt; <br>
&gt; So long as polling does not kick in on anything other than things <br>
&gt; associated with scope plugin I guess its an ok fix, even if it does not <br>
&gt; return to non-polling. As for adding to this release, unless @b4n <br>
&gt; &lt;https://github.com/b4n&gt; is happy he has tested it enough I would wait, <br>
&gt; lets not have to have a 1.32.1 to fix the fix :)<br>
&gt; <br>
<br>
The other side of the medal is that Geany itself is not affected, just a <br>
plugin that the author itself considers orphaned/unmaintaned, so I&#39;m not <br>
sure a non-trivial change inside Geany is justified.<br>
<br>
FWIW, I&#39;m not against merging the change, I don&#39;t feel very strong. I&#39;m <br>
rather indifferent since I don&#39;t use scope (doesn&#39;t support gtk3).<br>
<br>
But if we accept it, then better for 1.32. This is sitting for a long <br>
time already and has been tested by at least two individuals already.<br>
<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/1461#issuecomment-344633224">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ8mXSje_bdQb-vxYhZ1zRYKyiOB5ks5s2wb7gaJpZM4M3KqU">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ7RZPCbyN6BICLS7Ibrzt5NNTd0dks5s2wb7gaJpZM4M3KqU.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1461#issuecomment-344633224"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@kugel- in #1461: Am 15.11.2017 um 15:40 schrieb elextr:\n\u003e @kugel- \u003chttps://github.com/kugel-\u003e sadly I think we have to have a work \n\u003e around in Geany, even if there is a fix in Glib, there are at least 10 \n\u003e versions in the wild with the problem by @b4n \u003chttps://github.com/b4n\u003e's \n\u003e post above, so either Geany becomes tolerant of those versions, or \n\u003e (IIUC) scope is unusable on systems with those versions (and more \n\u003e versions until its fixed).\n\u003e \n\u003e So long as polling does not kick in on anything other than things \n\u003e associated with scope plugin I guess its an ok fix, even if it does not \n\u003e return to non-polling. As for adding to this release, unless @b4n \n\u003e \u003chttps://github.com/b4n\u003e is happy he has tested it enough I would wait, \n\u003e lets not have to have a 1.32.1 to fix the fix :)\n\u003e \n\nThe other side of the medal is that Geany itself is not affected, just a \nplugin that the author itself considers orphaned/unmaintaned, so I'm not \nsure a non-trivial change inside Geany is justified.\n\nFWIW, I'm not against merging the change, I don't feel very strong. I'm \nrather indifferent since I don't use scope (doesn't support gtk3).\n\nBut if we accept it, then better for 1.32. This is sitting for a long \ntime already and has been tested by at least two individuals already.\n\n"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1461#issuecomment-344633224"}}}</script>