Closed Bug 949578 Opened 7 years ago Closed 3 years ago
.h uses atlbase .h
This is problematic for people using free versions of the tools
But it wouldn't be problematic if bug 850890 were fixed (and the documentation updated to mention that you need WDK7.1 when building with VC*Express). My understanding (though I haven't investigated) is that CEHHelper.h must use atlbase.h. If that's the case, the workarounds for this issue are: Download the WDK7.1 and update your INCLUDE and LIB env vars appropriately -OR- --disable-metro
It's not required. We would have to migrate all of our CString uses over to buffers in the CEH and test harness. (Which I'd prefer we *not* do because I really like having use of ATL templates. :))
Removing an extra step for contributors to contribute would be nice. I mentioned that we should remove this in #windev a while ago but there's no rush since metro building is disabled by default for them
This doesn't block rollout.
No longer blocks: metrov1backlog
Mass close of bugs in obsolete product https://bugzilla.mozilla.org/show_bug.cgi?id=1350354
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.