[PATCH xts] Set XT_DISPLAYHOST for remote connections

Peter Harris pharris at opentext.com
Mon Jan 7 10:54:11 PST 2013


On 2013-01-04 21:12, Aaron Plattner wrote:
> On 01/02/2013 02:38 PM, Peter Harris wrote:
>> Some tests segfault if XT_DISPLAY is set and XT_DISPLAYHOST is not.
>>
>> Signed-off-by: Peter Harris <pharris at opentext.com>
> 
> Reviewed-by: Aaron Plattner <aplattner at nvidia.com>
> 
> Which test?

Xlib3/XOpenDisplay

> Tested-by: Aaron Plattner <aplattner at nvidia.com>

Thanks.

I just noticed, however, that

>> +        my @displayhost = split(/:/, $::vars{"DISPLAY"}, 2);
>> +        $::vars{"XT_DISPLAYHOST"} = $displayhost[0];

can't work for IPv6 literal addresses. I need to split on the last :,
not the first one.

I'll redo the patch.

Peter Harris
-- 
               Open Text Connectivity Solutions Group
Peter Harris                    http://connectivity.opentext.com/
Research and Development        Phone: +1 905 762 6001
pharris at opentext.com            Toll Free: 1 877 359 4866


More information about the xorg-devel mailing list