Windows installer always installs to C:/Program Files

[balane]

I prefer to install all of my dev apps in one directory (C:/apps) but when using the Node.js Windows installer it always gets installed in C:/Program Files regardless of what I put in the installer.

• Version: 4.4.2
• Platform: Windows 7 64-bit
  64-bit_
• Subsystem: Windows Installer

[eljefedelrodeodeljefe]

I can confirm there is a bug (I consider it as such) at least on Windows 7. Windows 10 works fine though. @mscdex @Fishrock123 this would need re-labeling.

[eljefedelrodeodeljefe]

@balane can you run msinfo32 and copy-paste the result here. I just did a fresh install after I bought a OEM Professional version and there it did work as expected. Before, I tested with modern_ie. So good news for us is it is not affecting everyone on windows...hopefully.

[balane]

What exactly are you looking for from the msinfo32 result?

[eljefedelrodeodeljefe]

Build number of Windows, Architecture etc. Before I go debugging I wanted to have as much information as possible. Minus security relevant stuff of course...

[balane]

OS Name Microsoft Windows 7 Enterprise
Version 6.1.7601 Service Pack 1 Build 7601
Other OS Description Not Available
OS Manufacturer Microsoft Corporation
System Manufacturer Dell Inc.
System Model Latitude E6510
System Type x64-based PC
Processor Intel(R) Core(TM) i7 CPU Q 740 @ 1.73GHz, 1734 Mhz, 4 Core(s), 8 Logical Processor(s)
BIOS Version/Date Dell Inc. A16, 12/5/2013
SMBIOS Version 2.6
Windows Directory C:\Windows
System Directory C:\Windows\system32
Boot Device \Device\HarddiskVolume2
Locale United States
Hardware Abstraction Layer Version = "6.1.7601.17514"
Time Zone Central Daylight Time
Installed Physical Memory (RAM) 8.00 GB
Total Physical Memory 7.93 GB
Available Physical Memory 4.17 GB
Total Virtual Memory 15.9 GB
Available Virtual Memory 12.4 GB
Page File Space 7.93 GB

[orangemocha]

cc @joaocgreis , our MSI expert.

[joaocgreis]

I could not reproduce this. I tried to create the registry values that the installer reads (Software/Node.js under HKLM and HKCU) and those are read correctly but the path chosen by the user is always used.

@balane can you try to create a installation log? Use:

msiexec /i <filename>.msi /l*v InstallLog.txt

[eljefedelrodeodeljefe]

Hmm. I couldn't reproduce (again) neither, even though it was reproducable earlier. It's likely related to Windows being under whatever circumstances. Just installing again then solves it though. I would vote for closing this until there is some logged reproduction. @joaocgreis msiexec /i C:\Users\myusername\Downloads\somepackage.msi /L*v install.txt would be logging sufficiently I assume?

[balane]

InstallLog.txt

From a quick glance through the log file looks like it's saying that it correctly installed to C:/apps/nodejs but it absolutely did not.

[mcnameej]

The directory change is happening on the client side...

MSI (c) (F4:48) [16:08:46:758]: PROPERTY CHANGE: Modifying INSTALLDIR property. Its current value is 'C:\Program Files\nodejs\'. Its new value: 'C:\apps\nodejs\'.

...but never makes it to the server side...

MSI (s) (58:9C) [16:08:57:831]: Ignoring disallowed property INSTALLDIR

This results in the server-side using the default value...

MSI (s) (58:9C) [16:08:57:984]: PROPERTY CHANGE: Adding INSTALLDIR property. Its value is 'C:\Program Files\nodejs\'.

It's what happens on the server side that counts.

The problem is that INSTALLDIR needs to be added to the SecureCustomProperties MSI property. Something like this:

diff --git a/tools/msvs/msi/product.wxs b/tools/msvs/msi/product.wxs
index eed53e4..dcd6b90 100755
--- a/tools/msvs/msi/product.wxs
+++ b/tools/msvs/msi/product.wxs
@@ -45,6 +45,7 @@
     <Property Id="ARPPRODUCTICON" Value="NodeIcon"/>
     <Property Id="ApplicationFolderName" Value="nodejs"/>
     <Property Id="WIXUI_INSTALLDIR" Value="INSTALLDIR"/>
+    <Property Id="SecureCustomProperties" Value="INSTALLDIR"/>

     <Property Id="INSTALLDIR">
       <RegistrySearch Id="InstallPathRegistry"

(To answer the next logical question... Sorry, NO, I can't submit a PR for this right now.)