Closed
Bug 20043
Opened 25 years ago
Closed 25 years ago
addFile() returns bogus -1 when installing older version
Categories
(Core Graveyard :: Installer: XPInstall Engine, defect, P3)
Tracking
(Not tracked)
M12
People
(Reporter: dveditz, Assigned: dveditz)
Details
Both addFile() and addDirectory() return version compare values -4, .. -1 when the script tries to install a file with an older version than an existing file and the force flag is not set. It is correct that the older file is not installed, but it should not be reported as an error to the script.
Assignee | ||
Updated•25 years ago
|
Assignee: cathleen → dveditz
Target Milestone: M12
Assignee | ||
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 1•25 years ago
|
||
You can now attempt to install an older component and it will correctly be skipped rather than return a bogus error.
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall Engine
Updated•25 years ago
|
Status: RESOLVED → REOPENED
Comment 4•25 years ago
|
||
not fixed. retested on build 2000011008-M13 (linux, nt and mac). returns -208 (invalid parameter) in logfile: ** ERROR (-208): Skipping: (null) Install script aborted. steps: 1) from http://jimbob/trigger2.html, select f_addsubcomp from dropdown. press trigger case. This installs version 1.0.1.1 in reg package f_addsubcomp. 2) then select f_addsubcomp_vi_no_update from dropdown. Script has older version (1.0.0.0) w/ force flag = false. Note: In 4 separate tests, I tested lower major, minor, version and build components of the version string, and in each case, it returns -208. (in each case, the .xpi included an earlier test file and it correctly did not install it). Also, I tested compareVersion forms of InstallTrigger object and they were returning expected values for each case.
Assignee | ||
Updated•25 years ago
|
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: FIXED → DUPLICATE
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•