30166 - Installer seems hard coded to look at c: drive's space

Status: VERIFIED FIXED

(SeaMonkey :: Installer, defect, P3)

Description

[Rick Elliott]

Trying to install the informal 0302 ja build on drive d: I get the message 
"insufficient disk space on drive c:" That's true enough but shouldn't matter, 
the installer files are all on my d: drive.

Comment 1

[Doron Rosenberg (IBM)]

is this still happening?

Comment 2

[Sean Su]

it might be.  the installer uses the TEMP folder to hold the .xpi files it 
downloads/uncompresses.  If TEMP is on c: drive, and there's not enough space on 
there, it will complain.

I think that the disk space information needs to be more descriptive.  I will 
see what I can do.

Comment 3

[timeless]

I don't have a c: drive.
But I do get very wierd errors.
See bug 52038

Assuming that's what is really happening I'm going to mark this as critical 
nsbeta3 relnote rtm.

I know that most people do have a c drive. But that is not required by windows 
nt.

Comment 4

[Daniel Veditz [:dveditz]]

We aren't hard-coded to C: -- we look for a TEMP or some such environment 
variable, and failing that fall back on what windows says is the temp folder 
(commonly a subfolder of the windows directory). If you don't have a C: drive 
it shouldn't be an issue for us as windows wouldn't be telling us that's where 
temp files go.

Clearer error messages would be nice. If that's all you're asking for and Sean 
thinks it's doable in the remaining time then feel free to appeal my nsbeta3- 
decision. We should think about change the summary to be more accurate, too. 
Would "Install dialog confusing when out of space for temp files" work?

Comment 5

[Sean Su]

this is now fixed when I fixed bug #51928.  When it complains about not having 
enough disk space, it will now show the full path to where it's complaining 
about.  In this bug's case, it should show something like c:\temp\ns_temp as not 
having enought space instead of simply C:\ (the user selected d: to be where the 
destination path is).

Comment 6

[Grace Bush]

verified on build 2000091409M18