-
Notifications
You must be signed in to change notification settings - Fork 12.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
64-bit .exe installer for windows has wrong default directory #24397
Comments
Rust should install to |
/cc @brson |
Neither Ant nor Python install to C:\tools\ by default, and searching I can't find any reference to C:\tools\ even being a thing? Both Python and Ant install to C:\Python and C:\ant, which is incredibly annoying, so please don't do that! |
@Diggsey I might be mistaken, but installing both through chocolatey stuck them in |
For an OS without much filesystem hierarchy, And all packages that do this reduce PATH clutter, since chocolatey symlinks their executables into |
The idea was to deprecate the .exe installer once .msi is known to be working satisfactory. |
@vadimcn Can we have a zip release too, that contains Rust and Cargo like the others? Linux and OS X have an archive release. Those are preferred in chocolatey because they don't involve the registry or any other Windows components. |
@alexchandel We do produce tarballs for windows. |
Seems like we should probably just stop producing the .exe's. That fix would go in https://github.com/rust-lang/rust-packaging. |
@brson I don't see a cargo+rustc nightly tarball for windows, although I do see ones for linux and osx. |
triage: We do not link to the .exe installer anymore, but we still produce them. I don't have access to windows at the moment to check if this bug has been resolved, but it's been a long time... |
Grabbing the Windows download link from downloads and swapping the Testing on win7-x64, the MSI extracts with lessmsi, but the directory structure is somewhat unusual so there's no one-liner to get the Rust folder out. The best I came up with was: lessmsi x rust-foo.msi \opt\ && move \opt\SourceDir\Rust \opt\rust && rmdir /s /q \opt\SourceDir This yields a Rust install rooted at However now that rustup.rs exists, multiple targets aren't really an issue anymore. Someone should make a Chocolatey package for rustup.rs... |
Can someone with access to Windows check what happens here today? |
@Mark-Simulacrum The latest version installs to |
…omatsakis Remove legacy InnoSetup GUI installer On Windows the InnoSetup `.exe` installer was superseded by the MSI installer long ago. It's no longer needed. The `.exe` installer hasn't been linked from the [other installation methods](https://forge.rust-lang.org/infra/other-installation-methods.html#standalone) page in many years. As far as I can tell the intent was always to remove this installer once the MSI proved itself. Though admittedly both installers feel very "legacy" at this point. Removing this would mean we only maintain one Windows GUI installer and would speed up the distribution phase. As a result of removing InnoSetup, this closes rust-lang#24397
…omatsakis Remove legacy InnoSetup GUI installer On Windows the InnoSetup `.exe` installer was superseded by the MSI installer long ago. It's no longer needed. The `.exe` installer hasn't been linked from the [other installation methods](https://forge.rust-lang.org/infra/other-installation-methods.html#standalone) page in many years. As far as I can tell the intent was always to remove this installer once the MSI proved itself. Though admittedly both installers feel very "legacy" at this point. Removing this would mean we only maintain one Windows GUI installer and would speed up the distribution phase. As a result of removing InnoSetup, this closes rust-lang#24397
…omatsakis Remove legacy InnoSetup GUI installer On Windows the InnoSetup `.exe` installer was superseded by the MSI installer long ago. It's no longer needed. The `.exe` installer hasn't been linked from the [other installation methods](https://forge.rust-lang.org/infra/other-installation-methods.html#standalone) page in many years. As far as I can tell the intent was always to remove this installer once the MSI proved itself. Though admittedly both installers feel very "legacy" at this point. Removing this would mean we only maintain one Windows GUI installer and would speed up the distribution phase. As a result of removing InnoSetup, this closes rust-lang#24397
The .exe installer tries to install to "C:\Program Files (x86)\Rust", even though it's the 64-bit version. It then fails to create the destination directory due to a permission error.
The .msi installer correctly defaults to "C:\Program Files\Rust ", and has no problems with permissions, I would expect the .exe installer to have the same behaviour.
The text was updated successfully, but these errors were encountered: