D21674: [balooctl] Streamline indexer control commands

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sat Jun 8 18:07:49 BST 2019


bruns created this revision.
bruns added reviewers: Baloo, ngraham, astippich, poboiko.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.

REVISION SUMMARY
  Currently, there are 7 commands which start/stop the indexer in some way:
  
  - stop/start/restart: stops or/and starts the indexer process, effective for the current session (on next login, the indexer is started when enabled).
  - suspend/resume: the indexer process keeps running, but does not process any file events (files added/delete/modified ...).
  - disable/enable: sets the enable state in the config, stops the indexer process and deletes the database. On enable (even when already enabled) forces a full reindex via the 'firstRun' flag. The deletion of the DB file was done silently.
  
  The large number of similar commands is confusing by itself, and more
  so its side effects.
  
  The restart command was also racy, as it tried to start baloo_file even
  when the old instance still held the DBus interface, then the new instance
  would directly exit.
  
  This removes the stop/start/restart command alltogether, and removes the
  deletion of the DB from the disable command. Enable/disable still
  starts/stops the 'baloo_file' process.
  
  To allow deletion of the DB via CLI, a new "purge" command is added. In
  case the indexer was enabled/running, it is kept enabled.

REPOSITORY
  R293 Baloo

BRANCH
  balooctl_start_stop

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

AFFECTED FILES
  src/tools/balooctl/main.cpp

To: bruns, #baloo, ngraham, astippich, poboiko
Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190608/201d858d/attachment.html>


More information about the Kde-frameworks-devel mailing list