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)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla7

People

(Reporter: t.matsuu, Assigned: t.matsuu)

References

Details

Attachments

(1 file)

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.
Blocks: 651858
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: nobody → t.matsuu
Keywords: checkin-needed
Status: NEW → ASSIGNED
Flags: in-testsuite-
Keywords: checkin-needed
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [inbound]
Target Milestone: --- → mozilla7
Version: unspecified → Trunk
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
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.

Attachment

General

Created:
Updated:
Size: