From c05fcc6a1d2693522d6a4b48b577882386b1f538 Mon Sep 17 00:00:00 2001 From: "martin.gano" Date: Wed, 4 Nov 2020 23:32:40 +0100 Subject: [PATCH] Tools: add Python 2 deprecation warning --- components/app_update/otatool.py | 4 + components/efuse/efuse_table_gen.py | 4 + components/spiffs/spiffsgen.py | 7 +- components/ulp/esp32ulp_mapgen.py | 6 + components/xtensa/trax/traceparse.py | 343 ++++++++++++++++++++ docs/en/get-started/linux-setup-scratch.rst | 17 +- docs/en/get-started/linux-setup.rst | 37 ++- docs/en/get-started/macos-setup-scratch.rst | 4 + docs/en/get-started/macos-setup.rst | 5 + tools/ci/check_build_warnings.py | 2 + tools/ci/config/build.yml | 5 + tools/idf.py | 6 + 12 files changed, 430 insertions(+), 10 deletions(-) create mode 100644 components/xtensa/trax/traceparse.py diff --git a/components/app_update/otatool.py b/components/app_update/otatool.py index 20f69fa923..86c630f5af 100755 --- a/components/app_update/otatool.py +++ b/components/app_update/otatool.py @@ -249,6 +249,10 @@ def _erase_ota_partition(target, ota_id): def main(): + if sys.version_info[0] < 3: + print("WARNING: Support for Python 2 is deprecated and will be removed in future versions.") + elif sys.version_info[0] == 3 and sys.version_info[1] < 6: + print("WARNING: Python 3 versions older than 3.6 are not supported.") global quiet parser = argparse.ArgumentParser("ESP-IDF OTA Partitions Tool") diff --git a/components/efuse/efuse_table_gen.py b/components/efuse/efuse_table_gen.py index 02940c27a5..eea308bd23 100755 --- a/components/efuse/efuse_table_gen.py +++ b/components/efuse/efuse_table_gen.py @@ -448,6 +448,10 @@ def create_output_files(name, output_table, debug): def main(): + if sys.version_info[0] < 3: + print("WARNING: Support for Python 2 is deprecated and will be removed in future versions.", file=sys.stderr) + elif sys.version_info[0] == 3 and sys.version_info[1] < 6: + print("WARNING: Python 3 versions older than 3.6 are not supported.", file=sys.stderr) global quiet global max_blk_len diff --git a/components/spiffs/spiffsgen.py b/components/spiffs/spiffsgen.py index de160294d3..11dc88aa14 100755 --- a/components/spiffs/spiffsgen.py +++ b/components/spiffs/spiffsgen.py @@ -16,8 +16,9 @@ # See the License for the specific language governing permissions and # limitations under the License. -from __future__ import division +from __future__ import division, print_function import os +import sys import io import math import struct @@ -451,6 +452,10 @@ class SpiffsFS(): def main(): + if sys.version_info[0] < 3: + print("WARNING: Support for Python 2 is deprecated and will be removed in future versions.") + elif sys.version_info[0] == 3 and sys.version_info[1] < 6: + print("WARNING: Python 3 versions older than 3.6 are not supported.") parser = argparse.ArgumentParser(description="SPIFFS Image Generator", formatter_class=argparse.ArgumentDefaultsHelpFormatter) diff --git a/components/ulp/esp32ulp_mapgen.py b/components/ulp/esp32ulp_mapgen.py index 52ad8a5f09..c40e137404 100755 --- a/components/ulp/esp32ulp_mapgen.py +++ b/components/ulp/esp32ulp_mapgen.py @@ -5,7 +5,9 @@ # Copyright (c) 2016-2017 Espressif Systems (Shanghai) PTE LTD. # Distributed under the terms of Apache License v2.0 found in the top-level LICENSE file. +from __future__ import print_function from optparse import OptionParser +import sys BASE_ADDR = 0x50000000 @@ -26,6 +28,10 @@ def gen_ld_h_from_sym(f_sym, f_ld, f_h): def main(): + if sys.version_info[0] < 3: + print("WARNING: Support for Python 2 is deprecated and will be removed in future versions.", file=sys.stderr) + elif sys.version_info[0] == 3 and sys.version_info[1] < 6: + print("WARNING: Python 3 versions older than 3.6 are not supported.", file=sys.stderr) description = ("This application generates .h and .ld files for symbols defined in input file. " "The input symbols file can be generated using nm utility like this: " "esp32-ulp-nm -g -f posix > ") diff --git a/components/xtensa/trax/traceparse.py b/components/xtensa/trax/traceparse.py new file mode 100644 index 0000000000..119e4c9e88 --- /dev/null +++ b/components/xtensa/trax/traceparse.py @@ -0,0 +1,343 @@ +#!/usr/bin/env python +# coding=utf-8 +# +# This script decodes Xtensa CPU trace dumps. It allows tracing the program +# execution at instruction level. +# +# Some trivia about the Xtensa CPU trace (TRAX): +# TRAX format mostly follows the IEEE-ISTO 5001-2003 (Nexus) standard. +# The following Nexus Program Trace messages are implemented by TRAX: +# - Indirect Branch Message +# - Syncronization Message +# - Indirect Branch with Synchronization Message +# - Correlation Message +# TRAX outputs compressed traces with 2 MSEO bits (LSB) and 6 MDO bits (MSB), +# packed into a byte. MSEO bits are used to split the stream into packets and messages, +# and MDO bits carry the actual data of the messages. Each message may contain multiple packets. +# +# This script can be used standalone, or loaded into GDB. +# When used standalone, it dumps the list of trace messages to stdout. +# When used from GDB, it also invokes GDB command to dump the list of assembly +# instructions corresponding to each of the messages. +# +# Standalone usage: +# traceparse.py +# +# Usage from GDB: +# xtensa-esp32-elf-gdb -n --batch program.elf -x gdbinit +# with the following gdbinit script: +# set pagination off +# set confirm off +# add-symbol-file rom.elf
+# source traceparse.py +# python parse_and_dump("/path/to/dump_file") +# +# Loading the ROM code is optional; if not loaded, disassembly for ROM sections of code +# will be missing. +# +### +# Copyright 2020 Espressif Systems (Shanghai) PTE LTD +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +from __future__ import print_function +import sys + +# Check if loaded into GDB +try: + assert gdb.__name__ == "gdb" + WITH_GDB = True +except NameError: + WITH_GDB = False + +# MSEO bit masks: +MSEO_PKTEND = 1 << 0 # bit 0: indicates the last byte of a packet +MSEO_MSGEND = 1 << 1 # bit 1: indicates the last byte of the message + +# Message types. The type is stored in the first 6 MDO bits or the first packet. +TVAL_INDBR = 4 # Indirect branch +TVAL_INDBRSYNC = 12 # Indirect branch w/ synchronisation +TVAL_SYNC = 9 # Synchronisation msg +TVAL_CORR = 33 # Correlation message + + +class TraxPacket(object): + def __init__(self, data, truncated=False): + self.data = data + self.size_bytes = len(data) + self.truncated = truncated + + def get_bits(self, start, count=0): + """ + Extract data bits from the packet + :param start: offset, in bits, of the part to be extracted + :param count: number of bits to extract; if omitted or zero, + extracts until the end of the packet + :return: integer containing the extracted bits + """ + start_byte = start // 6 + if count <= 0: + # all remaining bits + count = len(self.data) * 6 - start + bits_remaining = count + result = 0 + shift = 0 + for i, b in enumerate(self.data[start_byte:]): + # which bit in the byte is the starting bit + if i == 0: + # at start_byte: take the offset into account + start_bit = 2 + (start % 6) + else: + # every other byte: start after MSEO bits + start_bit = 2 + # how many bits do we need to copy from this byte + cnt_bits = min(bits_remaining, 8 - start_bit) + mask = (2 ** cnt_bits) - 1 + # take this many bits after the start_bit + bits = (b >> start_bit) & mask + # add these bits to the result + result |= bits << shift + # update the remaining bit count + shift += cnt_bits + bits_remaining -= cnt_bits + if bits_remaining == 0: + break + return result + + def __str__(self): + return "%d byte packet%s" % (self.size_bytes, " (truncated)" if self.truncated else "") + + +class TraxMessage(object): + def __init__(self, packets, truncated=False): + """ + Create and parse a TRAX message from packets + :param packets: list of TraxPacket objects, must not be empty + :param truncated: whether the message was truncated in the stream + """ + assert len(packets) > 0 + self.packets = packets + self.truncated = truncated + if truncated: + self.msg_type = None + else: + self.msg_type = self._get_type() + + # Start and end of the instruction range corresponding to this message + self.pc_start = 0 # inclusive + self.pc_end = 0 # not inclusive + self.pc_target = 0 # PC of the next range + self.is_exception = False # whether the message indicates an exception + self.is_correlation = False # whether this is a correlation message + + # message-specific fields + self.icnt = 0 + self.uaddr = 0 + self.dcont = 0 + + # decode the fields + if not truncated: + self._decode() + + def _get_type(self): + """ + :return: Message type, one of TVAL_XXX values + """ + return self.packets[0].get_bits(0, 6) + + def _decode(self): + """ Parse the packets and fill in the message-specific fields """ + if self.msg_type == TVAL_INDBR: + self.icnt = self.packets[0].get_bits(7, -1) + self.btype = self.packets[0].get_bits(6, 1) + self.uaddr = self.packets[1].get_bits(0) + self.is_exception = self.btype > 0 + elif self.msg_type == TVAL_INDBRSYNC: + self.icnt = self.packets[0].get_bits(8, -1) + self.btype = self.packets[0].get_bits(7, 1) + self.pc_target = self.packets[1].get_bits(0) + self.dcont = self.packets[0].get_bits(6, 1) + self.is_exception = self.btype > 0 + elif self.msg_type == TVAL_SYNC: + self.icnt = self.packets[0].get_bits(7, -1) + self.dcont = self.packets[0].get_bits(6, 1) + self.pc_target = self.packets[1].get_bits(0) + elif self.msg_type == TVAL_CORR: + self.icnt = self.packets[0].get_bits(12, -1) + self.is_correlation = True + else: + raise NotImplementedError("Unknown message type (%d)" % self.msg_type) + + def process_forward(self, cur_pc): + """ + Given the target PC known from the previous message, determine + the PC range corresponding to the current message. + :param cur_pc: previous known PC + :return: target PC after the current message + """ + assert not self.truncated + + next_pc = cur_pc + if self.msg_type == TVAL_INDBR: + next_pc = cur_pc ^ self.uaddr + self.pc_target = next_pc + self.pc_start = cur_pc + self.pc_end = self.pc_start + self.icnt + 1 + if self.msg_type == TVAL_INDBRSYNC: + next_pc = self.pc_target + self.pc_start = cur_pc + self.pc_end = cur_pc + self.icnt + 1 + if self.msg_type == TVAL_SYNC: + next_pc = self.pc_target + self.pc_start = next_pc - self.icnt + self.pc_end = next_pc + 1 + if self.msg_type == TVAL_CORR: + pass + return next_pc + + def process_backward(self, cur_pc): + """ + Given the address of the PC known from the _next_ message, determine + the PC range corresponding to the current message. + :param cur_pc: next known PC + :return: target PC of the _previous_ message + """ + assert not self.truncated + # Backward pass is only used to resolve addresses of messages + # up to the first SYNC/INDBRSYNC message. + # SYNC/INDBRSYNC messages are only handled in the forward pass. + assert self.msg_type != TVAL_INDBRSYNC + assert self.msg_type != TVAL_SYNC + + prev_pc = cur_pc + self.pc_target = cur_pc + if self.msg_type == TVAL_INDBR: + prev_pc ^= self.uaddr + self.pc_start = prev_pc + self.pc_end = prev_pc + self.icnt + 1 + if self.msg_type == TVAL_CORR: + pass + return prev_pc + + def __str__(self): + desc = "Unknown (%d)" % self.msg_type + extra = "" + if self.truncated: + desc = "Truncated" + if self.msg_type == TVAL_INDBR: + desc = "Indirect branch" + extra = ", icnt=%d, uaddr=0x%x, exc=%d" % (self.icnt, self.uaddr, self.is_exception) + if self.msg_type == TVAL_INDBRSYNC: + desc = "Indirect branch w/sync" + extra = ", icnt=%d, dcont=%d, exc=%d" % (self.icnt, self.dcont, self.is_exception) + if self.msg_type == TVAL_SYNC: + desc = "Synchronization" + extra = ", icnt=%d, dcont=%d" % (self.icnt, self.dcont) + if self.msg_type == TVAL_CORR: + desc = "Correlation" + extra = ", icnt=%d" % self.icnt + return "%s message, %d packets, PC range 0x%08x - 0x%08x, target PC 0x%08x" % ( + desc, len(self.packets), self.pc_start, self.pc_end, self.pc_target) + extra + + +def load_messages(data): + """ + Decodes TRAX data and resolves PC ranges. + :param data: input data, bytes + :return: list of TraxMessage objects + """ + messages = [] + packets = [] + packet_start = 0 + msg_cnt = 0 + pkt_cnt = 0 + + # Iterate over the input data, splitting bytes into packets and messages + for i, b in enumerate(data): + if (b & MSEO_MSGEND) and not (b & MSEO_PKTEND): + raise AssertionError("Invalid MSEO bits in b=0x%x. Not a TRAX dump?" % b) + + if b & MSEO_PKTEND: + pkt_cnt += 1 + packets.append(TraxPacket(data[packet_start:i + 1], packet_start == 0)) + packet_start = i + 1 + + if b & MSEO_MSGEND: + msg_cnt += 1 + try: + messages.append(TraxMessage(packets, len(messages) == 0)) + except NotImplementedError as e: + sys.stderr.write("Failed to parse message #%03d (at %d bytes): %s\n" % (msg_cnt, i, str(e))) + packets = [] + + # Resolve PC ranges of messages. + # Forward pass: skip messages until a message with known PC, + # i.e. a SYNC/INDBRSYNC message. Process all messages following it. + pc = 0 + first_sync_index = -1 + for i, m in enumerate(messages): + if pc == 0 and m.pc_target == 0: + continue + if first_sync_index < 0: + first_sync_index = i + pc = m.process_forward(pc) + + # Now process the skipped messages in the reverse direction, + # starting from the first message with known PC. + pc = messages[first_sync_index].pc_start + for m in reversed(messages[0:first_sync_index]): + if m.truncated: + break + pc = m.process_backward(pc) + + return messages + + +def parse_and_dump(filename, disassemble=WITH_GDB): + """ + Decode TRAX data from a file, print out the messages. + :param filename: file to load the dump from + :param disassemble: if True, print disassembly of PC ranges + """ + with open(filename, 'rb') as f: + data = f.read() + + messages = load_messages(data) + sys.stderr.write("Loaded %d messages in %d bytes\n" % (len(messages), len(data))) + + for i, m in enumerate(messages): + if m.truncated: + continue + print("%04d: %s" % (i, str(m))) + if m.is_exception: + print("*** Exception occurred ***") + if disassemble and WITH_GDB: + try: + gdb.execute("disassemble 0x%08x, 0x%08x" % (m.pc_start, m.pc_end)) # noqa: F821 + except gdb.MemoryError: # noqa: F821 + print("Failed to disassemble from 0x%08x to 0x%08x" % (m.pc_start, m.pc_end)) + + +def main(): + if sys.version_info[0] < 3: + print("WARNING: Support for Python 2 is deprecated and will be removed in future versions.", file=sys.stderr) + elif sys.version_info[0] == 3 and sys.version_info[1] < 6: + print("WARNING: Python 3 versions older than 3.6 are not supported.", file=sys.stderr) + if len(sys.argv) < 2: + sys.stderr.write("Usage: %s \n") + raise SystemExit(1) + + parse_and_dump(sys.argv[1]) + + +if __name__ == "__main__" and not WITH_GDB: + main() diff --git a/docs/en/get-started/linux-setup-scratch.rst b/docs/en/get-started/linux-setup-scratch.rst index 76991379bc..ad9928b8d7 100644 --- a/docs/en/get-started/linux-setup-scratch.rst +++ b/docs/en/get-started/linux-setup-scratch.rst @@ -13,15 +13,17 @@ To compile with ESP-IDF you need to get the following packages: - CentOS 7:: - sudo yum install git wget ncurses-devel flex bison gperf python pyserial python-pyelftools cmake ninja-build ccache + sudo yum -y update && sudo yum install git wget ncurses-devel flex bison gperf python3 python3-pip cmake ninja-build ccache + +CentOS 7 is still supported but CentOS version 8 is recommended for a better user experience. - Ubuntu and Debian:: - sudo apt-get install git wget libncurses-dev flex bison gperf python python-pip python-setuptools python-serial python-click python-cryptography python-future python-pyparsing python-pyelftools cmake ninja-build ccache libffi-dev libssl-dev + sudo apt-get install git wget libncurses-dev flex bison gperf python3 python3-pip python3-setuptools python3-serial python3-cryptography python3-future python3-pyparsing python3-pyelftools cmake ninja-build ccache libffi-dev libssl-dev dfu-util - Arch:: - sudo pacman -S --needed gcc git make ncurses flex bison gperf python-pyserial python-click python-cryptography python-future python-pyparsing python-pyelftools cmake ninja ccache + sudo pacman -Sy --needed gcc git make ncurses flex bison gperf python-pyserial python-cryptography python-future python-pyparsing python-pyelftools cmake ninja ccache dfu-util .. note:: CMake version 3.5 or newer is required for use with ESP-IDF. Older Linux distributions may require updating, enabling of a "backports" repository, or installing of a "cmake3" package rather than "cmake". @@ -33,7 +35,7 @@ Compile the Toolchain from Source - CentOS 7:: - sudo yum install gawk gperf grep gettext ncurses-devel python python-devel automake bison flex texinfo help2man libtool make + sudo yum install gawk gperf grep gettext ncurses-devel python3 python3-devel automake bison flex texinfo help2man libtool make - Ubuntu pre-16.04:: @@ -49,7 +51,7 @@ Compile the Toolchain from Source - Arch:: - TODO + sudo pacman -Sy --needed python-pip Create the working directory and go into it:: @@ -68,6 +70,11 @@ Build the toolchain:: Toolchain will be built in ``~/esp/crosstool-NG/builds/xtensa-esp32-elf``. Follow `instructions for standard setup `_ to add the toolchain to your ``PATH``. +Python 2 deprecation +==================== + +Python 2 reached its `end of life `_ and support for it in ESP-IDF will be removed soon. Please install Python 3.6 or higher. Instructions for popular Linux distributions are listed above. + Next Steps ========== diff --git a/docs/en/get-started/linux-setup.rst b/docs/en/get-started/linux-setup.rst index d5f2bba401..a56a3b18ca 100644 --- a/docs/en/get-started/linux-setup.rst +++ b/docs/en/get-started/linux-setup.rst @@ -11,11 +11,13 @@ To compile with ESP-IDF you need to get the following packages: - CentOS 7:: - sudo yum install git wget ncurses-devel flex bison gperf python cmake ninja-build ccache + sudo yum -y update && sudo yum install git wget flex bison gperf python3 cmake ninja-build ccache + +CentOS 7 is still supported but CentOS version 8 is recommended for a better user experience. - Ubuntu and Debian:: - sudo apt-get install git wget libncurses-dev flex bison gperf python python-pip python-setuptools cmake ninja-build ccache libffi-dev libssl-dev + sudo apt-get install git wget flex bison gperf python3 python3-pip python3-setuptools cmake ninja-build ccache libffi-dev libssl-dev dfu-util - Arch:: @@ -46,20 +48,47 @@ Before installing these packages you might need to add the author's public key t Alternatively, use crosstool-NG to compile a gdb that links against ncurses 6. +Setting up Python 3 as default for CentOS +----------------------------------------- + +CentOS 7 and older is providing Python 2.7 as the default interpreter. +Python 3 is recommended instead and can be installed in old distributions as follows, or please consult the documentation of your operating system for other recommended ways to achieve this:: + + sudo yum -y update && sudo yum install python3 python3-pip python3-setuptools + +Making Python 3 the default interpreter is possible by running:: + + sudo update-alternatives --install /usr/bin/python python /usr/bin/python3 10 && alias pip=pip3 + Setting up Python 3 as default for Ubuntu and Debian ---------------------------------------------------- -Ubuntu and Debian are still providing Python 2.7 as the default interpreter. Python 3 can be installed as follows:: +Ubuntu (version 18.04 and older) and Debian (version 9 and older) are still providing Python 2.7 as the default interpreter. +Python 3 is recommended instead and can be installed in old distributions as follows, or please consult the documentation of your operating system for other recommended ways to achieve this:: sudo apt-get install python3 python3-pip python3-setuptools Making Python 3 the default interpreter is possible by running:: - sudo update-alternatives --install /usr/bin/python python /usr/bin/python3 10 + sudo update-alternatives --install /usr/bin/python python /usr/bin/python3 10 && alias pip=pip3 .. note:: This is system-wide change which may affect all of the applications. +Fixing broken pip on Ubuntu 16.04 +================================= + +Package ``python3-pip`` could be broken without possibility to upgrade it. +Package has to be removed and installed manually using script `get-pip.py `_.:: + + apt remove python3-pip python3-virtualenv; rm -r ~/.local + rm -r ~/.espressif/python_env && python get-pip.py + +Python 2 deprecation +==================== + +Python 2 reached its `end of life `_ and support for it in ESP-IDF will be removed soon. Please install Python 3.6 or higher. Instructions for popular Linux distributions are listed above. + Next Steps ========== diff --git a/docs/en/get-started/macos-setup-scratch.rst b/docs/en/get-started/macos-setup-scratch.rst index faaeda54df..0d8d353be1 100644 --- a/docs/en/get-started/macos-setup-scratch.rst +++ b/docs/en/get-started/macos-setup-scratch.rst @@ -79,6 +79,10 @@ Build the toolchain:: Toolchain will be built in ``~/esp/ctng-volume/crosstool-NG/builds/xtensa-esp32-elf``. To use it, you need to add ``~/esp/ctng-volume/crosstool-NG/builds/xtensa-esp32-elf/bin`` to ``PATH`` environment variable. +Python 2 deprecation +==================== + +Python 2 reached its `end of life `_ and support for it in ESP-IDF will be removed soon. Please install Python 3.6 or higher. Instructions for macOS are listed above. Next Steps ========== diff --git a/docs/en/get-started/macos-setup.rst b/docs/en/get-started/macos-setup.rst index 8c49a6bd38..88737a4493 100644 --- a/docs/en/get-started/macos-setup.rst +++ b/docs/en/get-started/macos-setup.rst @@ -72,6 +72,11 @@ If the output is similar to ``Python 3.8.5``, your installation has been done su This is system-wide change which may affect all of the applications. +Python 2 deprecation +==================== + +Python 2 reached its `end of life `_ and support for it in ESP-IDF will be removed soon. Please install Python 3.6 or higher. Instructions for macOS are listed above. + Next Steps ========== diff --git a/tools/ci/check_build_warnings.py b/tools/ci/check_build_warnings.py index 285492d595..962451ad7d 100755 --- a/tools/ci/check_build_warnings.py +++ b/tools/ci/check_build_warnings.py @@ -29,6 +29,8 @@ IGNORE_WARNS = [ r"reassigning to symbol", r"changes choice state", r"crosstool_version_check\.cmake", + r"CryptographyDeprecationWarning", + r"Python 3 versions older than 3.6 are not supported." ] ] diff --git a/tools/ci/config/build.yml b/tools/ci/config/build.yml index 0c644001f2..0076c8e42c 100644 --- a/tools/ci/config/build.yml +++ b/tools/ci/config/build.yml @@ -77,8 +77,11 @@ build_ssc: - cd SSC - MAKEFLAGS= ./ci_build_ssc.sh +<<<<<<< HEAD build_esp_idf_tests_make: extends: .build_esp_idf_unit_test_template + variables: + PYTHON_VER: 3 script: - export EXTRA_CFLAGS=${PEDANTIC_CFLAGS} - export EXTRA_CXXFLAGS=${EXTRA_CFLAGS} @@ -94,6 +97,8 @@ build_esp_idf_tests_make: build_esp_idf_tests_cmake: extends: .build_esp_idf_unit_test_template + variables: + PYTHON_VER: 3 script: - export PATH="$IDF_PATH/tools:$PATH" - export EXTRA_CFLAGS=${PEDANTIC_CFLAGS} diff --git a/tools/idf.py b/tools/idf.py index b4ea9bead8..09c04b0dfd 100755 --- a/tools/idf.py +++ b/tools/idf.py @@ -133,6 +133,12 @@ def check_environment(): print("Setting IDF_PATH environment variable: %s" % detected_idf_path) os.environ["IDF_PATH"] = detected_idf_path + # check Python version + if sys.version_info[0] < 3: + print_warning("WARNING: Support for Python 2 is deprecated and will be removed in future versions.") + elif sys.version_info[0] == 3 and sys.version_info[1] < 6: + print_warning("WARNING: Python 3 versions older than 3.6 are not supported.") + # check Python dependencies print("Checking Python dependencies...") try: