Skip to content

Commit

Permalink
ci, doc: use pip install --user
Browse files Browse the repository at this point in the history
Since now most modern linux distributions will refuse to install
packages using pip install, we can use --user, so it does not
interfere with the systems site-packages.

The ideal solution is, however, to add cmake-format to the
nuttx-ci-linux container.

This is to avoid the following error when we did:

$ pip install cmake-format

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.
  • Loading branch information
casaroli committed Oct 12, 2024
1 parent 3b273b1 commit 3fd82ab
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
- name: Check Pull Request
run: |
echo "::add-matcher::nuttx/.github/nxstyle.json"
pip install cmake-format
pip install --user cmake-format
cd nuttx
commits="${{ github.event.pull_request.base.sha }}..HEAD"
git log --oneline $commits
Expand Down
2 changes: 1 addition & 1 deletion tools/checkpatch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ check_file() {
if ! command -v cmake-format &> /dev/null; then
if [ $cmake_warning_once == 0 ]; then
echo -e "\ncmake-format not found, run following command to install:"
echo " $ pip install cmake-format"
echo " $ pip install --user cmake-format"
cmake_warning_once=1
fi
fail=1
Expand Down

0 comments on commit 3fd82ab

Please sign in to comment.