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

[Sandro Knauß]

> On April 24, 2015, 1:14 nachm., 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 ?

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.


- Sandro


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


On April 24, 2015, 11:09 vorm., Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123484/
> -----------------------------------------------------------
> 
> (Updated April 24, 2015, 11:09 vorm.)
> 
> 
> 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/