Closed Bug 1490566 Opened 6 years ago Closed 6 years ago

mingw-clang x86 build does not configure because clang does not support --enable-stdcall-fixup

Categories

(Firefox Build System :: General: Unsupported Platforms, defect, P5)

defect

Tracking

(firefox-esr6064+ fixed, firefox64 fixed)

RESOLVED FIXED
mozilla64
Tracking Status
firefox-esr60 64+ fixed
firefox64 --- fixed

People

(Reporter: tjr, Assigned: tjr)

References

Details

Attachments

(1 file)

No description provided.
clang doesn't recognize --enable-stdcall-fixup as an option. We needed this for the mingw-gcc build (added in Bug 787653) because gcc had difficulty dealing with Angle's symbol linking
Comment on attachment 9009735 [details] Bug 1490566 Do not pass --enable-stdcall-fixup to clang in a x86 Windows build r=froydnj Nathan Froyd [:froydnj] has approved the revision.
Attachment #9009735 - Flags: review+
Keywords: checkin-needed
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/a52a76a43b14 Do not pass --enable-stdcall-fixup to clang in a x86 Windows build r=froydnj
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Comment on attachment 9009735 [details] Bug 1490566 Do not pass --enable-stdcall-fixup to clang in a x86 Windows build r=froydnj [ESR Uplift Approval Request] If this is not a sec:{high,crit} bug, please state case for ESR consideration: Build patch needed to get Tor's mingw-clang build running on the esr60 branch. User impact if declined: We won't be able to run mingwclang in automation and Tor will need to apply this patch themselves Fix Landed on Version: 64.0a1 / 20180919123806 Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): Only affects the mingw-clang build String or UUID changes made by this patch:
Attachment #9009735 - Flags: approval-mozilla-esr60?
Comment on attachment 9009735 [details] Bug 1490566 Do not pass --enable-stdcall-fixup to clang in a x86 Windows build r=froydnj OK for ESR60 uplift, support for (downstream) builds
Attachment #9009735 - Flags: approval-mozilla-esr60? → approval-mozilla-esr60+
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: