Closed
Bug 694811
Opened 13 years ago
Closed 13 years ago
Linking with VC11 fails with an undefined reference error to _IID_IAsyncOperation
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file, 1 obsolete file)
2.26 KB,
patch
|
bbondy
:
review+
|
Details | Diff | Splinter Review |
The SDK shipping with VC11 has renamed IAsyncOperation to IDataObjectAsyncCapability. This causes the link step to fail with an undefined reference error to _IID_IAsyncOperation.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Comment 2•13 years ago
|
||
Forgot to include the header file.
Attachment #567315 -
Attachment is obsolete: true
Attachment #567315 -
Flags: review?(netzen)
Attachment #567316 -
Flags: review?(netzen)
Comment 3•13 years ago
|
||
Try run for c5d82323886d is complete. Detailed breakdown of the results available here: https://tbpl.mozilla.org/?tree=Try&rev=c5d82323886d Results (out of 19 total builds): success: 15 warnings: 4 Builds available at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/eakhgari@mozilla.com-c5d82323886d
Updated•13 years ago
|
OS: Mac OS X → Windows 8
Comment 4•13 years ago
|
||
Comment on attachment 567316 [details] [diff] [review] Patch (v1) Review of attachment 567316 [details] [diff] [review]: ----------------------------------------------------------------- - Change looks good. - Verified that I get the problem building with VC2011 on Win7 and Windows SDK v7.1 and that the patch fixes it. - Verified that it doesn't cause a problem with VS2010 w/ 7.1 - Did not try other supported setups but I think it will be OK.
Attachment #567316 -
Flags: review?(netzen) → review+
Assignee | ||
Comment 5•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/0eed72fc0007
Target Milestone: --- → mozilla10
Comment 6•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0eed72fc0007
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•