Skip to content

Commit fc4a49c

Browse files
authored
Update mac.yml: fix pkg-config issue (#681)
* Update mac.yml: fix pkg-config issue * Update .github/workflows/mac.yml * deploy action should only run on the main branch And should be updated to vefsion 4 to work with new GH security changes * Update .github/workflows/mac.yml * Update mac.yml
1 parent 4a52768 commit fc4a49c

File tree

2 files changed

+12
-6
lines changed

2 files changed

+12
-6
lines changed

.github/workflows/mac.yml

+10-5
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,15 @@ jobs:
1616
steps:
1717
- uses: actions/checkout@main
1818

19+
- name: Deps
20+
# git build-essential wget unzip aspcud m4 pkg-config libshp-dev libopenblas-dev liblapacke-dev
21+
# homebrew in the workers comes pre-installed with the old pkg-config, which conflicts with the new pkgconf
22+
run: |
23+
brew unlink pkg-config
24+
brew install pkgconf aspcud openblas
25+
brew unlink pkgconf
26+
brew link pkg-config
27+
1928
- name: Use OCaml
2029
uses: ocaml/setup-ocaml@v2
2130
with:
@@ -24,10 +33,6 @@ jobs:
2433
# *.opam
2534
dune-cache: true
2635
allow-prerelease-opam: true
27-
28-
- name: Deps
29-
# git build-essential wget unzip aspcud m4 pkg-config libshp-dev libopenblas-dev liblapacke-dev
30-
run: brew install aspcud openblas
3136

3237
- name: OCaml Deps
3338
run: opam install ocaml-compiler-libs alcotest conf-openblas ctypes dune-configurator stdio npy
@@ -38,4 +43,4 @@ jobs:
3843
- name: Run tests
3944
run: |
4045
opam install owl-base
41-
opam exec -- dune runtest -j 1 --no-buffer -p owl
46+
opam exec -- dune runtest -j 1 --no-buffer -p owl

.github/workflows/main.yml

+2-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@ jobs:
3939
opam exec -- dune build @doc
4040
4141
- name: Deploy
42-
uses: peaceiris/actions-gh-pages@v3
42+
uses: peaceiris/actions-gh-pages@v4
43+
if: github.ref == 'refs/heads/main'
4344
with:
4445
github_token: ${{ secrets.GITHUB_TOKEN }}
4546
publish_dir: _build/default/_doc/_html

0 commit comments

Comments
 (0)