[izpack-devel] Proposed fix for bug #6059: Wrong Installation Path for Windows

[Hans Aikema]

I prepared a version of InstallerBase.java where I implemented my 
suggestion. Does any of you object to committing this code into CVS?
Does anyone have a localized windows version available that doesn't use 
'Program Files' as the base folder for applications to test this change? 
(I only have Dutch and English available which both use "Program Files" 
as the default base application folder; it works on these versions).

It would be nice to get this into IzPack 3.9. It removes the necessity 
for the win32-defaultpaths.properties file and takes into account 
localized application root folders for languages not yet listed in the 
win32-defaultpaths.properties file.


> As far as I know bug #6059 should be easy to fix.  The appropriate 
> Program Files directory is available (to my best knowledge in all 
> localized windows versions) in the System environment variable 
> ProgramFiles (no spaces).
>
> Replacing
>
>        if (OsVersion.IS_WINDOWS)
>        {
>            dir = buildWindowsDefaultPath();
>        }
>
> by
>
>        if (OsVersion.IS_WINDOWS)
>        {
>            dir = System.getenv("ProgramFiles");
>        }
>
> in the InstallerBase class should fix the default installation paths
>
> Has this code been tried before and found invalid for some localized 
> windows versions?
>
> kind regards,
> Hans Aikema
> _______________________________________________
> izpack-devel mailing list
> izpack-devel at lists.berlios.de
> http://lists.berlios.de/mailman/listinfo/izpack-devel
>