[Kde-pim] Review Request 119825: Added DB Indexer and Custom DB for Events and Posts

Laurent Montel montel at kde.org
Thu Aug 21 09:22:22 BST 2014



> On août 20, 2014, 12:11 après-midi, Laurent Montel wrote:
> > libkfapi was ported to kf5 ?
> 
> Martin Klapetek wrote:
>     No, not yet, this is supposed to be for the kde4 version of things, but given the ongoing transition, I guess it will just have to wait until all things are ready :S
> 
> Laurent Montel wrote:
>     To avoid to lose work. We can commit in kdepim-runtime. When libkfapi is ported we can adapt this code.
>     For the moment I didn't port this resources so it will not compile as previous :)
> 
> Nilesh Suthar wrote:
>     Can you guide me to with some tips to port libkfbapi, if it is the time to port now? 
>     Without porting libkfbapi how can we test this code?

Without porting libkfapi you can't test indeed.
For me you can commit you work, and after start to port libkfbapi. I don't know this lib so don't know what we need to port but I saw that it uses qjson so it needs a lot of work.
You can look at kdepimlibs master in each subdirectory how to adapt cmakelists.txt.
After that you need  to port to new qt5 json class.


- Laurent


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119825/#review64907
-----------------------------------------------------------


On août 20, 2014, 5:57 après-midi, Nilesh Suthar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119825/
> -----------------------------------------------------------
> 
> (Updated août 20, 2014, 5:57 après-midi)
> 
> 
> Review request for KDEPIM and Martin Klapetek.
> 
> 
> Repository: kdepim-runtime
> 
> 
> Description
> -------
> 
> Added Custom Database need by kpeople's addressbook
> 
> 
> Diffs
> -----
> 
>   resources/facebook/facebookresource_posts.cpp d2908d2 
>   resources/facebook/CMakeLists.txt 5e076a0 
>   resources/facebook/dbindexer.h PRE-CREATION 
>   resources/facebook/dbindexer.cpp PRE-CREATION 
>   resources/facebook/facebookresource.h df69a0e 
>   resources/facebook/facebookresource_events.cpp eacff64 
>   resources/facebook/facebookresource_friends.cpp a8c5f95 
> 
> Diff: https://git.reviewboard.kde.org/r/119825/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nilesh Suthar
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list