Skip to content

Commit

Permalink
Add zlib to macOS dependencies.
Browse files Browse the repository at this point in the history
This is to fix compilation of Pillow using Python 3.9.

Issue #162.
  • Loading branch information
ChrisCummins committed Mar 30, 2021
1 parent 2087177 commit 5ef1efc
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
10 changes: 9 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,10 @@ jobs:
run: |
brew install bazelisk zlib
python -m pip install -r compiler_gym/requirements.txt -r examples/requirements.txt -r leaderboard/requirements.txt -r tests/requirements.txt
env:
LDFLAGS: -L/usr/local/opt/zlib/lib
CPPFLAGS: -I/usr/local/opt/zlib/include
PKG_CONFIG_PATH: /usr/local/opt/zlib/lib/pkgconfig
if: matrix.os == 'macos-latest'

- name: Test
Expand Down Expand Up @@ -85,8 +89,12 @@ jobs:

- name: Install dependencies (macos)
run: |
brew install bazelisk
brew install bazelisk zlib
python -m pip install -r compiler_gym/requirements.txt -r examples/requirements.txt -r leaderboard/requirements.txt -r tests/requirements.txt
env:
LDFLAGS: -L/usr/local/opt/zlib/lib
CPPFLAGS: -I/usr/local/opt/zlib/include
PKG_CONFIG_PATH: /usr/local/opt/zlib/lib/pkgconfig
if: matrix.os == 'macos-latest'

- name: Install
Expand Down
5 changes: 2 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,11 +68,10 @@ newer equivalents).
If you prefer, you may build from source. This requires a modern C++ toolchain
and bazel.

On macOS you can use the system compiler and install
[bazelisk](https://github.com/bazelbuild/bazelisk) using
On macOS the required dependencies can be installed using
[homebrew](https://docs.brew.sh/Installation):

$ brew install bazelisk
$ brew install bazelisk zlib

On linux, install the required toolchain using:

Expand Down

0 comments on commit 5ef1efc

Please sign in to comment.