ngraham created this revision. ngraham added reviewers: Baloo, bruns. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. ngraham requested review of this revision.
REVISION SUMMARY Currently, if you add a folder to Baloo's `includeFolders` or `excludeFolders` lists, it cannot be removed again if it has been deleted. This patch fixes that for both `balooctl` and also the KCM. TEST PLAN mkdir ~/foobar-test balooctl config add excludeFolders ~/foobar-test rmdir ~/foobar-test balooctl config remove excludeFolders ~/foobar-test Without the patch, `balooctl` exits with an error, reports "Path does not exist", and does not remove the entry. With the patch, `balooctl` exits cleanly with no error and removes the path Works from the KCM, too. REPOSITORY R293 Baloo BRANCH fix-removing-locations-from-include-and-exclude-lists (branched from master) REVISION DETAIL https://phabricator.kde.org/D15844 AFFECTED FILES src/tools/balooctl/configcommand.cpp To: ngraham, #baloo, bruns Cc: kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams