Skip to content

Conversation

@casaroli
Copy link
Contributor

@casaroli casaroli commented Oct 12, 2024

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

Fixes current checks CI.

Testing

See the checks CI run.

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.
@github-actions github-actions bot added Area: CI Size: XS The size of the change in this PR is very small labels Oct 12, 2024
@casaroli casaroli changed the title ci: use venv for check [FIX CI] ci: use venv for check Oct 12, 2024
@casaroli casaroli marked this pull request as ready for review October 12, 2024 12:01
@GUIDINGLI
Copy link
Contributor

First CI check pass, no need wait all CI done, merge

@GUIDINGLI GUIDINGLI closed this Oct 12, 2024
@GUIDINGLI GUIDINGLI reopened this Oct 12, 2024
@GUIDINGLI GUIDINGLI merged commit a2b129f into apache:master Oct 12, 2024
@xiaoxiang781216
Copy link
Contributor

@casaroli could you cherry-pick the change to apps side?

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

Labels

Area: CI Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants