<div dir="ltr"><div><div><div>Hi Gianni,<br><br></div>I don't recall the decision making process w.r.t the xrandr 1.2 version check, it'll have been quite a few years back. My guess that was the minimum version that looked appropriate at the time.<br><br></div>Just throwing away the version check is unlikely to be generally safe. It'd only be safe if we could confirm that all versions of xrandr in circulation supported the functionality that osgViewer's X11 code paths require. A better solution than just removing the check would be to find the earliest version of xrandr that supports the features we need.<br><br></div><div>I'm curious, what system are you using for it to only have an old version of xrandr?<br></div><div><br></div>Robert.<br><div><br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 11 November 2015 at 16:07, Gianni Ambrosio <span dir="ltr"><<a href="mailto:g.ambrosio+osg@gmail.com" target="_blank">g.ambrosio+osg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
I'm trying to get the screen refresh rate from GraphicsWindowX11::getScreenSettings(...). Looking at the code I realized the point is the xrandr version. In fact supportsRandr() methods require xrandr version at least 1.2. But since I found the line:<br>
<br>
OSG_NOTICE << "You must build osgViewer with Xrandr 1.2 or higher for setScreenSettings support!" << std::endl;<br>
<br>
I suspect my xrandr 1.1.1 whould be enough to just read the sceen refresh rate (and it works fine indeed if remove the xrandr version check!).<br>
Moreover the supportsRandr() check is used just in two methods that simply read the screen settings while in setScreenSettings(...) it is not used at all.<br>
<br>
Now, in the actual code (I verified the OSG trunk), the xrand version check is really required?<br>
<br>
Regards,<br>
Gianni<br>
<br>
------------------<br>
Read this topic online here:<br>
<a href="http://forum.openscenegraph.org/viewtopic.php?p=65602#65602" rel="noreferrer" target="_blank">http://forum.openscenegraph.org/viewtopic.php?p=65602#65602</a><br>
<br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</blockquote></div><br></div>