D15943: [balooctl] Monitor also for state changes

2018-10-08 Thread Igor Poboiko
poboiko closed this revision. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D15943 To: poboiko, #frameworks, #baloo, ngraham Cc: ngraham, bruns, kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, abrahams

D15943: [balooctl] Monitor also for state changes

2018-10-06 Thread Stefan BrĂ¼ns
bruns added a comment. In D15943#337522 , @poboiko wrote: > In D15943#337385 , @ngraham wrote: > > > Works for me and the code looks sane. Looks like this fixes

D15943: [balooctl] Monitor also for state changes

2018-10-06 Thread Igor Poboiko
poboiko edited the summary of this revision. REPOSITORY R293 Baloo BRANCH monitor-state (branched from master) REVISION DETAIL https://phabricator.kde.org/D15943 To: poboiko, #frameworks, #baloo, ngraham Cc: ngraham, bruns, kde-frameworks-devel, ashaposhnikov, michaelh, astippich,

D15943: [balooctl] Monitor also for state changes

2018-10-06 Thread Igor Poboiko
poboiko added a comment. In D15943#337385 , @ngraham wrote: > Works for me and the code looks sane. Looks like this fixes https://bugs.kde.org/show_bug.cgi?id=364858, so can you add `BUG: 364858` and `FIXED-IN: 5.52` to the summary section and

D15943: [balooctl] Monitor also for state changes

2018-10-05 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Works for me and the code looks sane. Looks like this fixes https://bugs.kde.org/show_bug.cgi?id=364858, so can you add `BUG: 364858` and `FIXED-IN: 5.52` to the summary section and also

D15943: [balooctl] Monitor also for state changes

2018-10-05 Thread Igor Poboiko
poboiko retitled this revision from "[balooctl] Monitor also for index changes" to "[balooctl] Monitor also for state changes". poboiko edited the summary of this revision. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D15943 To: poboiko, #frameworks, #baloo Cc: bruns,