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