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
