D19508: [kconf_update] move from custom logging tech to qCDebug

Harald Sitter noreply at phabricator.kde.org
Mon Mar 4 10:41:15 GMT 2019


sitter created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
sitter requested review of this revision.

REVISION SUMMARY
  the previous logging tech got kind of defunct in 2014 (to the point where
  it was basically qCDebug). seeing as no one really complained it seems
  reasonable to just move to qCDebug instead and make use of category filters
  and other qdebug goodness (such as system logging facilities for the
  various platforms)
  
  - new logging category kf5.kconfig.update; at info level by default (i.e. unless otherwise configured kconf_update is now silent)
  - --debug cmdline option now also attempts to force-enable the debug mode on that category (and debugs that attempt in of itself, so we don't get confused by categories magically getting enabled).
  - all log() calls are now qCDebug calls
  - all logFileErorr() calls (which was context-sensitive to the .upd file parsing) have been changed to qCDebugFile
  - qCDebugFile is a new *macro* wrapping around qCDebug to give it file context
  - everything is now qCDebug instead of qDebug
  - arguments updated to drop excess quoting and spaces to reduce "noise"
  
  https://markmail.org/thread/ofaeqcabguxyohyb

TEST PLAN
  updater still works and debugs when debug is enabled

REPOSITORY
  R237 KConfig

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D19508

AFFECTED FILES
  CMakeLists.txt
  src/kconf_update/CMakeLists.txt
  src/kconf_update/kconf_update.cpp

To: sitter
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190304/5f577e21/attachment.html>


More information about the Kde-frameworks-devel mailing list