Closed Bug 3833 Opened 26 years ago Closed 26 years ago

Apprunner -mail needs the correct URL

Categories

(Core Graveyard :: Tracking, defect, P3)

All
Mac System 8.5
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sfraser_bugs, Assigned: jefft)

Details

The url loaded when you start apprunner with the -mail flag needs to be fixed to point to resource:/res/samples/mailshell.xul
Target Milestone: M3
Putting this on M3. Hope that's OK, jeff.
This is a one line change in \mozilla\xpfe\bootstrap\nsAppRunner.cpp from urlstr = "resource:/res/samples/MailAppShell.html"; to urlstr = "resource:/res/samples/mailshell.xul";
QA Contact: 3853 → 4080
Setting QA Contact to lchiang
Hardware: Macintosh → All
Changing platform to all.
Status: NEW → ASSIGNED
This is a very simple fix as pointed out by kostello. Chris, do we want to check this in for M3?
F:\mozilla\xpfe\bootstrap>cvs diff -c nsAppRunner.cpp Index: nsAppRunner.cpp =================================================================== RCS file: /cvsroot/mozilla/xpfe/bootstrap/nsAppRunner.cpp,v retrieving revision 1.28 diff -c -r1.28 nsAppRunner.cpp *** nsAppRunner.cpp 1999/03/16 22:06:23 1.28 --- nsAppRunner.cpp 1999/03/18 16:20:48 *************** *** 180,186 **** if (NS_SUCCEEDED(rv)) { if (cmdResult && (strcmp("1",cmdResult)==0)) ! urlstr = "resource:/res/samples/MailAppShell.html"; } } if (nsnull == urlstr) --- 180,186 ---- if (NS_SUCCEEDED(rv)) { if (cmdResult && (strcmp("1",cmdResult)==0)) ! urlstr = "resource:/res/samples/mailshell.xul"; } } if (nsnull == urlstr) F:\mozilla\xpfe\bootstrap>
ok. checkin and put a cvs comment to remind about moving the the M3 branch.
If I read correctlly, I should check in to the tip, put a comment saying that the fix is intended for M3. I am not checking in to the M3 branch. Is this correct?
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Fix checked in to the tip with comment that the fix is intended for M3. Approved by chofmann.
Status: RESOLVED → VERIFIED
verified Mar190459_m3 build.
Moving all Apprunner bugs past and present to Other component temporarily whilst don and I set correct component. Apprunner component will be deleted/retired shortly.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.