Patch for #2924: Windows installer should add installation information to registry #2948

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
4 participants

Related to #2924:

  • Added node.js installation path and version in the registry, under HKCU\Software\Joyent\node.js

Tested on Windows XP x86.

Owner

bnoordhuis commented May 1, 2012

@piscisaureus: Can you review this?

Installed v0.8.4 on Windows 7 x64 today (up from v0.8.1 x86), and the installation went smoothly except for the PATH. The installer appeared to remove the x86 %programfiles(x86)% path but didn't add in the x64 %programfiles% path. I tried a reboot, but still had to enter the PATH manually and set it to %programfiles%\nodejs and log off/on to be able to use node and npm.

@morganestes: this patch is not part of node.js (yet?). If you have trouble with the current installer, you may find more support by reporting an issue...

mcartoixa closed this Jul 27, 2012

mcartoixa reopened this Jul 27, 2012

Thanks to @sblom, this seems to have been integrated via #4694. Closing.

mcartoixa closed this Feb 11, 2013

sblom commented Feb 11, 2013

Only partially. Please keep issue #2924 open, because we're still working on figuring out the right way to let multiple versions live side by side, and we'll get more registry structure created correctly then. Meanwhile, there is a registry key being created, but it's not quite what you originally requested--i.e. it's installer-specific and doesn't include a version number yet.

@mcartoixa, I don't suppose you have time and interest to help me work on the design (and maybe even implementation) for side-by-side installations of multiple versions...

mcartoixa reopened this Feb 11, 2013

mcartoixa closed this Feb 12, 2013

@sblom As you guessed, I can afford very little time on these matters right now (and besides my WiX abilities are limited). Good luck!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment