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


Thanks for this, but would you mind rebasing your patch with this - 
https://git.reviewboard.kde.org/r/118670/


- Vishesh Handa


On June 14, 2014, 10:53 p.m., Matthew Dawson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118752/
> -----------------------------------------------------------
> 
> (Updated June 14, 2014, 10:53 p.m.)
> 
> 
> Review request for Baloo.
> 
> 
> Repository: baloo
> 
> 
> Description
> -------
> 
> Fix Baloo to work against frameworkified KFileMetaData.
> 
> Change Baloo to find KFileMetaData as a proper framework, and change the
> necessary include statements.
> 
> 
> Diffs
> -----
> 
>   src/file/lib/file.h 89c03e2c3ddcc7215ec8451caeeb6323b8eeed9b 
>   src/file/lib/autotests/filefetchjobtest.cpp 
> f9c49ff30ee119950603f1b9628c14fffe0bd5dd 
>   src/file/extractor/result.cpp b2d5615ce3645d6e0ff435b3771539e1aeea5380 
>   src/file/extractor/result.h 3dbde3d333076543f141859b074da47d1ed593c2 
>   src/file/extractor/app.cpp 00c250e5796c58b0fb1a7b7f8ba7849df557f56e 
>   src/file/extractor/app.h 9e674d369fb25538ed971dce9026fe53d697db3e 
>   src/file/basicindexingjob.cpp b53cd79ccded7ce901cfd80c1eea119c6f6ab121 
>   src/file/basicindexingjob.h f72680c499de5f6d48a1b2bdedceb1877101e3b2 
>   CMakeLists.txt 7a07aa465601d4d196e9fa9158c05d1573d0f90d 
>   src/file/lib/file_p.h a4fcf44325a360149fa646e8b8adb15c68a6b20e 
>   src/file/search/CMakeLists.txt 5549943aaf98026a698b4a4ae6bba4df01699012 
>   src/file/search/filesearchstore.cpp 
> 0b1a6201ed7ae12f776daf2392313406d0168de3 
>   src/tools/balooshow/main.cpp db0acb019c9bda696775150ccc4f6007fec7be82 
> 
> Diff: https://git.reviewboard.kde.org/r/118752/diff/
> 
> 
> Testing
> -------
> 
> Compiles and tests fine.
> 
> 
> Thanks,
> 
> Matthew Dawson
> 
>

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

Reply via email to