Eliminate fallback for Python environment setup #299
Merged
Conversation
This file contains hidden or 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
Removed fallback setup for Python environment in platform configuration.
Jason2866
added a commit
that referenced
this pull request
Oct 8, 2025
* Update Arduino and IDF versions in README * move penv setup to platform.py * simplify * call installer with penv Python * revert * fix scons error * try again to call tl-install from penv Python * install esptool later * esptool install later * remove warning esptool noise * sort imports * Replace remaining direct setup_python_environment call with platform.setup_python_env * wrong function name * fix: parameter name shadows function * Type hints: use Optional[str] for nullable arguments * Remove unused parameter penv_python * Avoid leaving sockets open * make github_actions bool * remove duplcate python check * fix endless recursion * show 1000 chars on failure with idf_tools.py * add "GIT_SSL_CAINFO" * use importlib for penv_setup.py * remove duplicate comment * update comments * no assert * no env.subst needed * Eliminate fallback for Python environment setup (#299) Removed fallback setup for Python environment in platform configuration. * Increase subprocess timeout for installations * Modify esptool path check to return None * Improve error handling for missing Python executable * Add urllib3 dependency with version constraint * Modify _setup_certifi_env to use python_exe argument Updated _setup_certifi_env to accept an optional python executable argument for better certifi path resolution. * remove fallback for certifi environment setup in penv_setup.py * Check for Python executable file in penv_setup * Change existence check to use isfile "python" for penv_dir * Refactor _setup_certifi_env function parameters Removed unused parameters and added GIT_SSL_CAINFO environment variable setup. * Change warnings to errors in package installation and install one by one. So a failure does not stop installing other packages. * Remove '--ng' option from esp_idf_size command * Update esp-idf-size package version to 2.0.0 * Update debugger package v16.3 * Add pydantic dependency version specification * Remove unused import in espidf.py Remove unused import of 'join' from os.path
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Removed fallback setup for Python environment in platform configuration.
Description:
Related issue (if applicable): fixes #
Checklist: