Closed Bug 102570 Opened 23 years ago Closed 23 years ago

Branch landing bug for simple MAPI

Categories

(MailNews Core :: Simple MAPI, defect, P1)

x86
Windows 98
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: tiantian, Assigned: tiantian)

References

Details

(Whiteboard: [PDT+] [ETA 10.04])

Attachments

(3 files)

This is to track the branch landing process of simple MAPI.
Priority: -- → P1
A private branch has been created on 9/28. QA has been building and testing from
the branch since noon of 9/28.

(1) We need Dan Veditz and Doug Turner to review and super review the logon part
in this private branch.

Instruction provided by Rajiv:

"Instruction for using the ?MAPI_SUPP_BRANCH? for the Mapi Support feature

   *.Get the latest 0.94 branch or the the trunk and build it 

   *.Now go to the dir where u have mozilla and get the branch from Mapi, do : 

            "cvs co -rMAPI_SUPP_BRANCH mozilla/mailnews/mapi". 

        This will get the latest branch for mapi support. The branch
MAPI_SUPP_BRANCH for mapi support is NOT the full tree, it is just the mapi  dir
within mozilla/mailnews. 

   *.Go to mozilla/mailnews/mapi, build the mapi dir : 'nmake -f makefile.win'.
 This completes the building part. 

   *.Now run mozilla from your build. Go to menuitem Edit/Preferences, this will
 bring up the Preferences dialog. Go to category Mail & NewsGroups. In the
General setting section u will see the checkbox 'Use Mozilla Mail as the default
mail application.' Check the checkbox to enable the MAPI support in Mozilla and
close the Prefs dialog. 

   *.You are now ready to test the mozilla's MAPI support. Run Word, WordPad,
Powerpoint, or Excel. Open a document in the app and do File/Send from the app.
You should see the Mozilla mailnews compose window come up with your document
attached to the opened mail.

     Type in the recipients (To, Cc, etc) and send the mail."


(2) Here's testing instruction provided by Trix:

"Authorization Checklist location:

http://eclient.mcom.com/qa/emojo/index.html


Instructions:

1. Run testcase and check the Pass/Fail checkbox accordingly.

2. Enter results in Results column

3. Enter your name and email address for each checklist.

4. If you wish to start over, press the Reset button.

5. After form completion, click on the Submit button to send your results.

6. Please route questions to trix@netscape.com ."

Depends on: 102382, 102387, 102392, 102560
Keywords: nsbranch+
Whiteboard: PDT
Attached file siimple mapi testing checklist (deleted) —
attached the testcases for testing outside our firewall. if testing is being
done inside our firewall, please goto: 

http://eclient.mcom.com/qa/emojo/tests/smapi/smapi.html

and send the form when completed.
Hi:

In case you wonder why we did not attach testing binary to this bug, here's the 
answer.

Trix has been trying to attach the testing binary to this bug, but it's too 
big, the bug system does not accept it. Hopefully, people can follow the build 
instruction and get a preview of this feature.

We'll try to make the testing binary available from outside firewall tomorrow.


we have standard ways of making test builds available, all you have to do is
ask. Talk to someone on the build team, maybe granrose@netscape.com
leaf is on it (i.e. helping with public posting of Mozilla 094 branch with
simple mapi), with support from asa, mscott and others.
Whiteboard: PDT → [PDT] [ETA 10.02]
removing the dependency on 102560, it is targetted for 0.9.6 where as 102570 is
branch landing for 0.9.4.
No longer depends on: 102560
Copied the query to find simple MAPI bugs below.

http://bugzilla.mozilla.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_s
tatus=REOPENED&email1=&emailtype1=substring&emailassigned_to1=1&email2=&emailtyp
e2=substring&emailreporter2=1&bugidtype=include&bug_id=&changedin=&votes=&chfiel
dfrom=&chfieldto=Now&chfieldvalue=&product=MailNews&component=Simple+MAPI&short_
desc=&short_desc_type=allwordssubstr&long_desc=&long_desc_type=allwordssubstr&bu
g_file_loc=&bug_file_loc_type=allwordssubstr&status_whiteboard=&status_whiteboar
d_type=allwordssubstr&keywords=&keywords_type=anywords&field0-0-0=noop&type0-0-0
=noop&value0-0-0=&cmdtype=doit&namedcmd=pdt%2B&newqueryname=&order=Reuse+same+so
rt+as+last+time
http://bugzilla.mozilla.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_s
tatus=REOPENED&email1=&emailtype1=substring&emailassigned_to1=1&email2=&emailtyp
e2=substring&emailreporter2=1&bugidtype=include&bug_id=&changedin=&votes=&chfiel
dfrom=&chfieldto=Now&chfieldvalue=&product=MailNews&component=Simple+MAPI&short_
desc=&short_desc_type=allwordssubstr&long_desc=&long_desc_type=allwordssubstr&bu
g_file_loc=&bug_file_loc_type=allwordssubstr&status_whiteboard=&status_whiteboar
d_type=allwordssubstr&keywords=&keywords_type=anywords&field0-0-0=noop&type0-0-0
=noop&value0-0-0=&cmdtype=doit&namedcmd=pdt%2B&newqueryname=&order=Reuse+same+so
rt+as+last+time


Trying again.
How about just clicking the "Show Dependency Tree" link above?
http://bugzilla.mozilla.org/showdependencytree.cgi?id=102570
No longer depends on: 102382, 102387, 102389, 102392
Status summary: (1) Test build and test plands has been made available to
Mozilla community and internal users. (2)Invitation for testing has been mailed
to cpd-all, and mailnews news group.

In progress:

(1)Trix: verify behaviour in 4.x. Provide summary of difference between 4.x and 6.x.
(2) Trix/Rajiv: commercial build with sMAPI feature
(3) Krishna: improve the dll copy solution, get r/sr stamp for log on part
(4) Rajiv: produce patch for the whole private branch, review the dll copy solution.
(5) Srilatha: create patch for dll copy solution, and get r/sr
(6) Tiantian: help with testing, get answer from Jeff with regard to user
requirement of platforms.


Whiteboard: [PDT] [ETA 10.02] → [PDT] [ETA 10.04]
Attached patch patch for packaging (deleted) — Splinter Review
Comment on attachment 51824 [details] [diff] [review]
patch for packaging

r=dveditz

In the future please provide more context with patches.
Attachment #51824 - Flags: review+
Comment on attachment 51810 [details] [diff] [review]
patch with the complete Partial Simple Mapi support code for branch landing

There are a lot of things that need to be restructured before
landing on the trunk, but for the branch r=dveditz

Not part of this landing, but the MAPI resources being delivered on windows
only break one of the l10n team's goals of platform-independent
language packs. Please work with Tao on this.
Comment on attachment 51810 [details] [diff] [review]
patch with the complete Partial Simple Mapi support code for branch landing

There are a lot of things that need to be restructured before
landing on the trunk, but for the branch r=dveditz

Not part of this landing, but the MAPI resources being delivered on windows
only break one of the l10n team's goals of platform-independent
language packs. Please work with Tao on this.
Attachment #51810 - Flags: review+
Adding 102798 as a dependency - a security issue that needs to be addressed
before landing.
Depends on: 102798
No longer depends on: 102798
No longer depends on: 95122
No longer depends on: 95113, 95116, 95117, 95121, 99526, 102308
Comment on attachment 51810 [details] [diff] [review]
patch with the complete Partial Simple Mapi support code for branch landing

for the branch only.
Attachment #51810 - Flags: superreview+
pls check this into the branch - PDT+
Whiteboard: [PDT] [ETA 10.04] → [PDT+] [ETA 10.04]
checked into the branch
Fixed and checked into the 0.9.4 branch, including the security warning dialog.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
The MAPI branch has not landed on the trunk, but perhaps you would like to open
a new bug for that instead of reopening this one.
tiantian or trix, can you open a new bug for landing on the trunk?
verified landing
Status: RESOLVED → VERIFIED
The MAPI branch has only landed on the 094 branch and not the trunk. I don't see
another bug for the trunk so this one's not done.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
OK. I opened the trunk landing bug. bug 104672

Marked as fixed for this one.
Status: REOPENED → RESOLVED
Closed: 23 years ago23 years ago
Resolution: --- → FIXED
correcting to verified status again.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: