classification
Title: Update "install launcher for all users" installer option
Type: enhancement Stage: patch review
Components: Windows Versions: Python 3.10
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: ZackerySpytz, eryksun, paul.moore, steve.dower, tim.golden, zach.ware
Priority: normal Keywords: patch

Created on 2020-09-23 22:45 by steve.dower, last changed 2021-02-19 04:06 by eryksun.

Pull Requests
URL Status Linked Edit
PR 24570 open ZackerySpytz, 2021-02-18 23:52
Messages (3)
msg377430 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2020-09-23 22:45
The "Install launcher for all users" option on the front page of the installer should be updated to read "Install py.exe launcher for all users" to reduce confusion.

However, we should also consider disabling this by default, so that the normal settings do not require admin privileges to install.

The reason we started with it enabled by default is because we had to stomp on the 3.4 and earlier py.exe, since there was no other way to make sure it wasn't an issue. That problem still remains, which likely means that if we detect an all-users launcher from a previous version we need to keep the option enabled, but at least for clean installs of 3.10 we can avoid the need for admin completely.
msg377432 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2020-09-23 22:46
Follow up thought: we could switch out the front page checkbox for "Upgrade existing py.exe launcher (recommended)" when we detect an existing per-machine install, and hide it entirely if we don't detect it. So at least non-admin users have an easy way to bypass the admin requirement.
msg387281 - (view) Author: Eryk Sun (eryksun) * (Python triager) Date: 2021-02-19 04:06
> but at least for clean installs of 3.10 we can avoid the need 
> for admin completely

Except updating PATHEXT always requires admin access. I do this manually by setting a user PATHEXT variable with %PATHEXT% as the first item and append the extensions that I need. This is okay because the system environment gets populated before the user environment, and a system environment that doesn't define PATHEXT is improperly configured. I don't know whether WiX/MSI can support this, but it's probably not worth the hassle and potential problems.
History
Date User Action Args
2021-02-19 04:06:54eryksunsetnosy: + eryksun
messages: + msg387281
2021-02-18 23:52:34ZackerySpytzsetkeywords: + patch
nosy: + ZackerySpytz

pull_requests: + pull_request23349
stage: needs patch -> patch review
2020-09-23 22:46:36steve.dowersetmessages: + msg377432
2020-09-23 22:45:27steve.dowercreate