12510 - AddSubcomponent()/AddDirectory() forms without vi don't inherit package vi

Status: VERIFIED FIXED

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

Description

[Jimmy Lee]

Build: 8/25/99 SeaMonkey build

1. From http://jimbob/trigger2.html, use drop-down menu and select one of the
   "a_getfolder..." jar files specific to Mac folders such as
   "a_getfolder_1_macshutdown" and trigger

Install.StartInstall("Acceptance: a_getfolder_1_macshutdown", regName, vi, 0);
f = Install.GetFolder("Mac Shutdown");
Install.AddSubcomponent(regName, jarSrc, f, jarSrc);

RESULT:
The installation of the file is fine.  The Install.log properly reflects the
installation.  The Version registry is missing the path and version info as
shown in the following example:

-----------------------------------
a_getfolder_1_macshutdown
  RefCount (S)=1
  Path (F)=null
-----------------------------------

EXPECTED RESULT:
The path and version info are shown similar to the example below:

-----------------------------------
a_getfolder_1_macshutdown
  RefCount (S)=1
  Path (F)=Webbies:System Folder:Shutdown Items:smrtupdt.txt
  Version (S)=1.0.1.22
-----------------------------------

Comment 1

[Cathleen]

Samir, do you have time to take a look at this Mac bug?

Comment 2

[Samir Gehani]

Sure. Let's be ambitious and set the target milestone to M10.

Comment 3

[Samir Gehani]

Version info is being set using my old registry and after replacing my old
registry with a new one. Used the test case cited: a_getfolder_1_macshutdown.

Comment 4

[Samir Gehani]

Alright, clarification for this test case. The last statement is not entirely
accurate. The version is being set for the *package node*. The version is not
being set for the *component*. The path and refcount are being set for
the component.

The real problem is cross-platform. The AddSubcomponent() form listed above
(and probably any other forms that don't take version information) fails to
assume the package version. This problem probably affects the AddDirectory()
forms also that don't take version information.

Verification of this test case requires that the
    <curr_comm>/getfolder_1_macshutdown/getfolder_1_macshutdown
node has the version, refcount, and path set correctly. (Note, the package name
for this test case is the same as the component name; hence
"getfoler_1_machsutdown" appears twice in the regname cited for verification.)

Changing summary to reflect problem described herein.

Comment 5

[Samir Gehani]

Confirmed that this bug plagued AddDirectory() too. The fix was separate than
the one for AddSubcomponent() and hence will need separate verification. A
sample test for AddDirectory() can be found at the trigger page:
    http://puma/xpinstall/test/trigger.html
    Testcase pulldown option: AddDir4

Fixes checked in.

Comment 6

[Jimmy Lee]

Build 9/7/99

Back to normal.

Comment 7

[leger]

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