Merge lp:~timo-jyrinki/compiz/ubuntu.compiz-09712ubuntu2 into lp:~ubuntu-desktop/compiz/precise
- ubuntu.compiz-09712ubuntu2
- Merge into precise
Proposed by
Timo Jyrinki
Status: | Merged |
---|---|
Merged at revision: | 808 |
Proposed branch: | lp:~timo-jyrinki/compiz/ubuntu.compiz-09712ubuntu2 |
Merge into: | lp:~ubuntu-desktop/compiz/precise |
Diff against target: |
475 lines (+254/-111) 12 files modified
debian/changelog (+75/-0) debian/compiz-plugins.install.armel (+11/-0) debian/patches/blacklist_precise_mesa80_intel_nouveau.patch (+43/-0) debian/patches/compiz-package-gles2.patch (+10/-9) debian/patches/fix_1095001.patch (+86/-0) debian/patches/force_unredirect_enabling.patch (+22/-0) debian/patches/revert_fix_933776_955035.patch (+0/-65) debian/patches/revert_fix_994841.patch (+0/-33) debian/patches/series (+3/-2) debian/patches/series.armel (+1/-0) debian/patches/workaround_broken_drivers.patch (+2/-1) debian/watch (+1/-1) |
To merge this branch: | bzr merge lp:~timo-jyrinki/compiz/ubuntu.compiz-09712ubuntu2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Didier Roche-Tolomelli | Approve | ||
Review via email: mp+162779@code.launchpad.net |
Commit message
* debian/
- Rename the blacklist patch, add verbose description and allow
fullscreen unredirection for also Intel and Nouveau users using
Mesa 9.0.x (Ubuntu 12.04.2 LTS or those who have opted in to LTS-Q
packages). (LP: #1167321)
* debian/
- Cherry-pick a patch from upstream to fix CPU usage (LP: #1095001)
Description of the change
For didrocks. Only the last two commits are new since newest precise release.
To post a comment you must log in.
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-11-16 17:06:56 +0000 | |||
3 | +++ debian/changelog 2013-05-07 13:14:15 +0000 | |||
4 | @@ -1,3 +1,78 @@ | |||
5 | 1 | compiz (1:0.9.7.12-0ubuntu2) UNRELEASED; urgency=low | ||
6 | 2 | |||
7 | 3 | * debian/patches/fix_1095001.patch | ||
8 | 4 | - Cherry-pick a patch from upstream to fix CPU usage (LP: #1095001) | ||
9 | 5 | * debian/patches/blacklist_precise_mesa80_intel_nouveau.patch: | ||
10 | 6 | - Rename the blacklist patch, add verbose description and allow | ||
11 | 7 | fullscreen unredirection for also Intel and Nouveau users using | ||
12 | 8 | Mesa 9.0.x (Ubuntu 12.04.2 LTS or those who have opted in to LTS-Q | ||
13 | 9 | packages). (LP: #1167321) | ||
14 | 10 | |||
15 | 11 | -- Timo Jyrinki <timo-jyrinki@ubuntu.com> Thu, 11 Apr 2013 12:05:04 +0300 | ||
16 | 12 | |||
17 | 13 | compiz (1:0.9.7.12-0ubuntu1) precise-proposed; urgency=low | ||
18 | 14 | |||
19 | 15 | [ Łukasz 'sil2100' Zemczak ] | ||
20 | 16 | * debian/patches/revert_fix_994841.patch, | ||
21 | 17 | debian/patches/revert_fix_933776_955035.patch | ||
22 | 18 | - Removed, upstreamed | ||
23 | 19 | * debian/watch: | ||
24 | 20 | - Changed to the correct LP tarball path | ||
25 | 21 | * debian/patches/compiz-package-gles2.patch: | ||
26 | 22 | - Updated GLES patches to properly apply to the new compiz version | ||
27 | 23 | |||
28 | 24 | [ Timo Jyrinki ] | ||
29 | 25 | * New upstream release. | ||
30 | 26 | - Compiz won't start if "unredirect fullscreen windows" is enabled | ||
31 | 27 | (LP: #980663) | ||
32 | 28 | - "Unredirect Fullscreen Windows" stay on top (unredirected) even | ||
33 | 29 | when they're not on top any more (or the output is transformed) | ||
34 | 30 | (LP: #1041047) | ||
35 | 31 | - Unredirect Fullscreen Windows sometimes fails to unredirect fullscreen | ||
36 | 32 | windows at all (LP: #1041066) | ||
37 | 33 | - "Unredirect Fullscreen Windows" stay on top (unredirected) even when | ||
38 | 34 | an RGBA window is stacked above it (LP: #1046661) | ||
39 | 35 | - scale mode is not visible if a fullscreen window is unredirected | ||
40 | 36 | (LP: #1047168) | ||
41 | 37 | - Unredirecting a fullscreen window on a secondary monitor causes that | ||
42 | 38 | monitor to flicker (LP: #1050749) | ||
43 | 39 | - "Unredirect Fullscreen Windows" makes multi-monitor rendering much | ||
44 | 40 | slower (LP: #1051885) | ||
45 | 41 | - [regression] Maximized window gets unredirected when it's not | ||
46 | 42 | fullscreen (LP: #1053902) | ||
47 | 43 | - Unredirected fullscreen windows freeze and stay on top when wall | ||
48 | 44 | sliding (Ctrl+Alt+Left/Right) (LP: #1084401) | ||
49 | 45 | - "Unredirect Fullscreen Windows" can cause significant tearing on | ||
50 | 46 | fullscreen windows (especially playing video) on some drivers | ||
51 | 47 | (LP: #1051802) | ||
52 | 48 | - HTML5 video in Firefox continues to tear (LP: #1086337) | ||
53 | 49 | - Add support for blacklisting some drivers from using unredirected | ||
54 | 50 | fullscreen windows. By default intel and nouveau on Mesa 8.0 | ||
55 | 51 | are blacklisted, configurable via ccsm. Users can upgrade to | ||
56 | 52 | Mesa 9.0 around 12.04.2 time. (LP: #1089246) | ||
57 | 53 | - Unredirect fullscreen windows should be the default for optimal | ||
58 | 54 | performance (LP: #1063690) | ||
59 | 55 | * debian/patches/workaround_broken_drivers.patch: | ||
60 | 56 | - Updated to apply. | ||
61 | 57 | * debian/patches/force_unredirect_enabling.patch: | ||
62 | 58 | - Enable unredirect_fullscreen_windows unconditionally. Because of | ||
63 | 59 | gconf hurdles we can't easily migrate existing users into the | ||
64 | 60 | new default otherwise. Unredirection can be easily disabled with | ||
65 | 61 | the new unredirect_match option, by blanking it completely, | ||
66 | 62 | including removing the '(any)' part. | ||
67 | 63 | * debian/patches/compiz-package-gles2.patch: | ||
68 | 64 | - Update for the new blacklisting feature as well | ||
69 | 65 | * debian/patches/blacklist_no_xml_double_escaping.patch: | ||
70 | 66 | - Cherry-pick fix for the default string (LP: #1091103) | ||
71 | 67 | |||
72 | 68 | [ Michael Terry ] | ||
73 | 69 | * debian/patches/blacklist_no_xml_double_escaping.patch: | ||
74 | 70 | - Update to also blacklist Mesa 9.0, which precise users may have from | ||
75 | 71 | the x-updates PPA. 9.0 seems to have similar problems as 8.0 on | ||
76 | 72 | Intel hardware. | ||
77 | 73 | |||
78 | 74 | -- Timo Jyrinki <timo-jyrinki@ubuntu.com> Fri, 14 Dec 2012 10:41:24 +0200 | ||
79 | 75 | |||
80 | 1 | compiz (1:0.9.7.8+bzr3121-0ubuntu1) precise-proposed; urgency=low | 76 | compiz (1:0.9.7.8+bzr3121-0ubuntu1) precise-proposed; urgency=low |
81 | 2 | 77 | ||
82 | 3 | * New upstream stable branch snapshot. | 78 | * New upstream stable branch snapshot. |
83 | 4 | 79 | ||
84 | === modified symlink 'debian/compiz-plugins.install.armel' | |||
85 | === target was u'compiz-plugins.install.armhf' | |||
86 | --- debian/compiz-plugins.install.armel 1970-01-01 00:00:00 +0000 | |||
87 | +++ debian/compiz-plugins.install.armel 2013-05-07 13:14:15 +0000 | |||
88 | @@ -0,0 +1,11 @@ | |||
89 | 1 | debian/tmp/usr/*/compiz/*annotate.* | ||
90 | 2 | debian/tmp/usr/*/compiz/*clone.* | ||
91 | 3 | debian/tmp/usr/*/compiz/*commands.* | ||
92 | 4 | debian/tmp/usr/*/compiz/*copytex.* | ||
93 | 5 | debian/tmp/usr/*/compiz/*dbus.* | ||
94 | 6 | debian/tmp/usr/*/compiz/*imgsvg.* | ||
95 | 7 | debian/tmp/usr/*/compiz/*inotify.* | ||
96 | 8 | debian/tmp/usr/*/compiz/*obs.* | ||
97 | 9 | debian/tmp/usr/*/compiz/*screenshot.* | ||
98 | 10 | debian/tmp/usr/*/compiz/*switcher.* | ||
99 | 11 | debian/tmp/usr/*/compiz/*water.* | ||
100 | 0 | 12 | ||
101 | === added file 'debian/patches/blacklist_precise_mesa80_intel_nouveau.patch' | |||
102 | --- debian/patches/blacklist_precise_mesa80_intel_nouveau.patch 1970-01-01 00:00:00 +0000 | |||
103 | +++ debian/patches/blacklist_precise_mesa80_intel_nouveau.patch 2013-05-07 13:14:15 +0000 | |||
104 | @@ -0,0 +1,43 @@ | |||
105 | 1 | Description: Correct precise specific blacklist string | ||
106 | 2 | This patch sets the correct 12.04 LTS (precise) specific blacklist | ||
107 | 3 | string for not enabling unredirection speedup on older Mesa versions | ||
108 | 4 | when using Intel or Nouveau. It has evolved a bit. | ||
109 | 5 | . | ||
110 | 6 | Firstly it was just a cherry-pick to remove double backslashes which | ||
111 | 7 | turned out not to work well from XML -> regexp engine. | ||
112 | 8 | . | ||
113 | 9 | Secondly "9.0" was added because an user reported that Mesa 9.0 from | ||
114 | 10 | x-updates PPA wasn't enough on Intel. The full 12.04.2 stack however is | ||
115 | 11 | good. | ||
116 | 12 | . | ||
117 | 13 | Thirdly as continuation from above, the string was modified so that | ||
118 | 14 | 9.0 (sharp) is still blacklisted, but 9.0.x (now in 12.04.2 LTS stack) | ||
119 | 15 | is not anymore. This takes care of the remaining PPA users while allowing | ||
120 | 16 | proper new stack users to have full gaming speed on Intel and Nouveau. | ||
121 | 17 | Author: Timo Jyrinki <timo.jyrinki@canonical.com> | ||
122 | 18 | Author: Michael Terry <michael.terry@canonical.com> | ||
123 | 19 | === modified file 'plugins/opengl/opengl.xml.in' | ||
124 | 20 | --- a/plugins/opengl/opengl.xml.in 2012-12-12 07:24:34 +0000 | ||
125 | 21 | +++ b/plugins/opengl/opengl.xml.in 2012-12-17 10:39:15 +0000 | ||
126 | 22 | @@ -46,7 +46,7 @@ | ||
127 | 23 | <option name="unredirect_driver_blacklist" type="string"> | ||
128 | 24 | <_short>Unredirect Driver Blacklist</_short> | ||
129 | 25 | <_long>If non-empty, specifies a POSIX (extended) regular expression to match against the OpenGL driver strings (newline separated): "GL_VENDOR\nGL_RENDERER\nGL_VERSION". If the regular expression matches a substring of that concatenation then no windows will ever be unredirected while using that particular graphics driver.</_long> | ||
130 | 26 | - <default>(nouveau|Intel).*Mesa 8\\.0</default> | ||
131 | 27 | + <default>(nouveau|Intel).*Mesa (8.0|9.0$)</default> | ||
132 | 28 | </option> | ||
133 | 29 | </options> | ||
134 | 30 | </plugin> | ||
135 | 31 | |||
136 | 32 | === modified file 'plugins/opengl/src/blacklist/tests/test-blacklist.cpp' | ||
137 | 33 | --- a/plugins/opengl/src/blacklist/tests/test-blacklist.cpp 2012-12-12 07:24:34 +0000 | ||
138 | 34 | +++ b/plugins/opengl/src/blacklist/tests/test-blacklist.cpp 2012-12-17 10:40:00 +0000 | ||
139 | 35 | @@ -29,7 +29,7 @@ | ||
140 | 36 | using namespace compiz::opengl; | ||
141 | 37 | |||
142 | 38 | -static const char *recommendedRegex = "(nouveau|Intel).*Mesa 8\\.0"; | ||
143 | 39 | +static const char *recommendedRegex = "(nouveau|Intel).*Mesa (8.0|9.0$)"; | ||
144 | 40 | |||
145 | 41 | TEST(DriverBlacklist, QuantalIntelIsGood) | ||
146 | 42 | { | ||
147 | 43 | EXPECT_FALSE (blacklisted (recommendedRegex, | ||
148 | 0 | 44 | ||
149 | === modified file 'debian/patches/compiz-package-gles2.patch' | |||
150 | --- debian/patches/compiz-package-gles2.patch 2012-11-12 08:19:15 +0000 | |||
151 | +++ debian/patches/compiz-package-gles2.patch 2013-05-07 13:14:15 +0000 | |||
152 | @@ -988,23 +988,23 @@ | |||
153 | 988 | 988 | ||
154 | 989 | void changePaintModifier (unsigned int, int); | 989 | void changePaintModifier (unsigned int, int); |
155 | 990 | diff -Nur '--exclude=.*' '--exclude=debian' compiz-0.9.7.8/plugins/opengl/CMakeLists.txt linaro-gles2-debianpatches/plugins/opengl/CMakeLists.txt | 990 | diff -Nur '--exclude=.*' '--exclude=debian' compiz-0.9.7.8/plugins/opengl/CMakeLists.txt linaro-gles2-debianpatches/plugins/opengl/CMakeLists.txt |
159 | 991 | --- compiz-0.9.7.8/plugins/opengl/CMakeLists.txt 2012-04-26 08:47:01.000000000 +0200 | 991 | --- compiz-0.9.7.8/plugins/opengl/CMakeLists.txt 2012-11-12 06:59:31 +0000 |
160 | 992 | +++ linaro-gles2-debianpatches/plugins/opengl/CMakeLists.txt 2012-04-20 11:41:59.241600000 +0200 | 992 | +++ linaro-gles2-debianpatches/plugins/opengl/CMakeLists.txt 2012-12-10 13:35:59 +0000 |
161 | 993 | @@ -2,7 +2,12 @@ | 993 | @@ -4,8 +4,13 @@ |
162 | 994 | 994 | ||
164 | 995 | include (CompizPlugin) | 995 | add_subdirectory (src/fsregion) |
165 | 996 | add_subdirectory (src/blacklist) | ||
166 | 996 | 997 | ||
167 | 997 | -find_package (OpenGL) | 998 | -find_package (OpenGL) |
168 | 998 | -if (OPENGL_FOUND) | 999 | -if (OPENGL_FOUND) |
170 | 999 | - compiz_plugin(opengl PLUGINDEPS composite LIBRARIES ${OPENGL_gl_LIBRARY} INCDIRS ${OPENGL_INCLUDE_DIR}) | 1000 | - compiz_plugin(opengl PLUGINDEPS composite LIBRARIES ${OPENGL_gl_LIBRARY} compiz_opengl_fsregion compiz_opengl_blacklist INCDIRS ${OPENGL_INCLUDE_DIR}) |
171 | 1000 | -endif () | 1001 | -endif () |
172 | 1001 | \ Kein Zeilenumbruch am Dateiende. | ||
173 | 1002 | +if (USE_GLES) | 1002 | +if (USE_GLES) |
175 | 1003 | + compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD "-DUSE_GLES -std=c++0x" LIBRARIES ${OPENGLES2_LIBRARIES} INCDIRS ${OPENGLES2_INCLUDE_DIR}) | 1003 | + compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD "-DUSE_GLES -std=c++0x" LIBRARIES ${OPENGLES2_LIBRARIES} compiz_opengl_fsregion compiz_opengl_blacklist INCDIRS ${OPENGLES2_INCLUDE_DIR}) |
176 | 1004 | +else (USE_GLES) | 1004 | +else (USE_GLES) |
177 | 1005 | + find_package (OpenGL) | 1005 | + find_package (OpenGL) |
178 | 1006 | + if (OPENGL_FOUND) | 1006 | + if (OPENGL_FOUND) |
180 | 1007 | + compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD -std=c++0x LIBRARIES ${OPENGL_gl_LIBRARY} INCDIRS ${OPENGL_INCLUDE_DIR}) | 1007 | + compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD -std=c++0x LIBRARIES ${OPENGL_gl_LIBRARY} compiz_opengl_fsregion compiz_opengl_blacklist INCDIRS ${OPENGL_INCLUDE_DIR}) |
181 | 1008 | + endif (OPENGL_FOUND) | 1008 | + endif (OPENGL_FOUND) |
182 | 1009 | +endif (USE_GLES) | 1009 | +endif (USE_GLES) |
183 | 1010 | + | 1010 | + |
184 | @@ -5639,7 +5639,7 @@ | |||
185 | 5639 | */ | 5639 | */ |
186 | 5640 | 5640 | ||
187 | 5641 | #ifndef _GNU_SOURCE | 5641 | #ifndef _GNU_SOURCE |
189 | 5642 | @@ -30,12 +32,28 @@ | 5642 | @@ -30,13 +32,29 @@ |
190 | 5643 | #endif | 5643 | #endif |
191 | 5644 | #include <errno.h> | 5644 | #include <errno.h> |
192 | 5645 | 5645 | ||
193 | @@ -5649,6 +5649,7 @@ | |||
194 | 5649 | +#include <errno.h> | 5649 | +#include <errno.h> |
195 | 5650 | + | 5650 | + |
196 | 5651 | #include "privates.h" | 5651 | #include "privates.h" |
197 | 5652 | #include "blacklist/blacklist.h" | ||
198 | 5652 | 5653 | ||
199 | 5653 | #include <dlfcn.h> | 5654 | #include <dlfcn.h> |
200 | 5654 | #include <math.h> | 5655 | #include <math.h> |
201 | 5655 | 5656 | ||
202 | === added file 'debian/patches/fix_1095001.patch' | |||
203 | --- debian/patches/fix_1095001.patch 1970-01-01 00:00:00 +0000 | |||
204 | +++ debian/patches/fix_1095001.patch 2013-05-07 13:14:15 +0000 | |||
205 | @@ -0,0 +1,86 @@ | |||
206 | 1 | === modified file 'plugins/opengl/src/paint.cpp' | ||
207 | 2 | --- a/plugins/opengl/src/paint.cpp 2012-12-13 07:42:28 +0000 | ||
208 | 3 | +++ b/plugins/opengl/src/paint.cpp 2013-01-03 03:30:42 +0000 | ||
209 | 4 | @@ -324,22 +324,31 @@ | ||
210 | 5 | if (w->alpha ()) | ||
211 | 6 | flags |= FullscreenRegion::Alpha; | ||
212 | 7 | |||
213 | 8 | + CompositeWindow *cw = CompositeWindow::get (w); | ||
214 | 9 | + | ||
215 | 10 | /* | ||
216 | 11 | * Windows with alpha channels can partially occlude windows | ||
217 | 12 | * beneath them and so neither should be unredirected in that case. | ||
218 | 13 | + * | ||
219 | 14 | + * Performance note: unredirectable.evaluate is SLOW because it | ||
220 | 15 | + * involves regex matching. Too slow to do on every window for | ||
221 | 16 | + * every frame. So we only call it if a window is redirected AND | ||
222 | 17 | + * potentially needs unredirecting. This means changes to | ||
223 | 18 | + * unredirect_match while a window is unredirected already may not | ||
224 | 19 | + * take effect until it is un-fullscreened again. But that's better | ||
225 | 20 | + * than the high price of regex matching on every frame. | ||
226 | 21 | */ | ||
227 | 22 | if (unredirectFS && | ||
228 | 23 | !blacklisted && | ||
229 | 24 | - unredirectable.evaluate (w) && | ||
230 | 25 | !(mask & PAINT_SCREEN_TRANSFORMED_MASK) && | ||
231 | 26 | !(mask & PAINT_SCREEN_WITH_TRANSFORMED_WINDOWS_MASK) && | ||
232 | 27 | - fs.isCoveredBy (w->region (), flags)) | ||
233 | 28 | + fs.isCoveredBy (w->region (), flags) && | ||
234 | 29 | + (!cw->redirected () || unredirectable.evaluate (w))) | ||
235 | 30 | { | ||
236 | 31 | unredirected.insert (w); | ||
237 | 32 | } | ||
238 | 33 | else | ||
239 | 34 | { | ||
240 | 35 | - CompositeWindow *cw = CompositeWindow::get (w); | ||
241 | 36 | if (!cw->redirected ()) | ||
242 | 37 | { | ||
243 | 38 | if (fs.allowRedirection (w->region ())) | ||
244 | 39 | |||
245 | 40 | === modified file 'plugins/opengl/src/privates.h' | ||
246 | 41 | --- a/plugins/opengl/src/privates.h 2012-12-13 07:42:28 +0000 | ||
247 | 42 | +++ b/plugins/opengl/src/privates.h 2013-01-03 03:30:42 +0000 | ||
248 | 43 | @@ -131,6 +131,9 @@ | ||
249 | 44 | CompSize rootPixmapSize; | ||
250 | 45 | |||
251 | 46 | const char *glVendor, *glRenderer, *glVersion; | ||
252 | 47 | + | ||
253 | 48 | + mutable CompString prevRegex; | ||
254 | 49 | + mutable bool prevBlacklisted; | ||
255 | 50 | }; | ||
256 | 51 | |||
257 | 52 | class PrivateGLWindow : | ||
258 | 53 | |||
259 | 54 | === modified file 'plugins/opengl/src/screen.cpp' | ||
260 | 55 | --- a/plugins/opengl/src/screen.cpp 2012-12-13 07:42:28 +0000 | ||
261 | 56 | +++ b/plugins/opengl/src/screen.cpp 2013-01-03 03:30:42 +0000 | ||
262 | 57 | @@ -603,7 +603,9 @@ | ||
263 | 58 | rootPixmapSize (), | ||
264 | 59 | glVendor (NULL), | ||
265 | 60 | glRenderer (NULL), | ||
266 | 61 | - glVersion (NULL) | ||
267 | 62 | + glVersion (NULL), | ||
268 | 63 | + prevRegex (), | ||
269 | 64 | + prevBlacklisted (false) | ||
270 | 65 | { | ||
271 | 66 | ScreenInterface::setHandler (screen); | ||
272 | 67 | } | ||
273 | 68 | @@ -1306,7 +1308,16 @@ | ||
274 | 69 | bool | ||
275 | 70 | PrivateGLScreen::driverIsBlacklisted (const char *regex) const | ||
276 | 71 | { | ||
277 | 72 | - return blacklisted (regex, glVendor, glRenderer, glVersion); | ||
278 | 73 | + /* | ||
279 | 74 | + * regex matching is VERY expensive, so only do it when the result might | ||
280 | 75 | + * be different to last time. The gl* variables never change value... | ||
281 | 76 | + */ | ||
282 | 77 | + if (prevRegex != regex) | ||
283 | 78 | + { | ||
284 | 79 | + prevBlacklisted = blacklisted (regex, glVendor, glRenderer, glVersion); | ||
285 | 80 | + prevRegex = regex; | ||
286 | 81 | + } | ||
287 | 82 | + return prevBlacklisted; | ||
288 | 83 | } | ||
289 | 84 | |||
290 | 85 | GLTexture::BindPixmapHandle | ||
291 | 86 | |||
292 | 0 | 87 | ||
293 | === added file 'debian/patches/force_unredirect_enabling.patch' | |||
294 | --- debian/patches/force_unredirect_enabling.patch 1970-01-01 00:00:00 +0000 | |||
295 | +++ debian/patches/force_unredirect_enabling.patch 2013-05-07 13:14:15 +0000 | |||
296 | @@ -0,0 +1,22 @@ | |||
297 | 1 | === modified file 'plugins/opengl/src/paint.cpp' | ||
298 | 2 | --- a/plugins/opengl/src/paint.cpp 2012-12-13 07:42:28 +0000 | ||
299 | 3 | +++ b/plugins/opengl/src/paint.cpp 2012-12-14 07:48:47 +0000 | ||
300 | 4 | @@ -224,8 +224,15 @@ | ||
301 | 5 | CompWindowList pl; | ||
302 | 6 | CompWindowList::reverse_iterator rit; | ||
303 | 7 | |||
304 | 8 | - unredirectFS = CompositeScreen::get (screen)-> | ||
305 | 9 | - getOption ("unredirect_fullscreen_windows")->value ().b (); | ||
306 | 10 | + /* | ||
307 | 11 | + * On 0.9.7 we set this always to true, since we couldn't easily migrate | ||
308 | 12 | + * the current users to the new default because of gconf problems. | ||
309 | 13 | + * | ||
310 | 14 | + * Disabling of fullscreen application unredirection can be easily | ||
311 | 15 | + * achieved by clearing out the unredirect_match option, ie. removing | ||
312 | 16 | + * also the '(any)' part and leaving that option blank. | ||
313 | 17 | + */ | ||
314 | 18 | + unredirectFS = true; | ||
315 | 19 | |||
316 | 20 | // This should be const but CompMatch is not const-friendly. | ||
317 | 21 | CompMatch &unredirectable = CompositeScreen::get (screen)-> | ||
318 | 22 | |||
319 | 0 | 23 | ||
320 | === removed file 'debian/patches/revert_fix_933776_955035.patch' | |||
321 | --- debian/patches/revert_fix_933776_955035.patch 2012-11-12 11:57:00 +0000 | |||
322 | +++ debian/patches/revert_fix_933776_955035.patch 1970-01-01 00:00:00 +0000 | |||
323 | @@ -1,65 +0,0 @@ | |||
324 | 1 | === modified file 'plugins/scale/src/privates.h' | ||
325 | 2 | --- old/plugins/scale/src/privates.h 2012-06-21 23:10:08 +0000 | ||
326 | 3 | +++ new/plugins/scale/src/privates.h 2012-11-12 08:37:15 +0000 | ||
327 | 4 | @@ -63,8 +63,6 @@ | ||
328 | 5 | void findBestSlots (); | ||
329 | 6 | bool fillInWindows (); | ||
330 | 7 | bool layoutThumbs (); | ||
331 | 8 | - bool layoutThumbsAll (); | ||
332 | 9 | - bool layoutThumbsSingle (); | ||
333 | 10 | |||
334 | 11 | SlotArea::vector getSlotAreas (); | ||
335 | 12 | |||
336 | 13 | |||
337 | 14 | === modified file 'plugins/scale/src/scale.cpp' | ||
338 | 15 | --- old/plugins/scale/src/scale.cpp 2012-06-21 23:10:08 +0000 | ||
339 | 16 | +++ new/plugins/scale/src/scale.cpp 2012-11-12 08:37:15 +0000 | ||
340 | 17 | @@ -684,47 +684,6 @@ | ||
341 | 18 | bool | ||
342 | 19 | PrivateScaleScreen::layoutThumbs () | ||
343 | 20 | { | ||
344 | 21 | - switch (type) { | ||
345 | 22 | - case ScaleTypeAll: | ||
346 | 23 | - return layoutThumbsAll (); | ||
347 | 24 | - case ScaleTypeNormal: | ||
348 | 25 | - default: | ||
349 | 26 | - return layoutThumbsSingle (); | ||
350 | 27 | - } | ||
351 | 28 | -} | ||
352 | 29 | - | ||
353 | 30 | -bool | ||
354 | 31 | -PrivateScaleScreen::layoutThumbsAll () | ||
355 | 32 | -{ | ||
356 | 33 | - windows.clear (); | ||
357 | 34 | - | ||
358 | 35 | - /* add windows scale list, top most window first */ | ||
359 | 36 | - foreach (CompWindow *w, screen->windows ()) | ||
360 | 37 | - { | ||
361 | 38 | - SCALE_WINDOW (w); | ||
362 | 39 | - | ||
363 | 40 | - if (sw->priv->slot) | ||
364 | 41 | - sw->priv->adjust = true; | ||
365 | 42 | - | ||
366 | 43 | - sw->priv->slot = NULL; | ||
367 | 44 | - | ||
368 | 45 | - if (!sw->priv->isScaleWin ()) | ||
369 | 46 | - continue; | ||
370 | 47 | - | ||
371 | 48 | - windows.push_back (sw); | ||
372 | 49 | - } | ||
373 | 50 | - | ||
374 | 51 | - if (windows.empty ()) | ||
375 | 52 | - return false; | ||
376 | 53 | - | ||
377 | 54 | - slots.resize (windows.size ()); | ||
378 | 55 | - | ||
379 | 56 | - return ScaleScreen::get (screen)->layoutSlotsAndAssignWindows (); | ||
380 | 57 | -} | ||
381 | 58 | - | ||
382 | 59 | -bool | ||
383 | 60 | -PrivateScaleScreen::layoutThumbsSingle () | ||
384 | 61 | -{ | ||
385 | 62 | bool ret = false; | ||
386 | 63 | std::map <ScaleWindow *, ScaleSlot> slotWindows; | ||
387 | 64 | CompWindowList allWindows; | ||
388 | 65 | |||
389 | 66 | 0 | ||
390 | === removed file 'debian/patches/revert_fix_994841.patch' | |||
391 | --- debian/patches/revert_fix_994841.patch 2012-11-13 09:38:42 +0000 | |||
392 | +++ debian/patches/revert_fix_994841.patch 1970-01-01 00:00:00 +0000 | |||
393 | @@ -1,33 +0,0 @@ | |||
394 | 1 | === modified file 'scripts/run-with-xvfb.sh' | ||
395 | 2 | --- old/scripts/run-with-xvfb.sh 2012-06-27 09:39:08 +0000 | ||
396 | 3 | +++ new/scripts/run-with-xvfb.sh 2012-11-13 09:18:47 +0000 | ||
397 | 4 | @@ -8,19 +8,18 @@ | ||
398 | 5 | exit 1 | ||
399 | 6 | fi | ||
400 | 7 | |||
401 | 8 | -# The script should still pass if you have a real Xserver and no Xvfb | ||
402 | 9 | -if [ ! -z "$DISPLAY" ]; then | ||
403 | 10 | - echo "Note: You are already running an Xserver." | ||
404 | 11 | - $1 | ||
405 | 12 | -else | ||
406 | 13 | - xvfb=`which Xvfb` | ||
407 | 14 | - if [ ! -x "$xvfb" ]; then | ||
408 | 15 | - echo "Error: Xvfb does not seem to be installed." | ||
409 | 16 | - exit 1 | ||
410 | 17 | - fi | ||
411 | 18 | +xvfb=`which Xvfb` | ||
412 | 19 | +if [ ! -x "$xvfb" ]; then | ||
413 | 20 | + echo "Error: Xvfb does not seem to be installed." | ||
414 | 21 | + exit 1 | ||
415 | 22 | +fi | ||
416 | 23 | |||
417 | 24 | +if [ -z "$DISPLAY" ]; then | ||
418 | 25 | Xvfb -ac -noreset -screen 0 1024x768x16 -help 2>/dev/null 1>&2 | ||
419 | 26 | XID=`for id in 101 102 103 104 105 106 107 197 199 211 223 227 293 307 308 309 310 311 491 492 493 494 495 496 497 498 499 500 501 502 503 504 505 506 507 508 509 991 992 993 994 995 996 997 998 999 1000 1001 1002 1003 1004 1005 1006 1007 1008 1009 4703 4721 4723 4729 4733 4751 9973 9974 9975 9976 9977 9978 9979 9980 9981 9982 9983 9984 9985 9986 9987 9988 9989 9990 9991 9992 9993 9994 9995 9996 9997 9998 9999 ; do test -e /tmp/.X$id-lock || { echo $id; exit 0; }; done; echo "No Display ports available, quit."; exit 1` | ||
420 | 27 | { Xvfb -ac -noreset -screen 0 1024x768x16 :$XID -nolisten tcp -auth /dev/null >/dev/null 2>&1 & trap "kill -15 $! " 0 HUP INT QUIT TRAP USR1 PIPE TERM ; } || { echo "Gtk+Tests:ERROR: Failed to start Xvfb environment for X11 target tests."; exit 1; } | ||
421 | 28 | DISPLAY=:$XID $1 | ||
422 | 29 | +else | ||
423 | 30 | + echo "Note: You are already running an Xserver." | ||
424 | 31 | + $1 | ||
425 | 32 | fi | ||
426 | 33 | |||
427 | 34 | 0 | ||
428 | === modified file 'debian/patches/series' | |||
429 | --- debian/patches/series 2012-11-13 09:38:42 +0000 | |||
430 | +++ debian/patches/series 2013-05-07 13:14:15 +0000 | |||
431 | @@ -1,5 +1,6 @@ | |||
432 | 1 | ubuntu-config.patch | 1 | ubuntu-config.patch |
433 | 2 | ccp_plugin.patch | 2 | ccp_plugin.patch |
434 | 3 | workaround_broken_drivers.patch | 3 | workaround_broken_drivers.patch |
437 | 4 | revert_fix_933776_955035.patch | 4 | force_unredirect_enabling.patch |
438 | 5 | revert_fix_994841.patch | 5 | blacklist_precise_mesa80_intel_nouveau.patch |
439 | 6 | fix_1095001.patch | ||
440 | 6 | 7 | ||
441 | === modified symlink 'debian/patches/series.armel' | |||
442 | === target was u'series.armhf' | |||
443 | --- debian/patches/series.armel 1970-01-01 00:00:00 +0000 | |||
444 | +++ debian/patches/series.armel 2013-05-07 13:14:15 +0000 | |||
445 | @@ -0,0 +1,1 @@ | |||
446 | 1 | compiz-package-gles2.patch | ||
447 | 0 | 2 | ||
448 | === modified file 'debian/patches/workaround_broken_drivers.patch' | |||
449 | --- debian/patches/workaround_broken_drivers.patch 2012-02-10 16:36:43 +0000 | |||
450 | +++ debian/patches/workaround_broken_drivers.patch 2013-05-07 13:14:15 +0000 | |||
451 | @@ -2,7 +2,7 @@ | |||
452 | 2 | =================================================================== | 2 | =================================================================== |
453 | 3 | --- compiz-0.9.7.0~bzr2995.orig/plugins/opengl/src/screen.cpp 2012-02-09 16:32:07.000000000 +0100 | 3 | --- compiz-0.9.7.0~bzr2995.orig/plugins/opengl/src/screen.cpp 2012-02-09 16:32:07.000000000 +0100 |
454 | 4 | +++ compiz-0.9.7.0~bzr2995/plugins/opengl/src/screen.cpp 2012-02-10 17:09:17.029015287 +0100 | 4 | +++ compiz-0.9.7.0~bzr2995/plugins/opengl/src/screen.cpp 2012-02-10 17:09:17.029015287 +0100 |
456 | 5 | @@ -25,6 +25,11 @@ | 5 | @@ -25,7 +25,12 @@ |
457 | 6 | * David Reveman <davidr@novell.com> | 6 | * David Reveman <davidr@novell.com> |
458 | 7 | */ | 7 | */ |
459 | 8 | 8 | ||
460 | @@ -12,6 +12,7 @@ | |||
461 | 12 | +#include <errno.h> | 12 | +#include <errno.h> |
462 | 13 | + | 13 | + |
463 | 14 | #include "privates.h" | 14 | #include "privates.h" |
464 | 15 | #include "blacklist/blacklist.h" | ||
465 | 15 | 16 | ||
466 | 16 | #include <dlfcn.h> | 17 | #include <dlfcn.h> |
467 | 17 | @@ -83,9 +88,24 @@ | 18 | @@ -83,9 +88,24 @@ |
468 | 18 | 19 | ||
469 | === modified file 'debian/watch' | |||
470 | --- debian/watch 2009-10-20 10:34:10 +0000 | |||
471 | +++ debian/watch 2013-05-07 13:14:15 +0000 | |||
472 | @@ -1,3 +1,3 @@ | |||
473 | 1 | version=3 | 1 | version=3 |
475 | 2 | http://releases.compiz.org/([\d\.]+)[02468]/ compiz-(\d.*)\.tar\.gz | 2 | https://launchpad.net/compiz-core/+download .*/compiz-core-([0-9.]+)\.tar\.bz2 |
476 | 3 | 3 |
Approved! and thanks for backporting the changes :)