[Kde-pim] Review Request 123484: Re-imported modified collectionsearchjob from zanshin to be shared.

[Laurent Montel]

> On avr. 24, 2015, 1:14 après-midi, Laurent Montel wrote:
> > Why don't move it directly in akonadi as you want to share it ?
> > It's in zanshin too it's better to move it in akonadi no ?
> 
> Sandro Knauß wrote:
>     Yes, the best would be to get to akonadi-search or any other repo, but
>     
>     the problem is l30 in personsearchjob.cpp: include <libkdepim/ldap/ldapclientsearch.h>
>     
>     so it needs libkdepim for that.
>     
>     So for the moment, i see the only solution in copying.

Ah yes indeed ldapclient...
too bad.
Ok I understand.


- Laurent


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


On avr. 27, 2015, 3:56 après-midi, Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123484/
> -----------------------------------------------------------
> 
> (Updated avr. 27, 2015, 3:56 après-midi)
> 
> 
> Review request for KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> (cherry picked from commit 30e818f9f6eddb3822f62f1aafabbd39029f1a51)
> 
> Re-importet modified personsearchjob from zanshin.
> 
> (cherry picked from commit 720ea7a504dbf097312fb75fa67313ee321bf15d)
> 
> Use personsearchjob and collectionsearchjob from libkdepim.
> 
> (cherry picked from commit c382227777ac9c8bfa18e3787434dc260a2be862)
> 
> move to frameworks
> 
> 
> Diffs
> -----
> 
>   libkdepim/CMakeLists.txt 271dbd00c5a8b0fd811268c6b1d73f7d9918e374 
>   libkdepim/job/collectionsearchjob.h PRE-CREATION 
>   libkdepim/job/collectionsearchjob.cpp PRE-CREATION 
>   libkdepim/job/person.h PRE-CREATION 
>   libkdepim/job/personsearchjob.h PRE-CREATION 
>   libkdepim/job/personsearchjob.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/123484/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sandro Knauß
> 
>

_______________________________________________
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/