Code review comment for lp://qastaging/~vanvugt/ubuntu/raring/ubuntu-settings/fix-1073202

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Hm, ok.

Well I'll try and get a fix out for the read/write to incorrect places
as soon as I can (its non-trivial so it will take me a while like I
said), but we should keep this in the pipeline - having multiple
values for the key in the override makes no sense anyways as compiz
just takes the first one and uses that.

On Wed, Nov 28, 2012 at 12:11 AM, Didier Roche <email address hidden> wrote:
> the bug is affecting *real systems*, and doesn't only impact this particular key but others (all the integrated keys) as well.
>
> And so, if I drop the override, as it's what we should do (having the value at only one place, compiz and not two), we'll still have the bug as the default is <default>[]</default>. So this won't fix it.
> --
> https://code.launchpad.net/~vanvugt/ubuntu/raring/ubuntu-settings/fix-1073202/+merge/134424
> You are reviewing the proposed merge of lp:~vanvugt/ubuntu/raring/ubuntu-settings/fix-1073202 into lp:ubuntu/ubuntu-settings.

--
Sam Spilsbury

« Back to merge proposal