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

Install Ruby and Python for macOS using setup-LANG #2909

Merged
merged 5 commits into from
Oct 17, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 16 additions & 7 deletions .github/actions/setup-dependencies/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,26 @@ runs:
shell: bash
if: runner.os == 'macOS'

# Python3 is already installed though Homebrew
- name: Install brew dependencies
run: brew install ruby node php lmdb mcpp || true
run: brew install node php lmdb mcpp || true
shell: bash
if: runner.os == 'macOS'

- name: Add Ruby and Python interpreters from brew to PATH
run: |
echo "$(brew --prefix ruby)/bin" >> $GITHUB_PATH
echo "$(brew --prefix python3)/bin" >> $GITHUB_PATH
shell: bash
# - name: Add Ruby and Python interpreters from brew to PATH
# run: |
# echo "$(brew --prefix ruby)/bin" >> $GITHUB_PATH
# echo "$(brew --prefix python3)/bin" >> $GITHUB_PATH
# shell: bash
# if: runner.os == 'macOS'

- uses: actions/setup-python@v5
with:
python-version: "3.12"
if: runner.os == 'macOS'

- uses: ruby/setup-ruby@v1
with:
ruby-version: "3.3"
if: runner.os == 'macOS'

- name: Install testing dependencies from pip
Expand Down
7 changes: 0 additions & 7 deletions .github/workflows/docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ on:

jobs:
ci:

runs-on: macos-15
steps:
- name: Checkout repository
Expand All @@ -27,9 +26,6 @@ jobs:
- name: Install doxygen and graphviz (a dependency of Doxygen for generating diagrams)
run: brew install doxygen graphviz || true

- name: Install virtualenv for Python
run: pipx install virtualenv

- name: Install docfx for C# API reference
run: brew install docfx

Expand Down Expand Up @@ -66,11 +62,8 @@ jobs:
run: |
make -C ../../cpp slice2py Ice IceDiscovery IceLocatorDiscovery
make -C ../
virtualenv .venv
source .venv/bin/activate
pip install -r requirements.txt
make html
deactivate

- name: Generate API reference for Swift
run: |
Expand Down
Loading