Closed
Bug 174597
Opened 23 years ago
Closed 23 years ago
make palm conduit part of the installer
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.2beta
People
(Reporter: rdayal, Assigned: rdayal)
Details
Attachments
(1 file, 2 obsolete files)
|
1.80 KB,
patch
|
ssu0262
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
Palm conduit needs to be added in the installer manifest.
| Assignee | ||
Comment 1•23 years ago
|
||
| Assignee | ||
Comment 2•23 years ago
|
||
Instead of using GetCurrentDirectory this change uses GetModuleFileName and
extracts the path out to use it for finding and loading the conduit. This
enables running the Conduit Installer from a short cut even with the current
dir not set to the mozilla installation dir.
Attachment #102949 -
Attachment is obsolete: true
| Assignee | ||
Comment 3•23 years ago
|
||
Attachment #102957 -
Attachment is obsolete: true
| Assignee | ||
Comment 4•23 years ago
|
||
Hi Sean, can u review this patch, thanks.
Comment on attachment 102959 [details] [diff] [review]
adding the check for index to end the while loop
r=ssu
Attachment #102959 -
Flags: review+
Comment 6•23 years ago
|
||
Comment on attachment 102959 [details] [diff] [review]
adding the check for index to end the while loop
sr=mscott
Attachment #102959 -
Flags: superreview+
| Assignee | ||
Updated•23 years ago
|
Severity: normal → critical
Priority: -- → P2
Target Milestone: --- → mozilla1.2beta
Comment 7•23 years ago
|
||
Comment on attachment 102959 [details] [diff] [review]
adding the check for index to end the while loop
a=asa for checkin to 1.2 (on behalf of drivers)
| Assignee | ||
Comment 8•23 years ago
|
||
This is already checked in. Marking this as fixed.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•23 years ago
|
QA Contact: nbaca → meehansqa
Comment 9•23 years ago
|
||
Trunk build 2003-02-20: WinXP
Verified Fixed. This has been fixed for awhile.
Status: RESOLVED → VERIFIED
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•