mirror of
https://github.com/espressif/esp-idf.git
synced 2025-08-03 20:54:32 +02:00
Tools: Don't check Python packages on every idf.py & cmake run
This commit is contained in:
@@ -273,27 +273,6 @@ function(__build_write_properties output_file)
|
|||||||
file(WRITE ${output_file} "${build_properties_text}")
|
file(WRITE ${output_file} "${build_properties_text}")
|
||||||
endfunction()
|
endfunction()
|
||||||
|
|
||||||
#
|
|
||||||
# Check if the Python interpreter used for the build has all the required modules.
|
|
||||||
#
|
|
||||||
function(__build_check_python)
|
|
||||||
idf_build_get_property(check __CHECK_PYTHON)
|
|
||||||
if(check)
|
|
||||||
idf_build_get_property(python PYTHON)
|
|
||||||
idf_build_get_property(idf_path IDF_PATH)
|
|
||||||
message(STATUS "Checking Python dependencies...")
|
|
||||||
execute_process(COMMAND "${python}" "${idf_path}/tools/idf_tools.py" "check-python-dependencies"
|
|
||||||
RESULT_VARIABLE result)
|
|
||||||
if(result EQUAL 1)
|
|
||||||
# check_python_dependencies returns error code 1 on failure
|
|
||||||
message(FATAL_ERROR "Some Python dependencies must be installed. Check above message for details.")
|
|
||||||
elseif(NOT result EQUAL 0)
|
|
||||||
# means check_python_dependencies.py failed to run at all, result should be an error message
|
|
||||||
message(FATAL_ERROR "Failed to run Python dependency check. Python: ${python}, Error: ${result}")
|
|
||||||
endif()
|
|
||||||
endif()
|
|
||||||
endfunction()
|
|
||||||
|
|
||||||
#
|
#
|
||||||
# Prepare for component processing expanding each component's project include
|
# Prepare for component processing expanding each component's project include
|
||||||
#
|
#
|
||||||
@@ -414,9 +393,6 @@ macro(idf_build_process target)
|
|||||||
|
|
||||||
__build_set_default(SDKCONFIG_DEFAULTS "")
|
__build_set_default(SDKCONFIG_DEFAULTS "")
|
||||||
|
|
||||||
# Check for required Python modules
|
|
||||||
__build_check_python()
|
|
||||||
|
|
||||||
idf_build_get_property(target IDF_TARGET)
|
idf_build_get_property(target IDF_TARGET)
|
||||||
|
|
||||||
if(NOT "${target}" STREQUAL "linux")
|
if(NOT "${target}" STREQUAL "linux")
|
||||||
|
18
tools/idf.py
18
tools/idf.py
@@ -89,24 +89,6 @@ def check_environment():
|
|||||||
except RuntimeError as e:
|
except RuntimeError as e:
|
||||||
raise FatalError(e)
|
raise FatalError(e)
|
||||||
|
|
||||||
# check Python dependencies
|
|
||||||
checks_output.append('Checking Python dependencies...')
|
|
||||||
try:
|
|
||||||
out = subprocess.check_output(
|
|
||||||
[
|
|
||||||
os.environ['PYTHON'],
|
|
||||||
os.path.join(os.environ['IDF_PATH'], 'tools', 'idf_tools.py'),
|
|
||||||
'check-python-dependencies',
|
|
||||||
],
|
|
||||||
env=os.environ,
|
|
||||||
)
|
|
||||||
|
|
||||||
checks_output.append(out.decode('utf-8', 'ignore').strip())
|
|
||||||
except subprocess.CalledProcessError as e:
|
|
||||||
print_warning(e.output.decode('utf-8', 'ignore'), stream=sys.stderr)
|
|
||||||
debug_print_idf_version()
|
|
||||||
raise SystemExit(1)
|
|
||||||
|
|
||||||
return checks_output
|
return checks_output
|
||||||
|
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user