[Kde-graphics-devel] libkmap and libkface moved to kdereview

[Andreas Pakulat]

On 25.11.10 08:40:05, Gilles Caulier wrote:
> 2010/11/24 Albert Astals Cid <aacid at kde.org>:
> > A Dimecres, 24 de novembre de 2010, Gilles Caulier va escriure:
> >> 2010/11/24 Albert Astals Cid <aacid at kde.org>:
> >> > A Dilluns, 22 de novembre de 2010, Gilles Caulier va escriure:
> >> >> Hi KDE teams,
> >> >>
> >> >> This summer, digiKam team has work on 2 new features : face detection
> >> >> and reverse geo-coding.
> >> >>
> >> >> These features have been implemented during Google Summer of code 2010.
> >> >> You can see more information at these pages :
> >> >>
> >> >> http://techbase.kde.org/Projects/Digikam/CodingSprint2010
> >> >>
> >> >> For Face detection we have implemented a shared library named "libkface"
> >> >>
> >> >> http://websvn.kde.org/trunk/kdereview/libkface/
> >> >>
> >> >> For Reverse-Geocoding we have implemented another shared library named
> >> >> "libkmap"
> >> >>
> >> >> http://websvn.kde.org/trunk/kdereview/libkmap/
> >> >>
> >> >> Both libraries have been hosted in a dedicated branch from KDE
> >> >> subversion repository. Following Albert Astals Cid tips from
> >> >> review-team mailing list, i moved this code to trunk/kdereview for 2
> >> >> weeks. Code is open for review by KDE developers.
> >> >>
> >> >> The plan is to include libkmap and libkface to kdegraphics/libs with
> >> >> KDE 4.7. These libraries are shared between digiKam/kipi-plugins, and
> >> >> of course with others part of KDE applications.
> >> >>
> >> >> As you can see, in digiKam and kipi-plugins 2.0 release plan targeted
> >> >> for may 2011, but this can be delayed if problem occurs :
> >> >>
> >> >> http://www.digikam.org/drupal/about/releaseplan
> >> >>
> >> >> Remember that digiKam team already maintain 3 shared libraries named
> >> >> libkipi, libkexiv2, and
> >> >> libkdcraw into kdegraphics/libs.
> >> >>
> >> >> if all is fine, somebody can plan to patch KDE 4.7 plan to include
> >> >> this libraries in TODO list ?
> >> >
> >> > libkmap seems to have a hard dependency on marble, in the past we didn't
> >> > want those, nor sure if that rule is still there. But if it is not, can
> >> > you turn that hard dependency into a soft one, please?
> >>
> >> digiKam already have an hard depency to marble, about geolocation
> >> feature, since 1.0.0 release. there is no problem with that...
> >
> > Well, yes it is, as it is now, moving it to kdegraphics will make kdegraphics
> > not compile if you don't have kdeedu installed, and that is totally
> > unacceptable.
> 
> Do you mean that it will be more logic to host libkmap with KDEEDU or
> better, into marble as well ?

No the point is that there must be no interdependencies between two
modules. So if two modules need a shared library it needs to go into
kdelibs (or kdepimlibs for pim-libraries) or it needs to be a completely
external dependencies outside of any of the KDE modules.

Andreas

-- 
Ships are safe in harbor, but they were never meant to stay there.