Closed
Bug 602533
Opened 14 years ago
Closed 14 years ago
Update required pixman version to 0.19.2
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(blocking2.0 final+)
RESOLVED
FIXED
mozilla2.0b8
Tracking | Status | |
---|---|---|
blocking2.0 | --- | final+ |
People
(Reporter: t.matsuu, Assigned: t.matsuu)
Details
Attachments
(1 file)
533 bytes,
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
"pixman_image_get_component_alpha" which is implemented in pixman-0.19 is called by gfx/cairo/cairo/src/cairo-surface.c. So we need to update required pixman version to 0.19.2 which is the minimum distributed version at http://cairographics.org/releases/.
Attachment #481520 -
Flags: review?(vladimir)
blocking2.0: --- → ?
Updated•14 years ago
|
blocking2.0: ? → final+
Updated•14 years ago
|
Assignee: nobody → t.matsuu
Comment on attachment 481520 [details] [diff] [review] Update required pixman version to 0.19.2 err.. this is the right thing to do for sure, but having in-tree cairo calling out to system pixman is quite broken!
Attachment #481520 -
Flags: review?(vladimir) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Can I add "checkin-needed"? Anyway "--enable-system-pixman" does not work before/after the patch will be applied. However I think we can get more appropriate feed back from developers if the patch is applied.
Wouldn't it make more sense to upgrade straight to 0.19.4? Since it brings the following perf improvements: - More ARM NEON fast paths [Siarhei Siamashka] - Faster NEAREST scaling [Siarhei] - Faster BILINEAR scaling [Soren Sandmann] - Faster glyph rendering [Soren] The first three are important for fennec and the last for linux as were leaking a lot of users to chromium because of perf. Fennec & the linux build of Firefox could use all the help it can get ATM in terms of speed.
Distros can choose to use 0.19.4, we will just require that they use 0.19.2 or later.
Assignee | ||
Comment 5•14 years ago
|
||
(In reply to comment #3) As Kyle said, distros "can" choose to use 0.19.4. This bug fixes "minimum required" pixman version. If mozilla products require new functions or APIs which are implemented after the release of 0.19.2, we have to set minimum required version to 0.19.4. If pixman-0.19.4 only have performance improvements and bug fixes, we should keep minimum required version to 0.19.2. As far as I have checked the release announcement of pixman-0.19.4, there is no new functions or APIs. http://lists.freedesktop.org/archives/pixman/2010-September/000571.html So I'll add "checkin-needed" with attachment 481520 [details] [diff] [review].
This looks ready to land.
Keywords: checkin-needed
Whiteboard: [need landing]
Assignee | ||
Updated•14 years ago
|
Attachment #481520 -
Flags: approval2.0?
Comment on attachment 481520 [details] [diff] [review] Update required pixman version to 0.19.2 This blocks, it doesn't need approval.
Attachment #481520 -
Flags: approval2.0?
Comment 8•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/81da4980adaa
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [need landing]
Target Milestone: --- → mozilla2.0b8
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•