Closed
Bug 1365047
Opened 7 years ago
Closed 7 years ago
MinGW build breaks with error: ‘eDllBlocklistInitFlagIsChildProcess’ was not declared
Categories
(Core :: General, defect)
Core
General
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.
Assignee | ||
Comment 1•7 years ago
|
||
I'm wondering if we can just turn on the DLL Blocklist for Mingw.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•7 years ago
|
||
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)
Comment 4•7 years ago
|
||
(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)
Comment 5•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Attachment #8867883 -
Flags: review?(nfroyd)
Assignee | ||
Comment 6•7 years ago
|
||
Alright browser starts up and is usable for me, so I think this is good to go.
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8867883 [details]
Bug 1365047 Turn on the Windows DLL Blocklist in MinGW
https://reviewboard.mozilla.org/r/139418/#review151530
Attachment #8867883 -
Flags: review+
Comment 8•7 years ago
|
||
Comment on attachment 8867883 [details]
Bug 1365047 Turn on the Windows DLL Blocklist in MinGW
I'll take this.
Attachment #8867883 -
Flags: review?(nfroyd)
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Updated•7 years ago
|
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
Comment 10•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•6 years ago
|
Assignee: nobody → tom
You need to log in
before you can comment on or make changes to this bug.
Description
•