[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#597357: unblock: cairo/1.8.10-6



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: freeze-exception

Please unblock the latest cairo upload.

cairo (1.8.10-6) unstable; urgency=low 
.
   * debian/patches/02_iceweasel-buggy-repeat.patch:
     + Revert logic of the patch for the infamous iceweasel rendering
       problems. The only broken X11 driver that is left seems to be
       fglrx and always enabling the workaround code in cairo causes
       severe performance degradations for everybody (Closes: #594245).
       Thanks for Josselin Mouette for the patch.
       Everybody using the fglrx driver should set the MOZ_CAIRO_BUGGY_REPEAT
       environment variable, everybody else should get a correctly working
       and fast cairo again.

I’m attaching the new version of the patch.

Cheers,
-- 
 .''`.      Josselin Mouette
: :' :
`. `'  “If you behave this way because you are blackmailed by someone,
  `-    […] I will see what I can do for you.”  -- Jörg Schilling
Index: cairo/src/cairo-xlib-display.c
===================================================================
--- cairo.orig/src/cairo-xlib-display.c	2010-08-17 05:52:51.868996101 +0200
+++ cairo/src/cairo-xlib-display.c	2010-08-17 05:52:54.692995371 +0200
@@ -66,6 +66,8 @@
 
 static cairo_xlib_display_t *_cairo_xlib_display_list;
 
+static int buggy_repeat_force = -1;
+
 static void
 _cairo_xlib_remove_close_display_hook_internal (cairo_xlib_display_t *display,
 						cairo_xlib_hook_t *hook);
@@ -338,6 +340,17 @@
 	    display->buggy_repeat = TRUE;
     }
 
+    /* XXX workaround; see https://bugzilla.mozilla.org/show_bug.cgi?id=413583 */
+    if (buggy_repeat_force == -1) {
+        if (getenv("MOZ_CAIRO_BUGGY_REPEAT"))
+            buggy_repeat_force = 1;
+        else
+            buggy_repeat_force = 0;
+    }
+
+    if (buggy_repeat_force)
+        display->buggy_repeat = TRUE;
+
     display->next = _cairo_xlib_display_list;
     _cairo_xlib_display_list = display;
 

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: