Closed Bug 945292 Opened 6 years ago Closed 6 years ago

Fix ANGLE compilation on mingw.


(Core :: Graphics, defect)

Windows 7
Not set





(Reporter: jacek, Assigned: jacek)


(Whiteboard: [qa-] leave open)


(2 files, 1 obsolete file)

Attached patch patch.diff (obsolete) — Splinter Review
The last update of ANGLE broke mingw build. I fixed it upstream:
The fix also contains fix from angle-build-case-sensitive.patch in our tree, so I removed it. I didn't create a new .patch file, because it won't be needed when updating to recent version. Let me know if you want that in tree anyways.
Attachment #8341124 - Flags: review?(bjacob)
Attachment #8341124 - Flags: review?(bjacob) → review?(jgilbert)
Attached patch partSplinter Review part. Other than using SSE2 for ImageSSE2.cpp, this avoids overrding OS_COMPILE_CFLAGS and OS_COMPILE_CXXFLAGS. The problem is that those contain arguments for generating .pp files, which are needed for mingw compilation. I don't see any harm in having MOZILLA_CLIENT defined here.
Attachment #8341153 - Flags: review?(mh+mozilla)
Comment on attachment 8341153 [details] [diff] [review] part

Review of attachment 8341153 [details] [diff] [review]:

::: gfx/angle/src/libEGL/
@@ +13,5 @@
>  # mozilla-config.h and -DMOZILLA_CLIENT automatically included and defined, so
>  # we just overwrite OS_COMPILE_C(XX)FLAGS.
> +endif

Just remove those four lines.
Attachment #8341153 - Flags: review?(mh+mozilla) → review+
OK, pushed with those removed. Thanks for review.
Whiteboard: leave open
Comment on attachment 8341124 [details] [diff] [review]

Review of attachment 8341124 [details] [diff] [review]:

We do need a record of this change in the tree, in case we don't upgrade all the way to tip.

::: gfx/angle/angle-build-case-sensitive.patch
@@ -1,5 @@
> -From: Jeff Gilbert <>
> -* * *
> -Bug 840577 - Fix ANGLE cross compilation on case sensitive OSes, relanded
> -
> -diff --git a/gfx/angle/src/libGLESv2/precompiled.h b/gfx/angle/src/libGLESv2/precompiled.h

This is a non-sensical change. You can see this patch applied if you check 'precompiled.h'.
Attachment #8341124 - Flags: review?(jgilbert) → review-
This version adds a patch file to the tree.
Attachment #8341124 - Attachment is obsolete: true
Attachment #8365572 - Flags: review?(jgilbert)
Attachment #8365572 - Flags: review?(jgilbert) → review+
Attachment #8365572 - Flags: checkin?
Attachment #8341153 - Flags: checkin+
Attachment #8365572 - Flags: checkin? → checkin+
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Whiteboard: leave open → [qa-] leave open
You need to log in before you can comment on or make changes to this bug.