Closed
Bug 886656
Opened 13 years ago
Closed 13 years ago
GTest linking fails with disable-metro 'module machine type 'x64' conflicts with target machine type 'X86'
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox24 disabled)
RESOLVED
FIXED
mozilla25
| Tracking | Status | |
|---|---|---|
| firefox24 | --- | disabled |
People
(Reporter: BenWa, Assigned: BenWa)
References
Details
Attachments
(1 file)
|
984 bytes,
patch
|
philor
:
review+
gps
:
review+
|
Details | Diff | Splinter Review |
msvcprtd.lib(MSVCP90D.dll) : fatal error LNK1112: module machine type 'x64' conflicts with target machine type 'X86'
https://tbpl.mozilla.org/php/getParsedLog.php?id=24546949&tree=Mozilla-Aurora#error1
Looks like somewhere the architecture type is getting mixed up. This error is happening on aurora where we disabled metro. Currently on central metro is enabled which disables gtest. For now gtest should be disabled completely on windows until we can fix these problems. Also there is no value in running GTest on aurora right now since we're still building up unit tests.
| Assignee | ||
Comment 1•13 years ago
|
||
Assignee: nobody → bgirard
Status: NEW → ASSIGNED
Comment 2•13 years ago
|
||
Comment on attachment 767024 [details] [diff] [review]
Disable GTest on windows
r=philor (oddly, nobody ever complains about that like they should), a=test-only, shipit!
Attachment #767024 -
Flags: review+
| Assignee | ||
Comment 3•13 years ago
|
||
Comment 4•13 years ago
|
||
Comment on attachment 767024 [details] [diff] [review]
Disable GTest on windows
Review of attachment 767024 [details] [diff] [review]:
-----------------------------------------------------------------
Just in case anyone was worried about bypassing build peer review. I hope this gets fixed soon!
Attachment #767024 -
Flags: review+
Updated•13 years ago
|
status-firefox24:
--- → disabled
| Assignee | ||
Comment 5•13 years ago
|
||
Comment on attachment 767024 [details] [diff] [review]
Disable GTest on windows
Inbound is closed.
Attachment #767024 -
Flags: checkin?
| Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Updated•13 years ago
|
Attachment #767024 -
Flags: checkin? → checkin+
Comment 6•13 years ago
|
||
Keywords: checkin-needed
Comment 7•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Updated•8 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•