Bug 99749 - Unrelated stderr output with DBUS_VERBOSE
Summary: Unrelated stderr output with DBUS_VERBOSE
Status: RESOLVED FIXED
Alias: None
Product: dbus
Classification: Unclassified
Component: core (show other bugs)
Version: git master
Hardware: Other Windows (All)
: medium normal
Assignee: D-Bus Maintainers
QA Contact: D-Bus Maintainers
URL:
Whiteboard: review+
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-02-10 07:35 UTC by Ralf Habacker
Modified: 2017-02-10 12:11 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Do not show unrelated stderr output with DBUS_VERBOSE when using windows debug port as output channel. (1.08 KB, patch)
2017-02-10 07:45 UTC, Ralf Habacker
Details | Splinter Review

Description Ralf Habacker 2017-02-10 07:35:18 UTC
On windows dbus source compiled with cmake can be configured to use the windows debug port displayed by dbgview or other debuggers with -DDBUS_USE_OUTPUT_DEBUG_STRING.

Running dbus-daemon or dbus-monitor from such a configured build e.g. 

set DBUS_VERBOSE=1
bin/dbus-daemon.exe --session 

or 

set DBUS_VERBOSE=1
bin/dbus-monitor.exe

returns a list of unwanted timestamps on stderr.

1486711202.032110 1486711202.078986 1486711202.079318 1486711202.080303 1486711202.081313 1486711202.081594 1486711202.081873 1486711202.082134 1486711202.082394 1486711202.082642

Those messages should be removed.
Comment 1 Ralf Habacker 2017-02-10 07:45:51 UTC
Created attachment 129466 [details] [review]
Do not show unrelated stderr output with DBUS_VERBOSE when using windows debug port as output channel.
Comment 2 Philip Withnall 2017-02-10 10:16:14 UTC
Comment on attachment 129466 [details] [review]
Do not show unrelated stderr output with DBUS_VERBOSE when using windows debug port as output channel.

Review of attachment 129466 [details] [review]:
-----------------------------------------------------------------

Seems reasonable to me.
Comment 3 Philip Withnall 2017-02-10 10:17:01 UTC
You might want to reword the first line of the commit message though so it’s a bit shorter.
Comment 4 Simon McVittie 2017-02-10 11:50:03 UTC
(In reply to Philip Withnall from comment #2)
> Seems reasonable to me.

And me.

(In reply to Philip Withnall from comment #3)
> You might want to reword the first line of the commit message though so it’s
> a bit shorter.

(The first line is the entire commit message, in fact.)

Yes please. Perhaps something more like this:

     Only print timestamps on stderr if verbose messages will go there

     On Windows, dbus can be configured to send DBUS_VERBOSE messages
     to the Windows debug port instead of stderr. If we're in that
     configuration, we already avoided printing thread information to
     stderr before each verbose message; do the same for timestamps.
Comment 5 Ralf Habacker 2017-02-10 12:11:54 UTC
committed to git master with mentioned commit message change applied


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.