Bug 2617 - Pixman 0.19.2 & Cairo 1.10.0
Summary: Pixman 0.19.2 & Cairo 1.10.0
Status: RESOLVED FIXED
Alias: None
Product: buildroot
Classification: Unclassified
Component: Outdated package (show other bugs)
Version: unspecified
Hardware: PC Linux
: P5 enhancement
Target Milestone: ---
Assignee: Thomas Petazzoni
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-20 15:24 UTC by Paul Burton
Modified: 2012-02-03 13:33 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:


Attachments
Update pixman to 0.19.2 (899 bytes, application/octet-stream)
2010-09-20 15:24 UTC, Paul Burton
Details
Update cairo to 1.10.0 (9.58 KB, patch)
2010-09-20 15:24 UTC, Paul Burton
Details
Update pixman to 0.19.2, take 2 (2.36 KB, patch)
2010-09-21 09:38 UTC, Paul Burton
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Burton 2010-09-20 15:24:14 UTC
Created attachment 2509 [details]
Update pixman to 0.19.2

Updates pixman & cairo to their current releases. This also exposes the cairo option to disable some floating point math - since it is actually made use of in this release.
Comment 1 Paul Burton 2010-09-20 15:24:40 UTC
Created attachment 2515 [details]
Update cairo to 1.10.0
Comment 2 Paul Burton 2010-09-21 09:38:39 UTC
Created attachment 2521 [details]
Update pixman to 0.19.2, take 2

This is a slightly cleaner version of the pixman patch in that it updates the no tests patch against the current version too.
Comment 3 Thomas Petazzoni 2010-12-13 22:56:48 UTC
Paul,

Could you give a few more details on why the new patch cairo-1.10.0-packttstructs.patch is necessary ? The description at the top of the patch doesn't really explain the why.

Thanks!
Comment 4 Paul Burton 2010-12-14 09:38:21 UTC
(In reply to comment #3)
> Paul,
> 
> Could you give a few more details on why the new patch
> cairo-1.10.0-packttstructs.patch is necessary ? The description at the top of
> the patch doesn't really explain the why.
> 
> Thanks!

Sure, essentially if you look at src/cairo-truetype-subset.c lines 98-111, you can see that the size of the structs must match the sum of the size of their fields, ie. have no padding, because these structs are being read from font files which obviously aren't padded to word boundaries on disk. At least with the compiler I'm using that was not the case by default, but the packed attribute ensures that padding isn't added and the checks pass.

This should probably be reported upstream, I'll get to that later today.
Comment 5 Thomas Petazzoni 2012-02-03 13:33:00 UTC
pixman is now at version 0.24.2 and cairo at version 1.10.2, so this bug can be closed.