D26496: Log categories: fix format adding DEFAULT_SEVERITY
meven abandoned this revision. meven added a comment. Thanks @mlaurent I just have old files installed on my system and got confused, I thought DEFAULT_SEVERITY was mandatory. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D26496 To: meven, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26496: Log categories: fix format adding DEFAULT_SEVERITY
mlaurent requested changes to this revision. mlaurent added a comment. This revision now requires changes to proceed. But you're wrong "log_kdebugsettings: In this file: "/home/meven/kde/usr/etc/xdg/kio.categories" this line "kf5.kio.cookiejar kcookiejar (KIO)" still use old format. We need to port it" it not for missing DEFAULT_SEVERITY argument it's just that kio.categories doesnt use new format. but it seems that it's an old version as here I don't see this warning for kio.categories. And you add DEFAULT_SEVERITY [WARNING] but ecm_qt_declare_logging_category doesn"t use WARNING as default argument for example: "ecm_qt_declare_logging_category(kio_trash_PART_DEBUG_SRCS HEADER kiotrashdebug.h IDENTIFIER KIO_TRASH CATEGORY_NAME kf5.kio.trash)" so this patch is not correct for sure. INLINE COMMENTS > kio.categories:17 > +kf5.kio.widgets KIOWidgets (KIO) DEFAULT_SEVERITY [WARNING] IDENTIFIER > [KIO_WIDGETS] > +org.kde.kio.kpasswdserver KPasswdServer (KIO) IDEFAULT_SEVERITY [WARNING] > DENTIFIER [category] > org.kde.kurifilter-localdomain KUriFilter Local Domain (KIO) > DEFAULT_SEVERITY [WARNING] IDENTIFIER [category] It's not correct when you paste DEFAULT_SEVERITY [WARNING] => you add "I" from identifier REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D26496 To: meven, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26496: Log categories: fix format adding DEFAULT_SEVERITY
meven created this revision. meven added reviewers: Frameworks, mlaurent. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY Prevents warning when launching kdebugsettings such as: log_kdebugsettings: In this file: "/home/meven/kde/usr/etc/xdg/kio.categories" this line "kf5.kio.cookiejar kcookiejar (KIO)" still use old format. We need to port it REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D26496 AFFECTED FILES kio.categories To: meven, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns