-
Notifications
You must be signed in to change notification settings - Fork 636
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Avoid PATH overflow from SetupPython calls (#11215)
Calling Installer.SetupPython from Python.Included makes sure Python is installed and also adds the install location to the PATH env variable. The location is added to the PATH even if it was already there, so this can potentially lead to an env variable overflow with the message 'Environment variable name or value is too long', after which the only way to get CPython3 back to work would be to restart Dynamo. The problem is avoided by remembering if Python was setup using a local field. This should reduce the number of calls to a maximum of 2, which is not the best (1 would be) but is far from resulting in an overflow.
- Loading branch information
Showing
2 changed files
with
32 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters