D27917: [Baloo] Fix firstRun and config change logic

2020-03-11 Thread Stefan Brüns 
This revision was automatically updated to reflect the committed changes.
Closed by commit R119:550cf9cafb19: [Baloo] Fix firstRun and config change 
logic (authored by bruns).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27917?vs=77183=77456

REVISION DETAIL
  https://phabricator.kde.org/D27917

AFFECTED FILES
  kcms/baloo/kcm.cpp
  kcms/baloo/kcm.h

To: bruns, #baloo, ngraham, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27917: [Baloo] Fix firstRun and config change logic

2020-03-07 Thread Stefan Brüns 
bruns created this revision.
bruns added reviewers: Baloo, ngraham.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
bruns requested review of this revision.

REVISION SUMMARY
  FirstRun should only be set if baloo was *never ever* run before, and
  this case is already handled by baloo internally.
  
  Do no try to do a baloo config refresh if it has just been quit.
  
  Also avoid triggering a config update immediately after starting baloo,
  it will do another walk of the directory tree.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27917

AFFECTED FILES
  kcms/baloo/kcm.cpp
  kcms/baloo/kcm.h

To: bruns, #baloo, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart