<div dir="ltr">Yeah, you are right. Perhaps for 3.1 we should fix this as we are then supposed to be offering abi stability.</div><span>
</span><p dir="ltr">Thoughts jaroslaw?<br>
</p><span>
</span><br><div class="gmail_quote"><div dir="ltr">On Fri, 7 Apr 2017, 12:28 Dag, <<a href="mailto:danders@get2net.dk">danders@get2net.dk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br class="gmail_msg">
<br class="gmail_msg">
Adam Pigg skrev den 2017-04-07 13:21:<br class="gmail_msg">
> The code is probably copied from kdb!<br class="gmail_msg">
><br class="gmail_msg">
> I think this is saying that kreport will only load plugins which claim<br class="gmail_msg">
> to be the same version, so nothing would prevent an out of source<br class="gmail_msg">
> plugin, but its version number must match for it to be loaded, this<br class="gmail_msg">
> way multiple versions can coexist, and only load the appropriate<br class="gmail_msg">
> plugins.<br class="gmail_msg">
Yes, but what is an appropriate plugin?<br class="gmail_msg">
Imho releasing a new kreport version does not neccessarily invalidate<br class="gmail_msg">
plugins.<br class="gmail_msg">
I cannot see that a plugin developed for 3.0 could not be usable with<br class="gmail_msg">
3.1 if the plugin interface is not changed, or am I missing something?<br class="gmail_msg">
<br class="gmail_msg">
><br class="gmail_msg">
> On Fri, 7 Apr 2017, 12:12 Dag, <<a href="mailto:danders@get2net.dk" class="gmail_msg" target="_blank">danders@get2net.dk</a>> wrote:<br class="gmail_msg">
><br class="gmail_msg">
>> Adam Pigg skrev den 2017-04-07 11:37:<br class="gmail_msg">
>>> I can try a clean build later, but it certainly should work!<br class="gmail_msg">
>> ok.<br class="gmail_msg">
>><br class="gmail_msg">
>> While we are looking into kreport, I have a question about kreport<br class="gmail_msg">
>> plugins.<br class="gmail_msg">
>> I find this code in KReportPluginManager.cpp:<br class="gmail_msg">
>> const QString expectedVersion = QString::fromLatin1("%1.%2")<br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
> .arg(KREPORT_STABLE_VERSION_MAJOR).arg(KREPORT_STABLE_VERSION_MINOR);<br class="gmail_msg">
>> foreach(QPluginLoader *loader, offers) {<br class="gmail_msg">
>> //QJsonObject json = loader->metaData();<br class="gmail_msg">
>> //kreportDebug() << json;<br class="gmail_msg">
>> //! @todo check version<br class="gmail_msg">
>> QScopedPointer<KReportPluginEntry> entry(new<br class="gmail_msg">
>> KReportPluginEntry);<br class="gmail_msg">
>> entry->setMetaData(loader);<br class="gmail_msg">
>> const KReportPluginMetaData *metaData = entry->metaData();<br class="gmail_msg">
>> if (metaData->version() != expectedVersion) {<br class="gmail_msg">
>><br class="gmail_msg">
>> I thought the metaData->version() should be some sort of plugin<br class="gmail_msg">
>> interface version, not the version of KReport itself?<br class="gmail_msg">
>> Afaics with current solution development of plugins outside of<br class="gmail_msg">
>> kreport<br class="gmail_msg">
>> is not possible.<br class="gmail_msg">
>> Have I misunderstood it completely, or maybe it is intentional?<br class="gmail_msg">
>><br class="gmail_msg">
>> ---<br class="gmail_msg">
>> Cheers,<br class="gmail_msg">
>><br class="gmail_msg">
>> Dag<br class="gmail_msg">
</blockquote></div>