kevingurney commented on PR #38660: URL: https://github.com/apache/arrow/pull/38660#issuecomment-2027586008
Before merging this in, @sgilmore10 and I were just reviewing the code changes one last time to make sure there weren't any lingering technical issues since this pull request has had a lot of comments over time. We noticed that `libarrow.1500.0.0.dylib` and `libarrow.so.1500.0.0` are for some reason being copied to `fsroot/+arrow` as well as `fsroot/+libmexclass/+proxy/`. They should only be copied once to `fsroot/+libmexclass/+proxy/`. This is likely a result of a bug in the `CMakeLists.txt` for the MATLAB bindings. We are going to take a quick look at this to see if we can push a fix before merging this PR. Our apologies for the slight delay. -- 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]
