Closed Bug 1008725 Opened 11 years ago Closed 10 years ago

/suite/installer/package-manifest.in needs to handle lack of mapi files when --disable-mapi is used

Categories

(SeaMonkey :: Build Config, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.29

People

(Reporter: nsITobin, Assigned: nsITobin)

Details

Attachments

(1 file)

In Bug 789787 changes were made to /mail /mailnews and /suite to handle the option to --disable-mapi for when the Microsoft MAPI headers were not present and building with VS2012.

While the patches do include fixes to allow suite to build, packaging was not updated to account for these changes.

I have ported over the changes from /mail/installer/package-manifest.in to the suite version of this file which will allow packaging to deal with not having those files.

NOTE: It was stated that suite would need this and it was indicated that it was included in the patches but apparently it wasn't.
Attached patch patch.diff β€” β€” Splinter Review
Assignee: nobody → email
Status: NEW → ASSIGNED
Attachment #8420660 - Flags: review?(bugzilla)
Comment on attachment 8420660 [details] [diff] [review]
patch.diff

Works fine!
Attachment #8420660 - Flags: review?(bugzilla) → review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/52b964909365
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.29
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: