Opened 10 years ago
Closed 9 years ago
#435 closed defect (fixed)
Windows 7 Full Install - Error in PATH variable
| Reported by: | Josh | Owned by: | Christian Franke |
|---|---|---|---|
| Priority: | major | Milestone: | Release 6.4 |
| Component: | all | Version: | 6.3 |
| Keywords: | windows, installer | Cc: |
Description
When installing via the "Full" option on Windows 7, the PATH variable is not being set correctly. It must be added with quotes around it. Spaces will cause the entire variable to be ignored.
Incorrect: C:\Program Files\smartmontools\bin
Correct: "C:\Program Files\smartmontools\bin"
Change History (7)
comment:1 by , 10 years ago
| Keywords: | installer added; install path variable 7 removed |
|---|
comment:2 by , 10 years ago
comment:3 by , 10 years ago
PATH was empty except for the value added. The value type is REG_SZ not REG_EXPAND_SZ if that makes a difference. I will try installs on more machines and see if I can reproduce.
comment:4 by , 10 years ago
| Milestone: | → Release 6.4 |
|---|---|
| Owner: | set to |
| Priority: | minor → major |
| Status: | new → accepted |
This is the unfortunate result of a bug in the NSIS function ReadRegStr. If the registry value is longer than 1023, it returns an empty string instead of failure.
As a workaround, uncheck "Add install dir to PATH" if the user part of the PATH is very long.
comment:5 by , 10 years ago
My particular issue was not due to string length. This was the only value in the user Path variable. Unfortunately I cannot seem to reproduce it now. I thought it might have to do with registry value type but doesn't seem to be the case. It refused to work and had to add quotes. Now it works fine without even if I uninstall and re-install. Sorry for the wasted ticket.
Could not reproduce.
The installer changes the (user specific part of the) PATH variable by changing the registry value
HKEY_CURRENT_USER\Environment\PATH. No quotes are needed then.Please check whether this registry value is longer than 1023. Then PATH manipulation fails due to a string length limitation of the NSIS installer. This is actually an open issue.