<blockquote>
<p>It seems a bad idea to apply a hacky patch to Geany where it affects all users, especially just before a release.</p>
</blockquote>
<p>The problem I see is, that it will always be seen as a hacky patch if only some people who tryout the scope plugin manually apply the patch to their private versions/installations of geany and no one else. Possible side effects will not be found this way I guess.</p>
<p>What do you think about the following:</p>
<ul>
<li>for 1.33 disable scope as suggested by <a href="https://github.com/elextr" class="user-mention">@elextr</a></li>
<li>after geany 1.33 and geany-plugins 1.33 have been released, apply the workaround patch in geany master (if <a href="https://github.com/b4n" class="user-mention">@b4n</a> agrees)</li>
<li>I could take maintenance of the scope plugin. I can't promise much improvements because I am also working on other plugins and projects but I used it a bit from time to time and like it more than the debugger plugin</li>
</ul>
<p>That way the patch would at least be tested by all developers in every day use. If we find any side-effects we can study them and decide to fix them or to remove the patch.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<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-plugins/issues/688#issuecomment-362892492">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJyAurO3WZpq1xZJqVeZ3Sv77UybCks5tRXUHgaJpZM4R4Y_y">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ8ZfMol1kXwBt89dZ8OEjKsG8kRIks5tRXUHgaJpZM4R4Y_y.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-plugins/issues/688#issuecomment-362892492"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue 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-plugins","title":"geany/geany-plugins","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-plugins"}},"updates":{"snippets":[{"icon":"PERSON","message":"@LarsGit223 in #688: \u003e It seems a bad idea to apply a hacky patch to Geany where it affects all users, especially just before a release.\r\n\r\nThe problem I see is, that it will always be seen as a hacky patch if only some people who tryout the scope plugin manually apply the patch to their private versions/installations of geany and no one else. Possible side effects will not be found this way I guess.\r\n\r\nWhat do you think about the following:\r\n- for 1.33 disable scope as suggested by @elextr\r\n- after geany 1.33 and geany-plugins 1.33 have been released, apply the workaround patch in geany master (if @b4n agrees)\r\n- I could take maintenance of the scope plugin. I can't promise much improvements because I am also working on other plugins and projects but I used it a bit from time to time and like it more than the debugger plugin\r\n\r\nThat way the patch would at least be tested by all developers in every day use. If we find any side-effects we can study them and decide to fix them or to remove the patch."}],"action":{"name":"View Issue","url":"https://github.com/geany/geany-plugins/issues/688#issuecomment-362892492"}}}</script>