Closed Bug 651861 Opened 13 years ago Closed 13 years ago

ipc/chromium compilation broken on mingw

Categories

(Core :: IPC, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jacek, Assigned: jacek)

Details

Attachments

(1 file)

Attached patch fix v1.0Splinter Review
There are a few problems with compilation on mingw. The attached patch fixes all of them (I'm not requesting review yet as I need to get one last fix committed to mingw-w64 project first). The changes are:

Makefile.in, debug_on_start.h:
Disable debug_on_start on mingw (make it depend on compiler instead of OS)

chromium-config.mk:
Make -DCOMPILER_MSVC depend on compiler instead of OS

atomicops.h:
Use MSVC version of tomicops_internals_* on mingw

atomicops_internals_x86_msvc.h:
MSC version check makes sense only on MSVC

file_path.cc, file_path.h, file_util.cc:
GCC lacks UNICODE version of ifstream constructor, but we use UNICODE paths on Windows. Workaround the problem by converting path to ASCII version.

message_loop.cc, platform_thread_win.cc:
Use MOZ_SEH_TRY/MOZ_SEH_EXCEPT instead of __try/__except to workaround GCC's lack of SEH support.

prtime.cc, time_win.cc:
Use proper macro for long long constants instead of MSC-specific syntax.

win_util.cc:
Do function pointer to pointer cast explicitly (GCC throws an error otherwise).
Attachment #527534 - Flags: review?(jones.chris.g)
Attachment #527534 - Flags: review?(jones.chris.g) → review+
Thanks for the review, patch pushed to m-c:

http://hg.mozilla.org/mozilla-central/rev/c447a4e42b62
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: