Closed Bug 523562 Opened 10 years ago Closed 10 years ago

Fix building of SeaMonkey Mac (and possibly part of Windows) on trunk following bug 516213 (Freshen WebGL implementation)

Categories

(SeaMonkey :: Build Config, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a1

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(2 files)

Bug 516213 means that we now require to link against OpenGL on windows & mac for static builds.

The attached patch will do that for trunk builds, and take account of 1.9.2 just in case we go there (its a copy & paste of what I fixed for Thunderbird minus a 1.9.1 ifdef).

I'll land this once we branch (hence no 1.9.1 ifdef), and if necessary it can be used to check the triggering of builders as it is low risk.
Attachment #407501 - Flags: review?(kairo)
Comment on attachment 407501 [details] [diff] [review]
The fix
[Checkin: Comment 2]

Thanks, Mark, this has been on my TODO for "once I'm out of all this 2.0 final trouble" for some time now, as I'd really like to get trunk nightlies to go green again.
I'm somewhat eager to see it land, but agree that it's probably best to wait for branching, given it comes up tomorrow :)
Attachment #407501 - Flags: review?(kairo) → review+
Checked in: http://hg.mozilla.org/comm-central/rev/054cff11b1e1
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
Depends on: 516858
Flags: in-testsuite-
Comment on attachment 407836 [details] [diff] [review]
(Bv1) Don't re-add obsolete 'mac' MOZ_WIDGET_TOOLKIT tests
[Checkin: See comment 5]

r=me for the actual code change, but please leave all the empty lines in, it's more readable with them than without them.
Attachment #407836 - Flags: review?(kairo) → review+
Attachment #407501 - Attachment description: The fix → The fix [Checkin: Comment 2]
Attachment #407836 - Attachment description: (Bv1) Don't re-add obsolete 'mac' MOZ_WIDGET_TOOLKIT tests → (Bv1) Don't re-add obsolete 'mac' MOZ_WIDGET_TOOLKIT tests [Checkin: See comment 5]
Comment on attachment 407836 [details] [diff] [review]
(Bv1) Don't re-add obsolete 'mac' MOZ_WIDGET_TOOLKIT tests
[Checkin: See comment 5]


http://hg.mozilla.org/comm-central/rev/53d99e4b5403
Bv1, with comment 4 suggestion(s).
OS: Mac OS X → All
You need to log in before you can comment on or make changes to this bug.