Closed Bug 789787 Opened 7 years ago Closed 6 years ago
Thunderbird build with Windows SDK 8
.0 fails because MAPI headers are missing
A Thunderbird build using the Windows SDK 8.0 fails because the MAPI headers aren't found. https://connect.microsoft.com/VisualStudio/feedback/details/728601/visual-c-mapiutil-h-and-xcmc-h-are-missing-from-platform-sdk-included-with-vc11-beta
Installing the MAPI headers separately seems to be the only option. I've hacked the install path into INCLUDE locally. http://www.microsoft.com/download/en/details.aspx?displaylang=en&id=12905 I couldn't find the path anywhere in the registry, so the reg trick we use to locate the DirectX SDK won't work. This sucks. We'll probably have to hardcode the default path and provide a configure option to set it.
We should check mapix.h in configure.in
Component: Simple MAPI → Build Config
Assignee: nobody → m_kato
Comment on attachment 721060 [details] [diff] [review] Part 1. Build config change (detect MAPI header and add --disable-mapi) Windows 8 SDK has no MAPI headers. Even if no MAPI header, we should support it with --disable-mapi. Also we can download MAPI header from microsoft.com.
Comment on attachment 721062 [details] [diff] [review] Part 2. can build mailnews if no mapi header If no MAPI header in build environment, we should support it even if MSVC.
Comment on attachment 721060 [details] [diff] [review] Part 1. Build config change (detect MAPI header and add --disable-mapi) Hi Makato, could you refresh this now that bug 846540 has landed - you need to move the changes to mail/configure.in and suite/configure.in. I'll be happy to review both patches - I can review them as one.
Comment on attachment 737783 [details] [diff] [review] Part 1. Build config change (detect MAPI header and add --disable-mapi) I need update again.
Comment on attachment 737785 [details] [diff] [review] Part 2. can build mailnews if no mapi header Sorry, I forget update suite/*
Comment on attachment 738367 [details] [diff] [review] Part 1. build config change sorry, I forget udapte suite/*
add --disable-mapi flag
Comment on attachment 738368 [details] [diff] [review] Part 1. build config change As this is just manually disabling, I think that's fine.
Attachment #738368 - Flags: review?(mbanner) → review+
and here's the unbitrotted patch for you.
I'm fine with this patch as well (attaching unbitrotted version).
I've just landed this as it'll help people struggling with the mapi headers. https://hg.mozilla.org/comm-central/rev/a5ad797c274a https://hg.mozilla.org/comm-central/rev/bfe4e360ee05
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
Doesn't Suite need this bit as well? https://hg.mozilla.org/comm-central/rev/bfe4e360ee05#l1.12
(In reply to Philip Chee from comment #19) > Doesn't Suite need this bit as well? > https://hg.mozilla.org/comm-central/rev/bfe4e360ee05#l1.12 https://hg.mozilla.org/comm-central/rev/a5ad797c274a has buildconfig change for suite too.
Where do the headers need to go? I'm assuming they aren't being looked for in C:\Office 2010 Developer Resources\Outlook 2010 MAPI Headers
Never mind. If anyone else ends up here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Windows_Prerequisites
You need to log in before you can comment on or make changes to this bug.