[PATCH app/xdpyinfo] Use XRANDR 1.2 extension for reporting dimensions and resolution per output

[Eric Anholt]

Pali Rohár <pali.rohar at gmail.com> writes:

> [ Unknown signature status ]
> On Saturday 06 May 2017 13:28:17 Julien Cristau wrote:
>> On Fri, May  5, 2017 at 10:08:14 +0200, Pali Rohár wrote:
>> > PING.
>> > 
>> > I would like to know if there is some problem with this and
>> > something needs to be reworked or if it can be accepted.
>> > 
>> > Currently xdpyinfo report bogus dimensions and resolution values
>> > and lot of people complain about this problem. It should be fixed.
>> 
>> Yes, there's a problem with this.  xrandr is the tool to get monitor
>> dimensions, xdpyinfo serves a different purpose.  You could add
>> support for xdpyinfo -ext RANDR, but changing the default output is
>> a bad idea IMO.
>
> But xdpyinfo already output dimensions and that value is incorrect.
>
> If you do not agree that it should be fixed, then I can send patch to 
> totally remove dimensions information from xdpyinfo to not provide 
> incorrect and bogus values anymore.

That would also not be accepted.  You don't get to rewrite output like
this in old CLI tools that have had consistent behavior for decades.  It
causes breakage for systems expecting old behavior, even when that
behavior is unfortunate and wrong.  You just build different, better
tools, and convince users to switch over.  In this case, that tool
exists and is called xrandr.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20170508/e230d1e1/attachment.sig>