----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/ -----------------------------------------------------------
(Updated Oct. 6, 2015, 9:42 p.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks, Albert Astals Cid and David Faure. Changes ------- Submitted with commit 0a39ba6ec01600efeb9e334fa457dd725e2e3749 by David Faure on behalf of Xuetian Weng to branch master. Repository: kservice Description ------- In 0e4d7247e27f82a9b79b19dbceb35b603baceb76, a new feature saving directory mtime in ksycoca db is introduced. In the related code, TimestampChecker() will check the mtime recursively, but the building process will only read the top level directories' mtime, which causes ksycoca db being recreated again and again. I hit this because my ~/.config/menus/applications-merged mtime newer than ~/.config/menus. And plasmashell is running with 100% since it's creating ksycoca db repeatedly. This patch change the building db procedure to use the same logic as in TimestampChecker(), so building and checking are consistent now. Diffs ----- autotests/ksycocatest.cpp 67f0700 src/sycoca/kbuildsycoca.cpp 00aaac3 src/sycoca/ksycoca.cpp 5209389 src/sycoca/ksycocautils_p.h c14057b Diff: https://git.reviewboard.kde.org/r/125537/diff/ Testing ------- Now a sub directory with larger timestamp than parent will not cause ksycoca data being recreated again and again. Thanks, Xuetian Weng
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
