<div dir="ltr">This is my response from private mail with Veaceslav :<div><br></div><div><div><font face="arial, sans-serif">I think you don't need to back-port you patch in libkexiv2 from KDE 4.10 branch... In fact you cannot back-port your patch and KDE 4.10 must preserve binary compatibility. Your patch break BC due to new methods add in library</font><br>

</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Don't forget that libkexiv2 is hosted in kdegraphics, which is a KDE component, released officially with KDE. Sure digiKam, because it's don't release at the same KDE time, it include last libkexiv2 code, but packagers can disable to use this library from digiKam tarball and use official libkexiv2 from KDE instead.</div>

<div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">This is the same workflow from libkipi and libkdcraw.</div><div style="font-family:arial,sans-serif;font-size:13px">

<br></div><div style="font-family:arial,sans-serif;font-size:13px">Currently, your patch from libkexiv2 will be included with KDE 4.11. To be sure that digiKam can be compiled with libkexiv2 from KDE 4.10, you need to wrap your code from Digikam::DMetadata class into conditional pre-processor rules, checking libkexiv2 version ID shared with libkexiv2/version.h. It's very easy to do...</div>

</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Gilles</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/3/4 Veaceslav Munteanu <span dir="ltr"><<a href="mailto:slavuttici@gmail.com" target="_blank">slavuttici@gmail.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sun, Mar 3, 2013 at 10:49 PM, Marcel Wiesweg <<a href="mailto:marcel.wiesweg@gmx.de">marcel.wiesweg@gmx.de</a>> wrote:<br>


><br>
><br>
>> Please use the private libs, new version of libkexiv2 is comming with<br>
>> KDE 4.10.2 and we operated few changes on this lib.<br>
><br>
> Did you really backport this change to the 4.10 branch? I'm not sure this is<br>
> really appropriate, it's a new feature, not a fix.<br>
<br>
</div>Marcel, I don't understand...<br>
<br>
I changes in libkexiv2 are needed for writting picassa faces and as<br>
Gilles suggested I increased API/ABI count of the libkexiv.<br>
Also I began to commit some pieces of code that is tested and work,<br>
because implementing rotations, I touch to much files so I will get a<br>
too big to review patch...If you want I can roll back the changes that<br>
won't make digikam not to compile with shared libkexiv...<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Veaceslav<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
</div></div></blockquote></div><br></div>