Merge lp:~mniess/ubuntu/precise/compiz/fix-screenshot into lp:ubuntu/precise/compiz

Proposed by Matthias Niess on 2012-04-03
Status: Superseded
Proposed branch: lp:~mniess/ubuntu/precise/compiz/fix-screenshot
Merge into: lp:ubuntu/precise/compiz
Diff against target: 46 lines (+26/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/fix-screenshots.patch (+17/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~mniess/ubuntu/precise/compiz/fix-screenshot
Reviewer Review Type Date Requested Status
Didier Roche Resubmit on 2012-04-17
Ubuntu branches 2012-04-03 Pending
Review via email: mp+100539@code.launchpad.net

This proposal has been superseded by a proposal from 2012-04-17.

Description of the Change

fix compiz screenshot plugin by removing the blue tint as a workaround (LP: #771875)

To post a comment you must log in.
Didier Roche (didrocks) wrote :

Thanks for this patch and help to make ubuntu better.

For compiz, we try to have all the patches reviewed upstream as they are closed to us. Can you please propose your fix against lp:compiz-core so that it's reviewed by them? Thanks :)

review: Resubmit

Unmerged revisions

263. By Matthias Niess on 2012-04-03

* debian/patches/fix-screenshots.patch:
  - fix compiz screenshot plugin by removing the blue tint as a
    workaround (LP: #771875)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-03-26 14:57:30 +0000
3+++ debian/changelog 2012-04-03 00:28:19 +0000
4@@ -1,3 +1,11 @@
5+compiz (1:0.9.7.2-0ubuntu5) precise; urgency=low
6+
7+ * debian/patches/fix-screenshots.patch:
8+ - fix compiz screenshot plugin by removing the blue tint as a
9+ workaround (LP: #771875)
10+
11+ -- Matthias Niess <mniess@gmail.com> Tue, 03 Apr 2012 02:23:41 +0200
12+
13 compiz (1:0.9.7.2-0ubuntu4) precise; urgency=low
14
15 * No-change upload to the release pocket instead of to -proposed, so we can
16
17=== added file 'debian/patches/fix-screenshots.patch'
18--- debian/patches/fix-screenshots.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/fix-screenshots.patch 2012-04-03 00:28:19 +0000
20@@ -0,0 +1,17 @@
21+## Description: add some description
22+## Origin/Author: add some origin or author
23+## Bug: bug URL
24+=== modified file 'plugins/screenshot/src/screenshot.cpp'
25+--- plugins/screenshot/src/screenshot.cpp 2012-02-13 15:28:06 +0000
26++++ plugins/screenshot/src/screenshot.cpp 2012-04-02 23:06:36 +0000
27+@@ -277,7 +277,8 @@
28+ glDisableClientState (GL_TEXTURE_COORD_ARRAY);
29+ glEnable (GL_BLEND);
30+ glColor4us (0x2fff, 0x2fff, 0x4fff, 0x4fff);
31+- glRecti (x1, y2, x2, y1);
32++ # workaround remove blue overlay
33++ # glRecti (x1, y2, x2, y1);
34+ glColor4us (0x2fff, 0x2fff, 0x4fff, 0x9fff);
35+ glBegin (GL_LINE_LOOP);
36+ glVertex2i (x1, y1);
37+
38
39=== modified file 'debian/patches/series'
40--- debian/patches/series 2012-03-26 18:41:09 +0000
41+++ debian/patches/series 2012-04-03 00:28:19 +0000
42@@ -8,3 +8,4 @@
43 exit_1_if_composite_cant_init.patch
44 always_replace.patch
45 reset_active_plugin_list.patch
46+fix-screenshots.patch

Subscribers

People subscribed via source and target branches