Bug 7968

Summary: [patch] xdpyinfo should list the virtual root window ID, as well as the actual root window ID
Product: xorg Reporter: Eugene St Leger <grimrc>
Component: App/otherAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED INVALID QA Contact: Xorg Project Team <xorg-team>
Severity: enhancement    
Priority: medium Keywords: patch
Version: 7.1 (2006.05)   
Hardware: All   
OS: All   
Whiteboard: 2011BRB_Reviewed
i915 platform: i915 features:
Attachments:
Description Flags
patch to xdpyinfo 1.0.1 to add virtual root window support and *improperly* add -version support; please extend/correct none

Description Eugene St Leger 2006-08-23 10:33:35 UTC 
xdpyinfo should list the virtual root window ID, as well as the actual root
window ID.  This would allow you to parse xdpyinfo's output and find the virtual
root window ID.  Then you can feed this to $SDL_WINDOWID and run an SDL
application as a screensaver (its video targeted at the virtual root window, as
required).

For more info on using SDL applications as screensavers, see this:
http://wiki.openttd.org/index.php?title=Openttd_%28x11%29_screensaver_howto

I've created a patch to xdpyinfo (1.0.1) that includes 'vroot.h' from
xscreensaver 5.00.  The license seems acceptable, but this should be verified. 
The patched xdpyinfo (1.0.2?) prints the virtual root window ID after the root
window ID.

I also added a -version option, and a version string in xdpyinfo.c.  The build
environment needs to support this too (such as the Makefile), but I can't do
this.  The man page also needs updating.

Thanks!
Comment 1 Eugene St Leger 2006-08-23 10:36:23 UTC 
Created attachment 6661 [details] [review]
patch to xdpyinfo 1.0.1 to add virtual root window support and *improperly* add -version support; please extend/correct

please correct/extend this patch to be cleaner!
Comment 2 Eugene St Leger 2006-08-24 10:57:34 UTC 
The 'vroot.h' added by my patch (from xscreensaver 5.00), and in particular its
function 'VirtualRootWindowOfScreen()', might be a candidate for inclusion with
Xorg, so that it can be reliably found in /usr/include/X11/ where Xlib.h is
expected to be found.  The 'Window * VirtualRootWindowOfScreen(Screen *)'
fuction is very similar to 'Window * RootWindowOfScreen(Screen *)' in Xlib.h,
and depends on it.  By including 'VirtualRootWindowOfScreen()' other Xorg
utilities, such as xwininfo, could use it, without duplication of files (vroot.h).
Comment 3 Daniel Stone 2007-02-27 01:33:18 UTC 
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 4 Adam Jackson 2018-06-12 19:09:59 UTC 
Mass closure: This bug has been untouched for more than six years, and is not
obviously still valid. Please reopen this bug or file a new report if you continue to experience issues with current releases.

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.