Re: [vdagent-linux 1/2] randr: remove monitors.xml on auto-configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 17, 2015 at 10:12 PM, Fabiano Fidêncio <fabiano@xxxxxxxxxxxx> wrote:
> Hey,
>
> Sorry for coming a bit late to the playground ...
>
> On Wed, Oct 8, 2014 at 3:35 PM, Marc-André Lureau
> <marcandre.lureau@xxxxxxxxx> wrote:
>>
>> On Wed, Oct 8, 2014 at 3:21 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx>
>> wrote:
>>>
>>> gnome2 is obsolete, not using client monitor config is getting obsolete,
>>> and removing a file which we don't own is a very heavy hammer,
>>
>>
>> A hammer? monitors.xml is a user config, safe to delete, "well-known" file
>> name (in gnomerr API reference)
>>
>> We could link with libgnome-desktop and call
>> gnome_rr_config_get_intended_filename() instead, I thought that was overkill
>> for something that is hardcoded.
>>
>> This file is not hidden or protected in any way, it doesn't use a private
>> namespace etc.
>>
>>>
>>> especially since this file is used by mutter in GNOME3 as well, so no,
>>> let's not do that upstream.
>>
>>
>> we don't want to restore user config when vdagent config applies.
>> Fortunately, with gnome3 & drm driver, that code path is not triggered.
>>
>> Having a random outcome is and a different output in upstream and downstream
>> is not better imho.
>
> I tend to agree with Marc-André here, basically because we do *not*
> want to restore user config anyway when vdagent config applies.
> +1 for having the patch upstream (and downstream).
>
> Best Regards,
> --
> Fabiano Fidêncio


Pushed both upstream ...
-- 
Fabiano Fidêncio
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]