https://bugs.kde.org/show_bug.cgi?id=375370
Nate Graham <n...@kde.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |RESOLVED Latest Commit| |https://commits.kde.org/bal | |oo/8a0b8270dcd389db38cf3a4c | |20d65d46a9dda5d7 Resolution|--- |FIXED Version Fixed In| |5.51 --- Comment #13 from Nate Graham <n...@kde.org> --- Git commit 8a0b8270dcd389db38cf3a4c20d65d46a9dda5d7 by Nathaniel Graham. Committed on 29/09/2018 at 22:59. Pushed by ngraham into branch 'master'. Allow removing non-existent folders from include and exclude lists 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. FIXED-IN: 5.51 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. Reviewers: #baloo, bruns Reviewed By: #baloo, bruns Subscribers: kde-frameworks-devel Tags: #frameworks, #baloo Differential Revision: https://phabricator.kde.org/D15844 M +3 -24 src/tools/balooctl/configcommand.cpp https://commits.kde.org/baloo/8a0b8270dcd389db38cf3a4c20d65d46a9dda5d7 -- You are receiving this mail because: You are watching all bug changes.