<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On 7 April 2013 06:31, Matthew Brush <span dir="ltr"><<a href="mailto:mbrush@codebrainz.ca" target="_blank">mbrush@codebrainz.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 14-04-06 05:04 AM, Dimitar Zhekov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
[...]<div class="im"><br>
Anyway, I was too categorical. The old hack of #defining GTK_COMPAT_H<br>
before including geanyplugin.h, and manually writing any compatibility<br>
code, will probably work fine with all Geany versions. That'll look a<br>
bit ugly, and IMHO, letting the plugins decide whether to include<br>
"gtkcompat.h" would be better...<br>
<br>
</div></blockquote>
<br>
I agree, gtkcompat.h should be optional, to be included explicitly only if a plugin needs/wants it. It does seem wrong to prevent plugins from targeting older versions of Geany from when this header didn't exist or forcing their min GTK+ to 2.24.<br>
</blockquote><div><br></div><div style>If it does then that is indeed a bug in gtkcompat.h since Geany still supports 2.16.  IIUC it defines stuff for versions that need it so the compat functions work on both old and new versions.  It certainly creates things for < 2.18 and < 2.20.</div>
<div style><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If we keep gtkcompat.h forced into all plugins maybe we should document the trick of blocking it out by predefining the guard macro like that.<br></blockquote><div><br></div><div style>Essentially the plugin API has changed at 1.24, it no longer implicitly includes GTK2, it now implicitly includes GTK2 and GTK3 and the compat functions, and to compile against it plugins do indeed need to change.  Such a change was inevitable to be able to move Geany to GTK3 since it isn't compatible with GTK2.   gtkcompat.h is always available because Colomban wants plugins to support Geany running on GTK from 2.16 to 3.8. </div>
<div style><br></div><div style>I think this needs to be clearly described in the plugin docs and more clearly announced to the plugin maintainer community.  If they don't want to change their plugin then they can continue to only compile against older versions, and that of course leads to the discussion in the other thread :)</div>
<div><br></div><div style>Cheers</div><div style>Lex</div><div style><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Matthew Brush<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
______________________________<u></u>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org" target="_blank">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-<u></u>bin/mailman/listinfo/devel</a><br>
</div></div></blockquote></div><br></div></div>