Merge lp://qastaging/~jamespharaoh/ubuntu/precise/compiz/fix-881329 into lp://qastaging/ubuntu/precise/compiz

Proposed by James Pharaoh
Status: Work in progress
Proposed branch: lp://qastaging/~jamespharaoh/ubuntu/precise/compiz/fix-881329
Merge into: lp://qastaging/ubuntu/precise/compiz
Diff against target: 1053 lines (+963/-1)
10 files modified
.pc/.quilt_patches (+1/-0)
.pc/.quilt_series (+1/-0)
.pc/applied-patches (+1/-0)
.pc/fix-881329.patch/plugins/move/src/move.cpp (+756/-0)
.pc/fix-881329.patch/plugins/move/src/move.h (+135/-0)
debian/changelog (+7/-0)
debian/patches/fix-881329.patch (+49/-0)
debian/patches/series (+1/-0)
plugins/move/src/move.cpp (+10/-1)
plugins/move/src/move.h (+2/-0)
To merge this branch: bzr merge lp://qastaging/~jamespharaoh/ubuntu/precise/compiz/fix-881329
Reviewer Review Type Date Requested Status
Sam Spilsbury (community) Approve
Ubuntu branches Pending
Review via email: mp+80314@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work, there is quite some source v3 diff noise but the patch fix-881329.patch is at the bottom of the diff in a reviewable state

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

Hi,

This looks fine, though can you propose it against lp:compiz-core ? This is the packaging branch.

review: Approve
Revision history for this message
Michael Terry (mterry) wrote :

James, thanks for your awesome work to get focus follows mouse working in compiz. But I'm gathering that Sam (a compiz maintainer) would rather have this proposed upstream first.

So I'm marking this branch as 'work in progress' so it doesn't show up in the sponsoring report. James, please propose a merge of your code into lp:compiz-core. Thanks!

Unmerged revisions

253. By James Pharaoh

* debian/patches/fix-881329.patch:
  - Alt+click should raise with raise-on-click disabled (LP: #881329)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: