Skip to content

Commit

Permalink
fix(tools/idf_tools.py): Improve the warning message about the enviro…
Browse files Browse the repository at this point in the history
…nment
  • Loading branch information
dobairoland committed Jun 13, 2024
1 parent eba93f0 commit 86b7960
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions tools/idf_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -2144,7 +2144,8 @@ def check_python_venv_compatibility(idf_python_env_path: str, idf_version: str)
raise SystemExit(1)
except OSError as e:
# perhaps the environment was generated before the support for VENV_VER_FILE was added
warn(f'Error while accessing the ESP-IDF version file in the Python environment: {e}')
warn(f'The following issue occurred while accessing the ESP-IDF version file in the Python environment: {e}. '
'(Diagnostic information. It can be ignored.)')


def action_export(args: Any) -> None:
Expand Down Expand Up @@ -2643,7 +2644,8 @@ def action_install_python_env(args): # type: ignore
with open(os.path.join(idf_python_env_path, VENV_VER_FILE), 'w') as f:
f.write(idf_version)
except OSError as e:
warn(f'Error while generating the ESP-IDF version file in the Python environment: {e}')
warn(f'The following issue occurred while generating the ESP-IDF version file in the Python environment: {e}. '
'(Diagnostic information. It can be ignored.)')

else:
# The embeddable Python for Windows doesn't have the built-in venv module
Expand Down

0 comments on commit 86b7960

Please sign in to comment.