kevingurney opened a new pull request, #37211:
URL: https://github.com/apache/arrow/pull/37211

   ### Rationale for this change
   
   Now that the old Feather V1 code and associated build infrastructure has 
been removed (#37204), it makes sense to start building the new, experimental 
MATLAB Interface code by default (without needing to [explicitly specify `-D 
MATLAB_ARROW_INTERFACE=ON`](https://github.com/apache/arrow/tree/main/matlab#build)).
   
   This pull request removes the `MATLAB_ARROW_INTERFACE` flag entirely, since 
setting it to `OFF` when we are building the MATLAB Interface code by default 
would essentially imply that no code should be built.
   
   ### What changes are included in this PR?
   
   1. Removed mention of `MATLAB_ARROW_INTERFACE` flag from MATLAB `README.md`.
   2. Removed conditional check for `MATLAB_ARROW_INTERFACE` flag from MATLAB 
`CMakeLists.txt`.
   3. Removed `MATLAB_ARROW_INTERFACE` flag from `matlab_build.sh` CI script.
   
   ### Are these changes tested?
   
   Yes.
   
   The MATLAB Interface is building as expected by default on my Debian 11 
machine.
   
   ### Are there any user-facing changes?
   
   Yes.
   
   1. The experimental MATLAB Interface APIs will now be built by default 
without users explicitly specifying `-D MATLAB_ARROW_INTERFACE=ON` to `cmake`.
   2. If users specify a value for `MATLAB_ARROW_INTERFACE`, the value will be 
ignored by the CMake build system.


-- 
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