Closed Bug 1365047 Opened 7 years ago Closed 7 years ago

MinGW build breaks with error: ‘eDllBlocklistInitFlagIsChildProcess’ was not declared

Categories

(Core :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: tjr, Assigned: tjr)

References

(Blocks 1 open bug)

Details

(Whiteboard: [tor])

Attachments

(1 file)

Bug 1363844 (https://hg.mozilla.org/mozilla-central/rev/7b2c34b67962) added a new constant eDllBlocklistInitFlagIsChildProcess. This constant is used in mozglue/build/WindowsDllBlocklist.cpp but it is only defined in mozglue/build/WindowsDllBlocklist.h when #if defined(_MSC_VER) && (defined(_M_IX86) || defined(_M_X64)) MinGW doesn't do that.
I'm wondering if we can just turn on the DLL Blocklist for Mingw.
boklm: I don't have a working MinGW or Tor build environment right now. Do you think you could test this patch and see if it works in Tor Browser? If so I'll try and land it.
Flags: needinfo?(boklm)
(In reply to Tom Ritter [:tjr] from comment #3) > boklm: I don't have a working MinGW or Tor build environment right now. Do > you think you could test this patch and see if it works in Tor Browser? If > so I'll try and land it. I have been able to build Tor Browser using this patch (rebased on esr52). Do you want me to upload the build somewhere, or just being able to build is enough?
Flags: needinfo?(boklm)
Attachment #8867883 - Flags: review?(nfroyd)
Alright browser starts up and is usable for me, so I think this is good to go.
Attachment #8867883 - Flags: review+
Comment on attachment 8867883 [details] Bug 1365047 Turn on the Windows DLL Blocklist in MinGW I'll take this.
Attachment #8867883 - Flags: review?(nfroyd)
Keywords: checkin-needed
Pushed by aklotz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1a6a5662822d Turn on the Windows DLL Blocklist in MinGW r=aklotz
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee: nobody → tom
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: