Linking with VC11 fails with an undefined reference error to _IID_IAsyncOperation

RESOLVED FIXED in mozilla10

Status

()

Core
Widget: Win32
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
mozilla10
x86
Windows 8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

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.
Created attachment 567315 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #567315 - Flags: review?(netzen)
Created attachment 567316 [details] [diff] [review]
Patch (v1)

Forgot to include the header file.
Attachment #567315 - Attachment is obsolete: true
Attachment #567315 - Flags: review?(netzen)
Attachment #567316 - Flags: review?(netzen)
Blocks: 687213

Comment 3

6 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
OS: Mac OS X → Windows 8
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+
http://hg.mozilla.org/integration/mozilla-inbound/rev/0eed72fc0007
Target Milestone: --- → mozilla10
https://hg.mozilla.org/mozilla-central/rev/0eed72fc0007
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.