Skip to content

Commit

Permalink
Install website requirements in venv (#1749)
Browse files Browse the repository at this point in the history
Apparently, macOS doesn't like to pip install into the system-wide
location, it prefers a venv.

Signed-off-by: Cary Phillips <[email protected]>
  • Loading branch information
cary-ilm authored May 13, 2024
1 parent 810e456 commit 37594c2
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions .github/workflows/website.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,14 @@ on:
- RB-*
paths:
- 'website/**'
- '.github/workflows/website_workflow.yml'
- '.github/workflows/website.yml'

pull_request:
branches-ignore:
- RB-*
paths:
- 'website/**'
- '.github/workflows/website_workflow.yml'
- '.github/workflows/website.yml'

permissions:
contents: read
Expand Down Expand Up @@ -61,12 +61,17 @@ jobs:
run: ./website/scripts/install_doxygen.sh ${DOXYGEN_VERSION}
shell: bash
- name: Install sphinx requirements
run: pip3 install -r website/requirements.txt
- name: Configure
run: cmake .. -DBUILD_WEBSITE=ON -DCMAKE_VERBOSE_MAKEFILE=ON
run: |
python3 -m venv _venv
source _venv/*/activate
pip3 install -r ../website/requirements.txt
cmake .. -DBUILD_WEBSITE=ON -DCMAKE_VERBOSE_MAKEFILE=ON
working-directory: _build
shell: bash
- name: Build
run: cmake --build . --target website --config Release
run: |
cmake --build . --target website --config Release
working-directory: _build
shell: bash


0 comments on commit 37594c2

Please sign in to comment.