87315 - Error on installation

Status: VERIFIED FIXED

(SeaMonkey :: Installer, defect)

Description

[Peter Jeziorek]

1) Grab the latest Win32 build 2001062206 and begin installing.
2) Downloads fine, during installation an error causes the installation to halt.

Error occurred during installation - Navigator: -214 DOES_NOT_EXIST

The install progress window says:

Currently installing Navigator (with about 1/10 of the progress bar filled)
Preparing file: Register Locale: (with about 1/5 of the progress bar filled)

Comment 1

[Peter Jeziorek]

adding keyword smoketest

Comment 2

[Peter Jeziorek]

Attachment: Copy of the Installation Log

Comment 3

[Sean Su]

Attachment: patch #1 (ns tree)

Comment 4

[Sean Su]

Attachment: patch #1 (moz tree)

Comment 5

[Sean Su]

The patches seem to fix the problem.  I think it was a cvs conflict last night 
between my checkins and Ian's.  I'm CC:'ing Ian just to make sure the patch is 
correct.

Ian, can I get an r= from you for the patches?

Comment 6

[Ian Oeschger (gone)]

Sean. Your patches remove the updates I am putting in from 87225 and 5687 to 
move the help content over into the en-US.jar. It (your pathc) may fix the 
problem, but I still need my localizable stuff over there. 

How can I get it over into en-US.jar/help without disrupting your patches from 
last night?

Comment 7

[Ian Oeschger (gone)]

Does a locale have to be registered from the langenus.jst file for some reason? 
Help is the only thing being packaged into en-US.jar in browser.jst. 

Comment 8

[Ian Oeschger (gone)]

Attachment: What about moving locale registration into langenus.jst instead? Not sure how this file works, exactly.

Comment 9

[Dawn Endico]

is 87319 a dup of this? on linux the net installer worked fine but there's some
kind of packaging problem and no content shows up in the help viewer.

Comment 10

[Ian Oeschger (gone)]

Attachment: patch for moving locale to langenus.jst files on all plats

Comment 11

[Sean Su]

reassigning to Ian per our conversation.

Comment 12

[Ian Oeschger (gone)]

cc'ing samir, too.
You guys want to take a look at my handywork and r it? I will check it in post 
haste.

Comment 13

[Samir Gehani]

Doh!  I should have caught this earlier.  Yes, moving it landenus.jst is the
right thing to do.  Not the first time this has bitten us.  r=sgehani  You'll
need to do the same for ns too.

Comment 14

[Sean Su]

r=ssu for ian's moz tree patch id=39697.  We're still missing a similar patch to 
the ns tree.

Comment 15

[Ian Oeschger (gone)]

Attachment: ns version of same

Comment 16

[Sean Su]

r=ssu for the ns tree patch too.  let's land this puppy!

Comment 17

[J.J. Enser]

Bug says Windows, but the patches affect all platforms ...
Which one do I need to respin ?
how come mac installer didn't fail ?

Comment 18

[Ian Oeschger (gone)]

Checked in on both sides. Mark fixed or hold out for further verification?

Comment 19

[Ian Oeschger (gone)]

jj
apparently the linux packaging fails silently. This morning endico and asa said 
no content or dtd strings were in their help viewer. And mac I don't know about.

Comment 20

[Sean Su]

FYI: for all platforms, only an installer respin is required once the fixed .jst 
files have been checked out.  A respin of the full client build is _not_ 
necessary.

Comment 21

[Ian Oeschger (gone)]

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

Comment 22

[Ian Oeschger (gone)]

updating plat to all.

Comment 23

[Ian Oeschger (gone)]

Marking FIXED.

Comment 24

[Sean Su]

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

Comment 25

[Boris Zbarsky [:bzbarsky]]

there's still nothing in the help viewer in build 2001-06-25-08 on Linux.  See
bug  87487

Comment 26

[Grace Bush]

verified, not happening on build 2001081703