[Merge] lp:~albertsmuktupavels/nautilus/white-screen-fix into lp:ubuntu/saucy/nautilus

Alberts Muktupāvels alberts.muktupavels at gmail.com
Wed Nov 13 20:07:57 UTC 2013


> The patch "works" but upstream said it's the wrong approch. We are not going
> to take a patch that is not technically right, that has a maitainance cost and
> can create problems, using multiple xserver is not a common case as well (so
> we are less likely to accept a runtime cost for most users only to benefit an
> uncommon scenario)

Why it's wrong approch? I still know only one reason - "<alex> seb128: seems a bit simple to me, we don't want to construct a new object each time, rather we want one per screen". I have tested patch. New object is constructed only once per screen.

> Upstream had some issues with those changes, and it seems like the sort of
> changes we are not going to SRU to the LTS (we would need at least a patch
> that is done right and get testing on unstable serie first, then we can
> discuss backporting).

What issues they had?


-- 
https://code.launchpad.net/~albertsmuktupavels/nautilus/white-screen-fix/+merge/180231
Your team Ubuntu branches is subscribed to branch lp:ubuntu/saucy/nautilus.



More information about the Ubuntu-reviews mailing list