Merge pull request #127 from still-flow/patch-1

Fix some Github CI red checks

.github/workflows/build.sh

@@ -2,7 +2,7 @@ set -e o pipefail
 source .github/workflows/gh_ci_envvars.sh
 
 
-GH_MYMAKE_ARGS=""
+GH_MYMAKE_ARGS="-fPIC"
 GH_AUTOTOOLS_CXXFLAGS="-W -Wall -Wextra -Wno-unused-parameter -Wno-maybe-uninitialized"
 
 HYPERROGUE_USE_GLEW=$GH_HYP_GLEW
@@ -14,8 +14,8 @@ export HYPERROGUE_USE_PNG=${HYPERROGUE_USE_PNG: -1}
 HYPERROGUE_USE_ROGUEVIZ=$GH_HYP_RVIZ
 export HYPERROGUE_USE_ROGUEVIZ=${HYPERROGUE_USE_ROGUEVIZ: -1}
 if [[ "$GH_HYP_RVIZ" == "rviz_1" ]]; then
-  GH_MYMAKE_ARGS+=" -rv"
-  GH_AUTOTOOLS_CXXFLAGS+=" -DCAP_ROGUEVIZ=1"
+  GH_MYMAKE_ARGS+=" -std=c++17 -rv"
+  GH_AUTOTOOLS_CXXFLAGS+=" -std=c++17 -DCAP_ROGUEVIZ=1"
 fi
 
 export CC=$GH_COMPILER

.github/workflows/github_ci.yml

@@ -48,10 +48,16 @@ jobs:
         hyper_use_rviz: [rviz_1, rviz_0]
         hyper_use_png: [png_1]
         hyper_use_glew: [glew_1]
+    defaults:
+      run:
+        shell: msys2 {0}
     steps:
+    - uses: msys2/setup-msys2@v2
+      with:
+        update: true
+        install: 'pactoys'
     - uses: actions/checkout@v2
     - name: envvars_setup
-      shell: cmd.exe /D /E:ON /V:OFF /C "C:\msys64\msys2_shell.cmd -defterm -msys2 -no-start -full-path -here {0}"
       run: |
         cat << ENDOFVARS > .github/workflows/gh_ci_envvars.sh
         GH_OS=${{ matrix.os }}
@@ -62,11 +68,13 @@ jobs:
         GH_HYP_GLEW=${{ matrix.hyper_use_glew }}
         ENDOFVARS
     - name: install_deps
-      run: cmd.exe /D /E:ON /V:OFF /C "C:\msys64\msys2_shell.cmd -defterm -msys2 -no-start -full-path -here .github/workflows/install_deps.sh"
+      env:
+        MSYSTEM: MSYS
+      run: .github/workflows/install_deps.sh
     - name: build
-      run: cmd.exe /D /E:ON /V:OFF /C "C:\msys64\msys2_shell.cmd -defterm -mingw64 -no-start -full-path -here .github/workflows/build.sh"
+      run: .github/workflows/build.sh
     - name: test_simple
-      run: cmd.exe /D /E:ON /V:OFF /C "C:\msys64\msys2_shell.cmd -defterm -mingw64 -no-start -full-path -here .github/workflows/test_simple.sh"
+      run: .github/workflows/test_simple.sh
 
   emscripten:
     runs-on: ubuntu-latest

.github/workflows/install_deps.sh

@@ -4,7 +4,7 @@ source .github/workflows/gh_ci_envvars.sh
 
 GH_DEPS_UBUNTU="$GH_COMPILER libsdl1.2-dev libsdl-ttf2.0-dev libsdl-gfx1.2-dev libsdl-mixer1.2-dev"
 GH_DEPS_MACOS="sdl sdl_gfx sdl_mixer sdl_ttf"
-GH_DEPS_MINGW64="$GH_COMPILER:x SDL:x SDL_ttf:x SDL_gfx:x SDL_mixer:x"
+GH_DEPS_MINGW64="$GH_COMPILER:x SDL:x SDL_ttf:x SDL_gfx:x SDL_mixer:x make"
 
 if [[ "$GH_HYP_GLEW" == "glew_1" ]]; then
   GH_DEPS_UBUNTU+=" libglew-dev"

Makefile.simple

@@ -81,14 +81,14 @@ endif
 
 ifeq (${TOOLCHAIN},clang)
   CXXFLAGS_STD = -std=c++11
-  CXXFLAGS_EARLY += -march=native
+  CXXFLAGS_EARLY += -march=native -fPIC
   CXXFLAGS_EARLY += -W -Wall -Wextra -Werror -pedantic
   CXXFLAGS_EARLY += -Wno-unused-parameter -Wno-implicit-fallthrough -Wno-maybe-uninitialized -Wno-unknown-warning-option
 endif
 
 ifeq (${TOOLCHAIN},gcc)
   CXXFLAGS_STD = -std=c++11
-  CXXFLAGS_EARLY += -march=native
+  CXXFLAGS_EARLY += -march=native -fPIC
   CXXFLAGS_EARLY += -W -Wall -Wextra -Werror -pedantic
   CXXFLAGS_EARLY += -Wno-unused-parameter -Wno-implicit-fallthrough -Wno-maybe-uninitialized
 endif