VC11 cannot compile nsWindow.cpp because it #includes <pbt.h>

RESOLVED FIXED in mozilla10

Status

()

Core
Widget
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)

This header was added in bug 464091.  We no longer support Windows CE, and on other versions of Windows, #including windows.h is enough.  The SDK headers shipping with VC11 no longer include this file.
Created attachment 567313 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #567313 - Flags: review?(netzen)
Blocks: 687213
Are you using Windows SDK 8.0 by the way or 7.0|7.1?
I assume you ar eusing Windows SDK 8 because the file is already included in 7.0 and 7.1.
OS: Mac OS X → Windows 8
Comment on attachment 567313 [details] [diff] [review]
Patch (v1)

Review of attachment 567313 [details] [diff] [review]:
-----------------------------------------------------------------

- 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 #567313 - Flags: review?(netzen) → review+
(In reply to Brian R. Bondy [:bbondy] from comment #2)
> Are you using Windows SDK 8.0 by the way or 7.0|7.1?

Sorry for the late reply.  I'm getting the problem with the SDK version shipping with VC2011 (I believe they call it 8.0).
http://hg.mozilla.org/integration/mozilla-inbound/rev/34bf415c96bb
Target Milestone: --- → mozilla10
You can tell by the way when you run start-msvc11.bat it will output something like:
Windows SDK version: 7.1
Also just out of curiosity are you using a 64bit machine?

If so does the developer preview put anything inside your Program files directory? Or only Program Files (x86)?

For me:
C:\Program Files\Microsoft SDKs\Windows 
has v7.0 and v7.1 folders

But:
C:\Program Files (x86)\Microsoft SDKs\Windows has
has v7.0A, v8.0 and v8.0A folders
(In reply to Brian R. Bondy [:bbondy] from comment #8)
> Also just out of curiosity are you using a 64bit machine?

Yep, a 64-bit VM.

> If so does the developer preview put anything inside your Program files
> directory? Or only Program Files (x86)?
> 
> For me:
> C:\Program Files\Microsoft SDKs\Windows 
> has v7.0 and v7.1 folders
> 
> But:
> C:\Program Files (x86)\Microsoft SDKs\Windows has
> has v7.0A, v8.0 and v8.0A folders

As far as I can tell, everything goes under Program Files (x86).
https://hg.mozilla.org/mozilla-central/rev/34bf415c96bb
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.