|
Bugzilla – Full Text Bug Listing |
| Summary: | fb patch for repeating pictures bug, required for better cairo, OOo cairo canvas performance | ||
|---|---|---|---|
| Product: | [openSUSE] SUSE LINUX 10.0 | Reporter: | Egbert Eich <eich> |
| Component: | X.Org | Assignee: | E-mail List <gnome-bugs> |
| Status: | RESOLVED INVALID | QA Contact: | Stefan Dirsch <sndirsch> |
| Severity: | Normal | ||
| Priority: | P5 - None | CC: | eich, gnome-bugs, ro |
| Version: | Preview 3 | ||
| Target Milestone: | --- | ||
| Hardware: | Other | ||
| OS: | All | ||
| URL: | https://bugs.freedesktop.org/show_bug.cgi?id=3566 | ||
| Whiteboard: | |||
| Found By: | Other | Services Priority: | |
| Business Priority: | Blocker: | --- | |
| Marketing QA Status: | --- | IT Deployment: | --- |
| Attachments: | cairo patch | ||
|
Description
Egbert Eich
2005-08-03 10:02:32 UTC
I am attaching patch for cairo package. This patch will be valid once suse xorg server will have patch in https://bugs.freedesktop.org/show_bug.cgi?id=3566 applied. There might be better way of identifing that. Right now it changes the test for buggy xorg server, assuming that xorg 6.8.2 is patched. Ideally we might indicate it in version or vendor string, but it doesn't seem very clean to me. Any better idea, or is it OK that way? Created attachment 44659 [details]
cairo patch
Cairo patch is prepared for submission. Please confirm, that patch for Xorg will be accepted for SuSE Linux 10.0. Stefan Dirsch is on vacation. Can you apply the patch for beta1? Or can I do it? patch added, xorg-x11 submitted to stable Cairo patch submitted. Closing. I have bugreports for b3/b4 about image scaling in OOo cairo canvas. It works better, but not completely right. So it will be better to remove at least the cairo patch so that cairo fallbacks to the workarounds again. cairo patch --> cairo maintainer egbert, any comment? What would you like me to comment on? I could comment on the patch for X.Org - if I could see it. However the attachment to the ticket https://bugs.freedesktop.org/show_bug.cgi?id=3566 now point to Matthias Hopf's gpg key instead of the patch. However the patch to fb has been reviewed by Keith - the author of the code - and applied to CVS. Therefore I guess it's OK. #10 Are we shipping this patch yet? #2 Radek does this patch still valid for cairo 1.0.0? > #10 Are we shipping this patch yet?
Yes, this patch is applied for 10.0.
--> /work/SRC/all/BASE/xorg-x11/p_bug100469.diff
I think the Xorg patch doesn't fix the problem completely (see my previous comment), so I would not recommend patching cairo anymore. Marking as invalid as per last comment. OK. In this case we need to remove the patch. Submitted for STABLE: Wed Jan 25 11:50:29 CET 2006 - sbrabec@suse.cz - Removed Xlib version fb repeating picture bug check (#100469). |