D18829: [balooctl] Normalize include/exclude pathes before using it for the config

Stefan BrĂ¼ns noreply at phabricator.kde.org
Thu Feb 7 20:17:05 GMT 2019


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

REVISION SUMMARY
  A directory can be added multiple times to the config by just adding a
  trailing slash to it, as the config removes any trailing slashes before
  storing it, and the `contains(path)` will not match.
  
  Normalize the path to the same format as used by the config and use it
  for both adding and removing it.
  
  While using it for removal is not necessary, it allows to use both
  variants and matches the "add" behavior, where both variants have the
  same effect.

REPOSITORY
  R293 Baloo

BRANCH
  submit

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

AFFECTED FILES
  src/tools/balooctl/configcommand.cpp

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


More information about the Kde-frameworks-devel mailing list