Bug 5429 - --enable-debug should be useful
Summary: --enable-debug should be useful
Status: NEW
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: janitor
Depends on:
Blocks:
 
Reported: 2005-12-27 12:33 UTC by Daniel Stone
Modified: 2009-10-05 22:12 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Stone 2005-12-27 12:33:49 UTC
passing --enable-debug should actually be useful.  there are numerous #ifdefs
throughout the server to enable a whole ton of stuff -- the most common is DEBUG
and NDEBUG, plus all the EXA fallback debugging.  for the drivers, there's stuff
like RADEON_DEBUG too.

it would be nice if the defines were all DEBUG or XORG_DEBUG or something, so
--enable-debug could just add -DDEBUG to cflags.
Comment 1 Daniel Stone 2007-02-27 01:29:34 UTC
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 2 Ben Byer 2009-10-02 00:18:39 UTC
Is this really just asking for someone to search and replace all #ifdef FOO_DEBUG with #ifdef DEBUG, or is there something more nuanced here?
Comment 3 Peter Hutterer 2009-10-05 22:12:10 UTC
the reason why all these debugs exist is AFAICT that most developers only want them for a specific subsystem. Once you enable everything, there's a lot of noise in the log file.

Something more nuanced would be nice. A simple solution could still be that DEBUG enables whatever local debug there is. This allows developers to manually enable bits without having the whole spew.
Or we standardise the debug output that it's easier to grep on the prefix.


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.