Remove randr 1.1 support in xfce4-display-settings?

[Alex Wolfson]

On 07/28/2010 05:58 PM, Alex Wolfson wrote:
> On 07/27/2010 05:41 PM, Alex Wolfson wrote:
>   
>> On 07/27/2010 05:30 PM, Lionel Le Folgoc wrote:
>>   
>>     
>>> On Tue, Jul 27, 2010 at 08:06:06PM +0000, Alex Wolfson wrote:
>>>   
>>>     
>>>       
>>>> Auke Kok <auke <at> foo-projects.org> writes:
>>>>
>>>>     
>>>>       
>>>>         
>>>>> Nick Schermer wrote:
>>>>>       
>>>>>         
>>>>>           
>>>>>> 2010/7/4 Jérôme Guelfucci <jeromeg <at> xfce.org>:
>>>>>>         
>>>>>>           
>>>>>>             
>>>>>>> I propose that we drop the randr 1.1 only support from our display stack. It
>>>>>>> brings very little benefits as (afaik) it is only useful for proprietary
>>>>>>> drivers which also offer their own graphical tool to handle displays AND it
>>>>>>> is a real maintenance burden and slows development (testing all scenarios is
>>>>>>> really annoying).
>>>>>>>           
>>>>>>>             
>>>>>>>               
>>>>>> I'd say go for it and focus on randr 1.2, the closed source driver
>>>>>> have custom tools (which work a lot better for them) afaik.
>>>>>>         
>>>>>>           
>>>>>>             
>>>>> yep. just kill it. That's what my colleagues would say :)
>>>>>
>>>>> Auke
>>>>>
>>>>>       
>>>>>         
>>>>>           
>>>> Hi all,
>>>>
>>>> I was trying to fix that display resolution settings, 
>>>> by porting xrandr code to xfce. 
>>>> I have very limited xfce experience so what Jerone is doing is of cause
>>>> better. 
>>>> I had ported display setting confirmation dialog from gnome so far,
>>>>  and then realized that randr 1.2 support is disabled in
>>>> dialogs/display-settings/main.c
>>>> /* not available yet */
>>>> #undef HAS_RANDR_ONE_POINT_TWO
>>>>  
>>>> So when this new Jerone's code will be available?
>>>> Can I wait for it (which is of cause preferable) or have to implement some
>>>> through away solution because I need it for the customer soon? 
>>>>
>>>> Thanks,
>>>>
>>>> Alex 
>>>>
>>>>     
>>>>       
>>>>         
>>> Hi,
>>>
>>> The code is already available on git.xfce.org, in the
>>> 'xrandr-display-settings' branch of xfce4-settings (not merged into
>>> master yet). You can get it and build it, it should work; it's not
>>> widely tested though, no more than three or four people afaik…
>>>
>>> You can already set many settings in the UI, except the output positions
>>> and which screen is primary (but xfconf-query or xrandr can be used for
>>> that anyway). However, Jerôme is in vacation with limited connectivity
>>> until the end of the week (iirc), and I don't think the new UI is ready,
>>> thus you'll still have the "old-not-so-shiny" one with a treeview and
>>> some comboboxes. ;)
>>>
>>> Regards,
>>> Lionel
>>>
>>>   
>>>     
>>>       
>> Hi Lionel,
>>
>> Thanks a lot for the quick response.
>> I will give it a try and post the feedback on the list.
>> BTW this is for the ARM based board.
>>
>> Thanks again,
>>
>> Alex
>>
>>   
>>     
> Hi,
>
> I build the new code for the ARM board, that runs xfce 4.6.4 (at least
> the name of the package is xfce4-settings-4.6.4)
>
> 1. xfce4-display-settings shows 4 available resolutions, xrandr shows 6
> 2. I can not change the resolution from the gui. Confirmation dialog
> appears, but nothing happened.
> 3. If I change resolution using xrandr and then run
> xfce4-display-settings, gui shows current resolution correctly
>
> I also built xfce4-settings for x86 (Ubuntu 10.04 LTS 64 bit) but gui
> did not start. I guess something needs to be installed, or xfce gui can
> not coexist with gnome?
>
> Are those issues known? I built with -g -O0 so hopefully I will be able
> to debug it tomorrow
>
> Thanks,
>
> Alex W.
>   

Hi,

Sorry for the premature panic.

I did not update all the programs on the board. I did not realized that
xfce4-display-helper is a separate executable.

After installing the whole package GUI works.
The only issue is that

xfce4-display-settings shows 4 available resolutions, xrandr shows 6

but I can leave with that for now.

Thank you for the great work,

Alex W.