Skip to content

Commit

Permalink
Attempt to fix build version
Browse files Browse the repository at this point in the history
  • Loading branch information
Tobias Wood committed Jun 28, 2021
1 parent f8852e6 commit 8769f84
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 5 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ jobs:
steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0
submodules: 'recursive'

- name: Install GNU tar
Expand Down Expand Up @@ -54,20 +55,23 @@ jobs:
else
TC="${{github.workspace}}/cmake/x64-linux-avx2.toolchain.cmake"
fi
cmake -B ${{github.workspace}}/build -S ${{github.workspace}} \
cd ${{github.workspace}}
cmake -B build -S . \
-DCMAKE_BUILD_TYPE=Release \
-DCMAKE_TOOLCHAIN_FILE="$TC"
cmake --build ${{github.workspace}}/build
cmake --build build
- name: Tests
shell: bash
run: |
if [ "${{runner.os}}" != "macOS" ]; then # MacOS runners do not have AVX2
${{github.workspace}}/build/prufung
cd ${{github.workspace}}/build
./prufung
fi
- name: Save release
run: |
cd ${{github.workspace}}
mv ./build/riesling ./
ALL="riesling"
if [ "${{runner.os}}" == "macOS" ]; then
Expand All @@ -82,7 +86,7 @@ jobs:
- name: Upload
uses: actions/upload-artifact@v1
with:
path: ./${{matrix.config.artifact}}
path: ${{github.workspace}}/${{matrix.config.artifact}}
name: ${{matrix.config.artifact}}

release:
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ resulting `riesling` executable to somewhere on your `$PATH`, for instance
`/usr/local/bin`. That's it.

- MacOS Catalina or higher users should use `curl` to download the binary, i.e.
`curl -L https://github.com/spinicist/QUIT/releases/download/v1.0/riesling-macos.tar.gz`.
`curl -L https://github.com/spinicist/riesling/releases/download/v1.0/riesling-macos.tar.gz`.
This is because Safari now sets the quarantine attribute of all downloads,
which prevents them being run as the binary is unsigned. It is possible to
remove the quarantine flag with `xattr`, but downloading with `curl` is more
Expand Down

0 comments on commit 8769f84

Please sign in to comment.