<p></p>
<blockquote>
<p dir="auto">master is a development branch, nothing that has any guarantees for stability. Unless the bug remains unfixed for a long time its IMO ok to work on fixing it without hysterically reverting changes. Anyone can checkout the commit before the crash is introduced locally and work from that.</p>
</blockquote>
<p dir="auto">The problem with this is it is not just a problem with Geany, but plugins too, and those not in G-P could take an unknown amount of time to be fixed.</p>
<p dir="auto">Geany is too scared of reverting, as I said its not a poor reflection on the person who merged the change, its just pragmatic.</p>
<blockquote>
<p dir="auto">On our side we should definitely increment the plugin API so that plugins can adapt to the behavior (or reject running on older Geany).</p>
</blockquote>
<blockquote>
<blockquote>
<p dir="auto">Of course, then we have to deal with it, no question about that.</p>
</blockquote>
</blockquote>
<p dir="auto">The problem is that incrementing the API does nothing to stop plugins that have not been modified from running.  Geany only tests its API is greater than that the plugin asks for.</p>
<blockquote>
<p dir="auto">Incrementing the ABI, on the other hand, does not make sense. It cannot fix the plugin code. All it does is giving the false idea that recompiling the affected plugin resolves the situation.</p>
</blockquote>
<p dir="auto">Yes, agreed, but at least it stops plugins that have not been recompiled from running and possibly crashing.</p>
<p dir="auto">Like I said above currently we can't stop an unfixed plugin from being run, so if Geany is going to start making changes to the plugin interface with non-backward compatible crashing changes we need to introduce a test to reject plugins that say they need an API <em>less than</em> the new number.  At least that will require the plugin author to manually change something, and might prompt them to look why its needed.</p>
<p dir="auto">Have reopened since <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1099651679" data-permission-text="Title is private" data-url="https://github.com/geany/geany/issues/3098" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/3098/hovercard" href="https://github.com/geany/geany/pull/3098">#3098</a> (thanks <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> for the quick turnaround) does not address the plugin issue.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/geany/geany/issues/3095#issuecomment-1010443094">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ357OTCZN7TMJC4CVDUVSZC5ANCNFSM5LV3I2KA">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAIOWJ2OT7W33UEKVBLSGLLUVSZC5A5CNFSM5LV3I2KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHQ5CGVQ.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><geany/geany/issues/3095/1010443094</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/issues/3095#issuecomment-1010443094",
"url": "https://github.com/geany/geany/issues/3095#issuecomment-1010443094",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>