Skip to content

Commit

Permalink
Merge pull request #192 from jannis-baum/issue/191-brew-install-doesn…
Browse files Browse the repository at this point in the history
…t-put-viv-on-path-macos-sequoia

macOS 15.1.1 fixes
  • Loading branch information
jannis-baum authored Nov 23, 2024
2 parents 23c3e71 + 878c28e commit 5d07d16
Showing 1 changed file with 24 additions and 1 deletion.
25 changes: 24 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,15 +66,26 @@ jobs:
steps:
- name: checkout
uses: actions/checkout@v4
- name: get Homebrew Node
# lists node dependencies, these need to be mirrored for the Homebrew package
run: |
brew update
brew install node
brew deps --installed --direct node > macos-deps.txt
- name: build
run: |
yarn
VIV_VERSION="${{ github.ref_name }}" make macos
- name: upload artifact
- name: upload artifact (macOS build)
uses: actions/upload-artifact@v4
with:
name: vivify-macos
path: build/macos
- name: upload artifact (macOS dependency list)
uses: actions/upload-artifact@v4
with:
name: macos-deps
path: macos-deps.txt

release:
name: Release
Expand Down Expand Up @@ -103,6 +114,12 @@ jobs:
chmod +x ./vivify-linux/* ./vivify-macos/*
tar -czf vivify-linux.tar.gz vivify-linux
tar -czf vivify-macos.tar.gz vivify-macos
# macos dependencies -----------------------------------------------------
- name: download macos dependencies
uses: actions/download-artifact@v4
with:
name: macos-deps
path: macos-deps
# release notes ----------------------------------------------------------
# actions/checkout seems to have a bug where annotated tags are converted
# to simple tags so we have to fetch them explicitly again
Expand All @@ -122,6 +139,12 @@ jobs:
>> release.md
echo -e "macOS: \`$(sha256sum vivify-macos.tar.gz | cut -d' ' -f1)\`" \
>> release.md
echo -e '\n**macOS Homebrew dependencies**\n\n```plain' \
>> release.md
cat macos-deps/macos-deps.txt \
>> release.md
echo -e '```' \
>> release.md
- name: print release notes for logs
run: cat release.md

Expand Down

0 comments on commit 5d07d16

Please sign in to comment.