14902 - Version Registry: Duplicate node appears after patching

Status: RESOLVED WONTFIX

(Core Graveyard :: Installer: XPInstall Engine, defect, P3)

Description

[Jimmy Lee]

Build: 9/24/99 SeaMonkey build

1. From http://jimbob/trigger2.html, click drop-down button and select
   f_addwinpatch_exe and click Trigger case button to install file to be
   patched
2. From http://jimbob/trigger2.html, click drop-down button and select
   f_winpatch_exe and click Trigger case button to patch

var regName = "f_winpatch.exe";
var jarSrc = "charmap.exe.gdf";
var vi = "1.0.1.11";

Install.StartInstall("Functional: f_winpatch_exe", "f_winpatch_exe", "1.0.1.11",
0);
f = Install.GetFolder("Program", "f_winpatch_exe");
err = Install.Patch(regName, vi, jarSrc, f, "clock.exe", true);
Install.LogComment("Install Patch = " + err);

RESULT:
File on disk appears to be patched correctly.  The Install.log is accurate in
describing the patching.  A dump of the Version registry shows the following:

Subtree Dump of /Version Registry/Netscape/Communicator #20
--------------------------------
f_winpatch_exe
    Version (S)=1.0.1.11
f_winpatch_exe/f_winpatch.exe
    Path (F)=C:/Program Files/moz924/x86rel/f_winpatch_exe/clock.exe
    Version (S)=1.0.1.11
f_winpatch_exe/f_winpatch_exe
    RefCount (S)=1
    Path (F)=C:\Program Files\moz924\x86rel\f_winpatch_exe\clock.exe
    Version (S)=1.0.1.10

EXPECTED RESULT:
There is only one leaf node that corresponds to the patched file.  Notice the
use of "/" versus "\" in the Path.  Can this be part of the reason?

Comment 1

[Cathleen]

possible dup of 12020

Comment 2

[Cathleen]

reassign to dveditz.  possible dup of his bug 12020

Comment 3

[leger]

Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine

Comment 4

[Cathleen]

we don't register individual files anymore.
push out to M30

Comment 5

[Daniel Veditz [:dveditz]]

Taking advantage of new "Future" target milestone.

Comment 6

[Steve Smart]

*** Bug 156292 has been marked as a duplicate of this bug. ***

Comment 7

[Daniel Veditz [:dveditz]]

The xpinstall script engine has been removed from the trunk, bugs in it are obsolete.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/962a76f33c14
[fenix] For https://github.com/mozilla-mobile/fenix/issues/14902 - Disabling bottom toolbar animation now works (https://github.com/mozilla-mobile/fenix/pull/14927)