<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace;font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On 7 April 2017 at 13:31, Adam Pigg <span dir="ltr"><<a href="mailto:adam@piggz.co.uk" target="_blank">adam@piggz.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><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></blockquote><div><br><div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">​Definitely Calligra master (and any upstream consumers) â€‹shall prepare to use out 3.1+ APIs ASAP. 3.0 will disappear from the wild and will not be maintained.<br><br></div><div style="font-family:monospace,monospace;font-size:small;display:inline" class="gmail_default">Note for Dag, this is not going to be a habit, it's one time situation before we can enter into API stability phase...<br><br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><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" target="_blank">danders@get2net.dk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
Adam Pigg skrev den 2017-04-07 13:21:<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> The code is probably copied from kdb!<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> I think this is saying that kreport will only load plugins which claim<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> to be the same version, so nothing would prevent an out of source<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> plugin, but its version number must match for it to be loaded, this<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> way multiple versions can coexist, and only load the appropriate<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> plugins.<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
Yes, but what is an appropriate plugin?<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
Imho releasing a new kreport version does not neccessarily invalidate<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
plugins.<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
I cannot see that a plugin developed for 3.0 could not be usable with<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
3.1 if the plugin interface is not changed, or am I missing something?<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> On Fri, 7 Apr 2017, 12:12 Dag, <<a href="mailto:danders@get2net.dk" class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg" target="_blank">danders@get2net.dk</a>> wrote:<span class="m_2995685517224970718gmail-"><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> Adam Pigg skrev den 2017-04-07 11:37:<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>>> I can try a clean build later, but it certainly should work!<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg"></span>
>> ok.<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> While we are looking into kreport, I have a question about kreport<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> plugins.<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> I find this code in KReportPluginManager.cpp:<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> const QString expectedVersion = QString::fromLatin1("%1.%2")<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
> .arg(KREPORT_STABLE_VERSION_MA<wbr>JOR).arg(KREPORT_STABLE_VERSIO<wbr>N_MINOR);<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> foreach(QPluginLoader *loader, offers) {<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> //QJsonObject json = loader->metaData();<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> //kreportDebug() << json;<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> //! @todo check version<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> QScopedPointer<KReportPluginEn<wbr>try> entry(new<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> KReportPluginEntry);<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> entry->setMetaData(loader);<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> const KReportPluginMetaData *metaData = entry->metaData();<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> if (metaData->version() != expectedVersion) {<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> I thought the metaData->version() should be some sort of plugin<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> interface version, not the version of KReport itself?<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> Afaics with current solution development of plugins outside of<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> kreport<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> is not possible.<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> Have I misunderstood it completely, or maybe it is intentional?<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> ---<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> Cheers,<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>><br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
>> Dag<br class="m_2995685517224970718gmail-m_-8422622247448026252gmail_msg">
</blockquote></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="m_2995685517224970718gmail_signature">regards, Jaroslaw Staniek<br><br>KDE:<br>: A world-wide network of software engineers, artists, writers, translators<br>: and facilitators committed to Free Software development - <a href="http://kde.org" target="_blank">http://kde.org</a><br>Calligra Suite:<br>: A graphic art and office suite - <a href="http://calligra.org" target="_blank">http://calligra.org</a><br>Kexi:<br>: A visual database apps builder - <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a><br>Qt Certified Specialist:<br>: <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/<wbr>jstaniek</a></div>
</div></div>