Skip to content

Commit

Permalink
Fix mac ci failures (#1130)
Browse files Browse the repository at this point in the history
* update setup haxe

* [ci] Add arm64 support to test suite

* [ci] Run x86_64 mac tests as well

* [ci] Use arm64 output folder for arm64 tests

This is already in .gitignore

---------

Co-authored-by: Aidan Lee <[email protected]>
  • Loading branch information
tobil4sk and Aidan63 authored Jul 15, 2024
1 parent 6d0b37b commit 40025d6
Show file tree
Hide file tree
Showing 5 changed files with 41 additions and 19 deletions.
20 changes: 17 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,18 +30,32 @@ jobs:
ext: .dll
os: windows-latest

MacOS:
MacOS64:
strategy:
fail-fast: false
matrix:
haxe: [ 4.3.4, latest ]
uses: ./.github/workflows/test.yml
name: Test MacOS
name: Test MacOS (x86_64)
with:
haxe: ${{ matrix.haxe }}
arch: 64
sep: /
ext: .dylib
os: macos-13

MacOSArm:
strategy:
fail-fast: false
matrix:
haxe: [ 4.3.4, latest ]
uses: ./.github/workflows/test.yml
name: Test MacOS (Arm64)
with:
haxe: ${{ matrix.haxe }}
arch: Arm64
sep: /
ext: .dylib
os: macos-latest

Linux32:
Expand Down Expand Up @@ -70,4 +84,4 @@ jobs:
arch: 64
sep: /
ext: .dso
os: ubuntu-latest
os: ubuntu-latest
2 changes: 1 addition & 1 deletion .github/workflows/package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
- name: Checkout
uses: actions/checkout@v3

- uses: Aidan63/setup-haxe@e4b347bc45596200a68804b232be9d73f80868a6
- uses: Aidan63/setup-haxe@3d3101bcd0a2001699fc8295f4d9eddd0724d3e9
with:
haxe-version: 4.3.4

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/setup/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ runs:
using: composite
steps:
- name: install haxe
uses: Aidan63/setup-haxe@e4b347bc45596200a68804b232be9d73f80868a6
uses: Aidan63/setup-haxe@3d3101bcd0a2001699fc8295f4d9eddd0724d3e9
with:
haxe-version: ${{ inputs.haxe }}

Expand Down
31 changes: 17 additions & 14 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ on:
required: true
type: string

env:
HXCPP_ARCH_FLAG: ${{ inputs.arch == 'Arm64' && 'HXCPP_ARM64' || format('HXCPP_M{0}', inputs.arch) }}

jobs:
haxe:
runs-on: ${{ inputs.os }}
Expand All @@ -33,7 +36,7 @@ jobs:
with:
haxe: ${{ inputs.haxe }}
- name: build
run: haxe compile.hxml -D HXCPP_M${{ inputs.arch }}
run: haxe compile.hxml -D ${{ env.HXCPP_ARCH_FLAG }}
- name: run
run: bin${{ inputs.sep }}TestMain

Expand All @@ -52,13 +55,13 @@ jobs:
haxe: ${{ inputs.haxe }}
- name: build project
working-directory: test/cffi/project
run: haxelib run hxcpp build.xml -Ddebug -DHXCPP_M${{ inputs.arch }}
run: haxelib run hxcpp build.xml -Ddebug -D${{ env.HXCPP_ARCH_FLAG }}
- name: build
run: haxe compile.hxml --debug -D HXCPP_M${{ inputs.arch }}
run: haxe compile.hxml --debug -D ${{ env.HXCPP_ARCH_FLAG }}
- name: build (utf8)
run: haxe compile-utf8.hxml --debug -D HXCPP_M${{ inputs.arch }}
run: haxe compile-utf8.hxml --debug -D ${{ env.HXCPP_ARCH_FLAG }}
- name: build (neko)
run: haxe compile-neko.hxml --debug -D HXCPP_M${{ inputs.arch }}
run: haxe compile-neko.hxml --debug -D ${{ env.HXCPP_ARCH_FLAG }}
- name: copy
run: cp project/ndll/*/prime${{ inputs.ext }} bin/neko/prime.ndll
- name: run
Expand Down Expand Up @@ -91,7 +94,7 @@ jobs:
with:
haxe: ${{ inputs.haxe }}
- name: build
run: haxe compile.hxml ${{ matrix.suffix }} -D HXCPP_M${{ inputs.arch }}
run: haxe compile.hxml ${{ matrix.suffix }} -D ${{ env.HXCPP_ARCH_FLAG }}
- name: run
run: bin${{ inputs.sep }}TestMain${{ matrix.suffix }}

Expand All @@ -109,10 +112,10 @@ jobs:
with:
haxe: ${{ inputs.haxe }}
- name: build
run: haxe compile.hxml -D HXCPP_M${{ inputs.arch }}
run: haxe compile.hxml -D ${{ env.HXCPP_ARCH_FLAG }}
- name: run
run: bin${{ inputs.sep }}App-debug

native:
runs-on: ${{ inputs.os }}
name: native
Expand All @@ -127,7 +130,7 @@ jobs:
with:
haxe: ${{ inputs.haxe }}
- name: build
run: haxe compile.hxml -D HXCPP_M${{ inputs.arch }}
run: haxe compile.hxml -D ${{ env.HXCPP_ARCH_FLAG }}
- name: run
run: bin${{ inputs.sep }}Native

Expand Down Expand Up @@ -167,7 +170,7 @@ jobs:
- name: build
run: haxe compile${{ inputs.arch }}.hxml
- name: run
run: cpp${{ inputs.arch }}${{ inputs.sep }}Test
run: ${{ inputs.arch == 'Arm64' && 'arm64' || format('cpp{0}', inputs.arch) }}${{ inputs.sep }}Test

cppia:
strategy:
Expand All @@ -190,9 +193,9 @@ jobs:
with:
haxe: ${{ inputs.haxe }}
- name: build host
run: haxe compile-host.hxml -D HXCPP_M${{ inputs.arch }}
run: haxe compile-host.hxml -D ${{ env.HXCPP_ARCH_FLAG }}
- name: build client
run: haxe compile-client.hxml -D HXCPP_M${{ inputs.arch }}
run: haxe compile-client.hxml -D ${{ env.HXCPP_ARCH_FLAG }}
- name: run
run: bin${{ inputs.sep }}CppiaHost bin${{ inputs.sep }}client.cppia ${{ matrix.suffix }}

Expand Down Expand Up @@ -225,6 +228,6 @@ jobs:
- name: install haxe libs
run: haxelib install compile-cpp.hxml --always
- name: build
run: haxe compile-cpp.hxml -D HXCPP_M${{ inputs.arch }} -D no_http
run: haxe compile-cpp.hxml -D ${{ env.HXCPP_ARCH_FLAG }} -D no_http
- name: run
run: bin${{ inputs.sep }}cpp${{ inputs.sep }}TestMain-debug
run: bin${{ inputs.sep }}cpp${{ inputs.sep }}TestMain-debug
5 changes: 5 additions & 0 deletions test/std/compileArm64.hxml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
-m Test
-D HXCPP_ARM64
-L hx4compat
-L utest
--cpp arm64

0 comments on commit 40025d6

Please sign in to comment.