On Thu, Feb 28, 2019 at 12:43 PM Harald Sitter <sit...@kde.org> wrote: > > ...and I don't understand why > > Hi! > > this commit [1] wrapped KonfUpdate::log's update.log stream in `#if 0` > and thus disabling the update.log writing, replacing it with logging > to stderr instead. Why it does that eludes me though. It seems > entirely unrelated to the rest of the commit.
Sounds like he didn't go through review. > Should the update.log writing happen? I expect this won't be super > reliable because I think these days the application itself forks > kconf_update. If so, we should just drop the if block and update > documentation accordingly I guess (e.g. see [2]) If anything, instead of going directly to stderr it should be using QDebug and go through journald (or whatever the system uses), no? > [1] > https://cgit.kde.org/kconfig.git/commit/?id=94419b6f0cd469dc7cb0184a6fb48c1fe304c4d8 > [2] https://bugs.kde.org/show_bug.cgi?id=404904