If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

File op: fileMove logs inconsistent info to Install.log when successful

VERIFIED FIXED

Status

Core Graveyard
Installer: XPInstall Engine
P3
normal
VERIFIED FIXED
18 years ago
2 years ago

People

(Reporter: Jimmy Lee, Assigned: Sean Su)

Tracking

Trunk
All
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
Build: 10/18/99 SeaMonkey build

1. From http://jimbob/trigger2.html, click drop-down button and select
   a_fileop_filemove.  Click Trigger case button.

startInstall("Acceptance: a_fileop_filemove", "acceptance", "1.1.1.1", 0);
f = getFolder("Program");
g = getFolder("Temporary");
addFile("fileop_filemove", "2.2.2.2", jarSrc, f, jarSrc, true);
fileSource = f + jarSrc;
fileTarget = g;
err1 = fileMove(fileSource, fileTarget);
logComment("Path to fileSource = " + fileSource);
logComment("Path to fileTarget = " + fileTarget);
logComment("fileMove returns = " + err1);

RESULT:
The installation is successful.  The Install.log returns:

---------------------------------------------------------------------------
http://jimbob/jars/a_fileop_filemove.xpi     --     10/19/1999 13:50:06
---------------------------------------------------------------------------

     Acceptance: a_fileop_filemove
     -----------------------------

     ** Path to fileSource = C:\Program Files\moz1018\x86rel\smrtupdt.txt
     ** Path to fileTarget = C:\TEMP\
     ** fileMove returns = 0
     Item [1/2]	Installing: C:\Program Files\moz1018\x86rel\smrtupdt.txt
     Item [2/2]	Move File: C:\TEMP\smrtupdt.txt to C:\TEMP\

     Install completed successfully
     Finished Installation  10/19/1999 13:50:11

EXPECTED RESULT:
The line in question is:
     Item [2/2]	Move File: C:\TEMP\smrtupdt.txt to C:\TEMP\

It seems that it should be similar to:
     Item [2/2]	Move File: C:\Program Files\moz1018\x86rel\smrtupdt.txt to
                           C:\TEMP\

Updated

18 years ago
Assignee: cathleen → ssu
(Assignee)

Updated

18 years ago
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 1

18 years ago
I overhauled the fileop functions to better detect and deal with errors.  This
is now fixed.
(Reporter)

Updated

18 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 2

18 years ago
Build 1999-10-27-09-M11(WIN), 1999-10-27-08-M11(MAC), 1999-10-27-08-M11(LINUX)

Looks good.

Comment 3

18 years ago
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.