Skip to content

Commit 3046501

Browse files
authored
apacheGH-41435: [CI][MATLAB] Add job to build and test MATLAB Interface on macos-14 (apache#41592)
### Rationale for this change Currently, the MATLAB interface is built and tested on `macos-12` - not `macos-14` - because the version of `mathworks/libmexclass` depends on used to not support `macos-14`. However, now that apache#41400 is closed, the version of `mathworks/libmexclass` the MATLAB interface depends on works on `macos-14`, so we will be able to build and test the MATLAB interface on `macos-14`. **Note**: When adding support for ARM-based macOS builds, we discovered an issue with the way in which we package the MLTBX files for the MATLAB Interface to Arrow. Currently, we bundle all shared libraries for all platforms (.dll, .dylib, and .so) into one large "monolithic" MLTBX file. Unfortunately, putting all platform-specific files into one MLTBX file poses an issue when we support multiple ISAs (e.g. x86 and ARM) because builds for the same operating system with different ISAs will have the same shared library file names. In other words, we will have a library named libarrowproxy.dylib for both ARM and x86 macOS builds. Therefore, we are going to hold off on adding ARM-based macOS builds to the crossbow packaging workflow for now until we have a chance to properly explore alternative packaging approaches. For example, we may want to consider having platform-specific MLTBX files. However, we still think it is worthwhile to add CI support for `macos-14` in the meantime. ### What changes are included in this PR? 1. Added workflow to build and test the MATLAB interface on `macos-14` as well as `macos-12`. ### Are these changes tested? N/A. ### Are there any user-facing changes? No. ### Future Directions 1. Add crossbow packaging workflow on `macos-14` once we determine how to package the interface for both ARM-based and Intel-based mac ISAs. * GitHub Issue: apache#41435 Authored-by: Sarah Gilmore <[email protected]> Signed-off-by: Sarah Gilmore <[email protected]>
1 parent f672027 commit 3046501

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

.github/workflows/matlab.yml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,16 @@ jobs:
9898
select-by-folder: matlab/test
9999
strict: true
100100
macos:
101-
name: AMD64 macOS 12 MATLAB
102-
runs-on: macos-12
101+
name: ${{ matrix.architecture }} macOS ${{ matrix.macos-version }} MATLAB
102+
runs-on: macos-${{ matrix.macos-version }}
103103
if: ${{ !contains(github.event.pull_request.title, 'WIP') }}
104+
strategy:
105+
matrix:
106+
include:
107+
- architecture: AMD64
108+
macos-version: "12"
109+
- architecture: ARM64
110+
macos-version: "14"
104111
steps:
105112
- name: Check out repository
106113
uses: actions/checkout@v4

0 commit comments

Comments
 (0)