Code review comment for lp:~compiz-team/compiz/compiz.fix_1080989

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

Ah. Run it in a gnome shell session to reproduce. In any case I can change
the name of the tests and the branches, we should probably rename the tests
to stop using the word "Test" for the reason you specify.
On 20/11/2012 5:34 PM, "Daniel van Vugt" <email address hidden>
wrote:

> Review: Needs Fixing
>
> I can't reproduce anything like bug 1080989 to verify. But I can see that
> this seems to fix bug 1070817. Can we make it bug 1070817 instead or are
> they separate?
>
> Also, the copyright header on new files should be one of your own :)
> --
>
> https://code.launchpad.net/~compiz-team/compiz/compiz.fix_1080989/+merge/135065
> Your team Compiz Maintainers is subscribed to branch lp:compiz.
>

« Back to merge proposal