Skip to content

Commit 34bf75c

Browse files
committed
fix: use bash shell instead of MSYS2 for Windows CI builds
GitHub Actions windows-latest runner already has g++ and ar via MinGW. Using bash shell directly avoids PATH issues between MSYS2 and setup-go.
1 parent 5a8a088 commit 34bf75c

1 file changed

Lines changed: 8 additions & 12 deletions

File tree

.github/workflows/ci.yml

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -41,31 +41,27 @@ jobs:
4141
run: |
4242
clang++ --version
4343
44-
- name: Install MinGW (Windows)
45-
if: runner.os == 'Windows'
46-
uses: msys2/setup-msys2@v2
47-
with:
48-
msystem: MINGW64
49-
update: true
50-
install: mingw-w64-x86_64-gcc make
51-
5244
- name: Build (portable, Unix)
5345
if: runner.os != 'Windows'
5446
run: make portable
5547

5648
- name: Build (portable, Windows)
5749
if: runner.os == 'Windows'
58-
shell: msys2 {0}
59-
run: make portable
50+
shell: bash
51+
run: |
52+
g++ -std=c++11 -I. -O2 -c hnsw_wrapper.cc
53+
ar rcs libhnsw.a hnsw_wrapper.o
54+
CGO_CXXFLAGS="-I. -std=c++11" go build
6055
6156
- name: Run tests (Unix)
6257
if: runner.os != 'Windows'
6358
run: make test
6459

6560
- name: Run tests (Windows)
6661
if: runner.os == 'Windows'
67-
shell: msys2 {0}
68-
run: make test
62+
shell: bash
63+
run: |
64+
CGO_CXXFLAGS="-I. -std=c++11" go test -v -count=1 -timeout 120s ./...
6965
7066
benchmark:
7167
name: Benchmark

0 commit comments

Comments
 (0)