Closed Bug 930232 Opened 8 years ago Closed 2 years ago
Allow building with Min
GW on Win32
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/20100101 Firefox/19.0 (Beta/Release) Build ID: 20130215130331 Steps to reproduce: Trying to compile NSPR on Win32 using MinGW, Actual results: build failed due to MSVC++-specific function mappings (_InterlockedIncrement() e.g.). Expected results: This patch adds a test for MinGW and maps the minimal needed functions.
Jacek, could you explain how you made pratom.h work with MinGW in bug 594738? Thanks.
Attached a new and more complete patch ; the former one wasn't enough to allow some NSPR dependencies to build (NSS e.g.). Sorry for that. Details : - we now include <windows.h> here. If we don't do this, each file requiring PR_ATOM function will be required to include <windows.h> itself, which may not be portable/transparent for a developer. - MinGW doesn't have the undescored version of functions, as can be seen in : https://gitorious.org/mingw/mingw-w32api/source/240e5bbe9b18a18f3c975ba5ac923a4c537f3bb6:include/winbase.h
Attachment #821299 - Attachment is obsolete: true
Hello, any news on this ? Thanks.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Assignee: wtc → nobody
Status: ASSIGNED → NEW
QA Contact: jjones
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.