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