From 8769f842de17b6e651f90fe165df43df8ef1507b Mon Sep 17 00:00:00 2001 From: Tobias Wood Date: Mon, 28 Jun 2021 14:16:25 +0100 Subject: [PATCH] Attempt to fix build version --- .github/workflows/build.yml | 12 ++++++++---- README.md | 2 +- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 3d5122f6..3f951439 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -27,6 +27,7 @@ jobs: steps: - uses: actions/checkout@v2 with: + fetch-depth: 0 submodules: 'recursive' - name: Install GNU tar @@ -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 @@ -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: diff --git a/README.md b/README.md index 130dbff2..6c7df3fa 100644 --- a/README.md +++ b/README.md @@ -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