mirror of
https://github.com/espressif/esp-idf.git
synced 2025-08-03 20:54:32 +02:00
tools: Catch OSError which is raised when the tool is not available
Closes https://github.com/espressif/esp-idf/issues/7140
This commit is contained in:
@@ -857,12 +857,15 @@ def get_python_env_path():
|
|||||||
with open(version_file_path, "r") as version_file:
|
with open(version_file_path, "r") as version_file:
|
||||||
idf_version_str = version_file.read()
|
idf_version_str = version_file.read()
|
||||||
else:
|
else:
|
||||||
|
idf_version_str = ''
|
||||||
try:
|
try:
|
||||||
idf_version_str = subprocess.check_output(['git', 'describe'],
|
idf_version_str = subprocess.check_output(['git', 'describe'],
|
||||||
cwd=global_idf_path, env=os.environ).decode()
|
cwd=global_idf_path, env=os.environ).decode()
|
||||||
|
except OSError:
|
||||||
|
# OSError should cover FileNotFoundError and WindowsError
|
||||||
|
warn('Git was not found')
|
||||||
except subprocess.CalledProcessError as e:
|
except subprocess.CalledProcessError as e:
|
||||||
warn('Git describe was unsuccessul: {}'.format(e))
|
warn('Git describe was unsuccessul: {}'.format(e.output))
|
||||||
idf_version_str = ''
|
|
||||||
match = re.match(r'^v([0-9]+\.[0-9]+).*', idf_version_str)
|
match = re.match(r'^v([0-9]+\.[0-9]+).*', idf_version_str)
|
||||||
if match:
|
if match:
|
||||||
idf_version = match.group(1)
|
idf_version = match.group(1)
|
||||||
|
Reference in New Issue
Block a user