Bug 882731 - Debug output in kdelibs-udisks2-backend.patch should be disabled
Summary: Debug output in kdelibs-udisks2-backend.patch should be disabled
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: kdelibs
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Lukáš Tinkl
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-03 00:01 UTC by Sandro Mani
Modified: 2015-07-13 08:11 UTC (History)
11 users (show)

Fixed In Version: kdelibs-4.9.4-5.fc18
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-01-12 01:20:11 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Sandro Mani 2012-12-03 00:01:16 UTC
Description of problem:
There are some uncommented qDebug statements in the kdelibs-udisks2-backend.patch which results in occasional
"/org/freedesktop/UDisks2/drives/WDC_WD2500BEVS_08VAT1_WD_WXEY08VT2199" 's interface "org.freedesktop.UDisks2.Drive.Ata" changed props: 
         modified: "SmartTemperature" : QVariant(double, 303.15) 
         modified: "SmartUpdated" : QVariant(qulonglong, 1354491783) 
         modified: "SmartTemperature" : QVariant(double, 303.15) 
         modified: "SmartUpdated" : QVariant(qulonglong, 1354491783) 
style messages appearing in log windows of applications which write their errors to the standard streams and redirect the output to the log window (and of course, it appears on the terminal and in .xsession-errors otherwise).

Would be nice to silence these debug messages.

Version-Release number of selected component (if applicable):
kdelibs-4.9.3-4.fc19.x86_64

Comment 1 Kevin Kofler 2012-12-03 17:46:44 UTC
Yes, please make your code STFU, terminal spam sucks!

Comment 2 Rex Dieter 2012-12-03 18:04:53 UTC
In fairness, the code has kDebug statements, which should be disabled by default.

Sandro, can you confirm with "kdebugdialog" whether "disable all debug output" option is checked or not?

Comment 3 Daniel Vrátil 2012-12-03 18:27:31 UTC
Actually Solid is using QDebug, because it can't depend on any KDE code.

Anyway, shouldn't QDebug output be suppressed in release packages?

Comment 4 Kevin Kofler 2012-12-03 19:08:19 UTC
No, only KDebug is.

Comment 5 nucleo 2012-12-14 02:38:34 UTC
All backend debug messages gone immediately after installing kdelibs-4.9.4-5.fc18.

Comment 6 Kevin Kofler 2012-12-24 02:18:34 UTC
Lukáš, why did you close this? The update is not pushed yet, in fact not even queued!

Comment 7 Fedora Update System 2012-12-24 02:19:13 UTC
kdelibs-4.9.4-5.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/kdelibs-4.9.4-5.fc18

Comment 8 Fedora Update System 2012-12-24 22:09:00 UTC
Package kdelibs-4.9.4-5.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kdelibs-4.9.4-5.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-20918/kdelibs-4.9.4-5.fc18
then log in and leave karma (feedback).

Comment 9 Fedora Update System 2013-01-12 01:20:13 UTC
kdelibs-4.9.4-5.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.