Re: Review Request 128957: Make KF5Baloo optional

2016-11-04 Thread Andreas Sturmlechner

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

(Updated Nov. 4, 2016, 9:01 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma and David Edmundson.


Changes
---

Submitted with commit 2b7258fdd86bff594799a154f651ce396c4e69a8 by Andreas 
Sturmlechner to branch master.


Repository: plasma-desktop


Description
---

https://mail.kde.org/pipermail/kde-frameworks-devel/2016-September/037734.html

Regardless of the current state of Baloo, it is not very deeply tied into 
Plasma. Usage in plasma-desktop comes down to providing the file search kcm.


Diffs
-

  CMakeLists.txt cf2e79a8eaf12c944fea845ff688840a561abf6b 
  kcms/CMakeLists.txt 53ca49a4d17a05c3d8d9a6a4c2e37a6ec0fd9284 

Diff: https://git.reviewboard.kde.org/r/128957/diff/


Testing
---

Several days of Plasma-5 without any issues. Usage of krunner without any 
segfaults.


Thanks,

Andreas Sturmlechner



Re: Review Request 128957: Make KF5Baloo optional

2016-11-04 Thread David Edmundson

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


Ship it!




Ship It!

- David Edmundson


On Oct. 20, 2016, 8:07 p.m., Andreas Sturmlechner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128957/
> ---
> 
> (Updated Oct. 20, 2016, 8:07 p.m.)
> 
> 
> Review request for Plasma and David Edmundson.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> https://mail.kde.org/pipermail/kde-frameworks-devel/2016-September/037734.html
> 
> Regardless of the current state of Baloo, it is not very deeply tied into 
> Plasma. Usage in plasma-desktop comes down to providing the file search kcm.
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt cf2e79a8eaf12c944fea845ff688840a561abf6b 
>   kcms/CMakeLists.txt 53ca49a4d17a05c3d8d9a6a4c2e37a6ec0fd9284 
> 
> Diff: https://git.reviewboard.kde.org/r/128957/diff/
> 
> 
> Testing
> ---
> 
> Several days of Plasma-5 without any issues. Usage of krunner without any 
> segfaults.
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>



Re: Review Request 128957: Make KF5Baloo optional

2016-10-20 Thread Andreas Sturmlechner

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

(Updated Oct. 20, 2016, 8:07 p.m.)


Review request for Plasma and David Edmundson.


Changes
---

rebased on top of latest changes, changed from OPTIONAL to RECOMMENDED 
dependency.


Repository: plasma-desktop


Description
---

https://mail.kde.org/pipermail/kde-frameworks-devel/2016-September/037734.html

Regardless of the current state of Baloo, it is not very deeply tied into 
Plasma. Usage in plasma-desktop comes down to providing the file search kcm.


Diffs (updated)
-

  CMakeLists.txt cf2e79a8eaf12c944fea845ff688840a561abf6b 
  kcms/CMakeLists.txt 53ca49a4d17a05c3d8d9a6a4c2e37a6ec0fd9284 

Diff: https://git.reviewboard.kde.org/r/128957/diff/


Testing
---

Several days of Plasma-5 without any issues. Usage of krunner without any 
segfaults.


Thanks,

Andreas Sturmlechner



Review Request 128957: Make KF5Baloo optional

2016-09-20 Thread Andreas Sturmlechner

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

Review request for Plasma.


Repository: plasma-desktop


Description
---

https://mail.kde.org/pipermail/kde-frameworks-devel/2016-September/037734.html

Regardless of the current state of Baloo, it is not very deeply tied into 
Plasma. Usage in plasma-desktop comes down to providing the file search kcm.


Diffs
-

  CMakeLists.txt cf2e79a8eaf12c944fea845ff688840a561abf6b 
  kcms/CMakeLists.txt 53ca49a4d17a05c3d8d9a6a4c2e37a6ec0fd9284 

Diff: https://git.reviewboard.kde.org/r/128957/diff/


Testing
---

Several days of Plasma-5 without any issues. Usage of krunner without any 
segfaults.


Thanks,

Andreas Sturmlechner