Skip to content

[Fixes CI now!] ci: use venv for check#2712

Merged
xiaoxiang781216 merged 1 commit into
apache:masterfrom
casaroli:fix-check
Oct 13, 2024
Merged

[Fixes CI now!] ci: use venv for check#2712
xiaoxiang781216 merged 1 commit into
apache:masterfrom
casaroli:fix-check

Conversation

@casaroli

@casaroli casaroli commented Oct 12, 2024

Copy link
Copy Markdown
Contributor

Summary

To avoid the following CI error:

This environment is externally managed
--> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    If you wish to install a non-Debian packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.

    See /usr/share/doc/python3.12/README.venv for more information.

Impact

CI checks are fixed now

Testing

Check the logs.

To avoid the following CI error:

This environment is externally managed
--> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    If you wish to install a non-Debian packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.

    See /usr/share/doc/python3.12/README.venv for more information.
@casaroli casaroli changed the title ci: use venv for check [Fixes CI now!] ci: use venv for check Oct 12, 2024

@cederom cederom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @casaroli !! :-)

@cederom

cederom commented Oct 12, 2024

Copy link
Copy Markdown
Contributor

As in apache/nuttx#14176 this fixes the CI "build / fetch-source" worker error on nuttx-apps repo too :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants