Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.github/zephyr: switch to python 3.10 on windows #9468

Merged
merged 1 commit into from
Sep 13, 2024

Commits on Sep 12, 2024

  1. .github/zephyr: switch to python 3.10 on windows

    Zephyr commit b3b8360f3993 ("west: runners: Add `west rtt` command
    with pyocd implementation") adds some functionality to the
    west commands making use of the pipe ("|") operator for
    function return type hinting. As per PEP 604 [1], this operator
    can be used for writing union types starting from python 3.10.
    Since the SOF windows builds use python 3.8 this leads to
    the CI failing.
    
    To fix this, switch to using python 3.10. This is not a
    problem for Linux CI jobs as they already use python 3.10.
    
    The following is a snippet of a failed windows CI job
    regarding this:
    
    File "D:\a\sof\sof\workspace\zephyr\scripts/west_commands\runners\core.py",
    line 780, in ZephyrBinaryRunner
    
        def get_rtt_address(self) -> int | None:
    
        TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'
    
    [1]: https://peps.python.org/pep-0604/
    
    Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
    LaurentiuM1234 committed Sep 12, 2024
    Configuration menu
    Copy the full SHA
    9467934 View commit details
    Browse the repository at this point in the history