15220 - need wallet files in commercial install

Status: VERIFIED FIXED

(SeaMonkey :: Installer, defect, P3)

Description

[Cathleen]

We need to make sure wallet files are getting packaged up in commercial install.

Steve, please send me a list of your wallet files.  (dll, xpt, xuls, css...) for
all platforms...  (win, mac, unix).   Then, assign the bug back to me, so I can
add them to the package manifest file.

Thanks!

Comment 1

[Stephen P. Morse]

All the following files can be found in subdirectories under
mozilla/extensions/wallet.  Thanks.


wallet.dll
wlltvwrs.dll

cookieviewer.xpt
nsICookieViewer.xpt
nsIWalletEditor.xpt
walleteditor.xpt
nsISignonViewer.xpt
signonviewer.xpt
nsIWalletPreview.xpt
walletpreview.xpt

CookieViewer.js
WalletEditor.js
SignonViewer.js
WalletPreview.js

CookieViewer.xul
WalletEditor.xul
SignonViewer.xul
WalletPreview.xul

CookieViewer.css
SignonViewer.css

CookieViewer.dtd
SignonViewer.dtd

CookieViewer.properties
WalletEditor.properties
SignonViewer.properties
cookie.properties
wallet.properties
WalletPreview.properties

Comment 2

[Cathleen]

I was able to find most of your listed files under dist/../bin direcotry, except
the following files:

nsIcookieViewer.xpt
nsIWalletEditor.xpt
nsISignonViewer.xpt
nsIWalletPreview.xpt
SignonViewer.css
SignonViewer.dtd

are they accidentally not exported, or obsolete?

Comment 3

[Cathleen]

steve can you check if the files listed above are still valid?  -thanks!

Comment 4

[Cathleen]

windows packaging manifest file is updated, without listing those not found
files.

still need to update for mac & unix

Comment 5

[Stephen P. Morse]

All the files I listed are still valid.

.xpt files are in mozilla\dist\WIN32_D.OBJ\bin\components
.css files are in mozilla\dist\WIN32_D.OBJ\bin\chrome\wallet\skin\default
.dtd files are in mozilla\dist\WIN32_D.OBJ\bin\chrome\wallet\locale\en-US

Comment 6

[Cathleen]

steve, the following files are still missing from my dist:

nsIcookieViewer.xpt
nsIWalletEditor.xpt
nsISignonViewer.xpt
nsIWalletPreview.xpt

I believe they are probably old xpt files that still left in your dist
directory.  Looks like they're already been renamed, like this:

nsIcookieViewer.xpt --> cookieviewer.xpt

Please let me know if this is correct.


Also, on the mac builds, it appears your .xpt files in
components folder (cookieviewer.xpt, walleteditor.xpt, signonviewer.xpt &
walletpreview.xpt) are combined into one .xpt file, wallet.xpt.  If this is not
correct, please let me know.

Marking bug as fixed.  All necessary changes applied to all three platform's
packaging list.  (unless, hearing from Steve otherwise).

Comment 7

[Stephen P. Morse]

Yes, it appears that my script is generating cookieviewer.xpt and
nsICookieViewer.xpt as identical files and copying only the first one into the
dist directory.  So the nsICookieViewer is just in my local directory and is
never being used -- an accident of my build script no doubt.  Same goes for
nsISignonViewer.xpt, nsIWalletEditor.xpt and nsIWalletPreview.xpt.  So you can
ignore these files.

I don't know enough about the mac build to know if having a single xpt file
there is correct wherease we have four xpt files on the other platforms.  Are
there any mac people that you can ask about this?

Comment 8

[Grace Bush]

verified on 19991205 builds

Comment 9

[leger]

Updating component from Install Wizard to Installer.