Closed
Bug 667286
Opened 14 years ago
Closed 14 years ago
cairo.h should be included as "cairo.h" instead of "cairo/cairo.h"
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
VERIFIED
FIXED
mozilla7
People
(Reporter: t.matsuu, Assigned: t.matsuu)
References
Details
Attachments
(1 file)
2.33 KB,
patch
|
bas.schouten
:
review+
|
Details | Diff | Splinter Review |
After fixing bug 651858, the build failed with system cairo.
Some new files in gfx/2d include cairo.h as "cairo/cairo.h"
We should include cairo.h as "cairo.h" and add $MOZ_CAIRO_CFLAGS into gfx/2d/Makefile.in. It's the same manner as the other files which include cairo.h.
Assignee | ||
Comment 1•14 years ago
|
||
Attachment #542017 -
Flags: review?(roc)
Comment 3•14 years ago
|
||
Comment on attachment 542017 [details] [diff] [review]
Include cairo.h as "cairo.h" instead of "cairo/cairo.h"
Review of attachment 542017 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #542017 -
Flags: review?(roc) → review+
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → t.matsuu
Keywords: checkin-needed
Assignee | ||
Updated•14 years ago
|
Status: NEW → ASSIGNED
Comment 4•14 years ago
|
||
Flags: in-testsuite-
Keywords: checkin-needed
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [inbound]
Target Milestone: --- → mozilla7
Version: unspecified → Trunk
Comment 5•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Comment 7•14 years ago
|
||
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a2) Gecko/20110706 Firefox/7.0a2
Verified fixed - detailed by the changeset in comment 5
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•