Skip to content

Commit

Permalink
build: use CXX_FLAGS to add flags
Browse files Browse the repository at this point in the history
Setting CMAKE_CXX_FLAGS on the command line overrides the defaults
generated by CMake. We want to add flags to the compiler command line,
hence CXX_FLAGS is the correct variable.

- fix some typos
- reformat run parts to improve readability
  • Loading branch information
stefanb2 committed Nov 4, 2023
1 parent d91c48e commit 7bf6114
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions .github/workflows/build-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,10 @@ jobs:
- name: Add msbuild to PATH
uses: microsoft/[email protected]

- name: install node
- name: Install node
uses: actions/setup-node@master

- name: Install node-gpy
- name: Install node-gyp
shell: powershell
run: |
npm install --global node-gyp@latest
Expand All @@ -52,7 +52,12 @@ jobs:
setup-python: false

- name: Configure
run: cmake -B ${{ runner.temp }}/build . "-DCMAKE_CXX_FLAGS=-I${{ github.workspace }}/scrypt-windows/scrypt-1.1.6/lib" -DCMAKE_LIBRARY_PATH=${{ github.workspace }}/scrypt-windows/build/Release
run: >
cmake -B ${{ runner.temp }}/build
"-DCXX_FLAGS=-I${{ github.workspace }}/scrypt-windows/scrypt-1.1.6/lib"
"-DCMAKE_LIBRARY_PATH=${{ github.workspace }}/scrypt-windows/build/Release"
.
- name: Build
run: cmake --build ${{ runner.temp }}/build
run: |
cmake --build ${{ runner.temp }}/build

0 comments on commit 7bf6114

Please sign in to comment.