Copilot commented on code in PR #55427:
URL: https://github.com/apache/arrow/pull/55427#discussion_r4494477989


##########
dev/tasks/matlab/github.yml:
##########
@@ -70,8 +70,8 @@ jobs:
     strategy:
       matrix:
         platform:
-            - { architecture: "AMD64", macos-version: "15-intel", 
architecture-suffix: "x64"}
-            - { architecture: "ARM64", macos-version: "14", 
architecture-suffix: "arm64"}
+            - { architecture: "AMD64", macos-version: "26-intel", 
architecture-suffix: "x64"}
+            - { architecture: "ARM64", macos-version: "26", 
architecture-suffix: "arm64"}

Review Comment:
   These jobs package shared libraries into the published MATLAB toolbox, and 
neither the MATLAB build nor its nested Arrow build sets a macOS deployment 
target. Moving to macOS 26 therefore raises the default minimum OS of those 
libraries, risking compatibility with older macOS versions. Set 
`MACOSX_DEPLOYMENT_TARGET` in the build environment so both builds retain the 
intended minimum version for each architecture.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to