You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/15 13:53:21 UTC

[GitHub] [arrow] raulcd commented on pull request #14643: MINOR: [Python][CI] Add DYLD_LIBRARY_PATH to avoid requiring PYARROW_BUNDLE_ARROW_CPP on macOS job

raulcd commented on PR #14643:
URL: https://github.com/apache/arrow/pull/14643#issuecomment-1315340427

   @kou @AlenkaF I tested removing the skip for the cython test and also adding to the test the following, as seen on the intermediate commits, but did not solve the issue for the cython test, that's why I opted to continue skipping it:
   ```
           elif sys.platform == 'darwin':
               delim, var = ':', 'LD_LIBRARY_PATH'
               subprocess_env['DYLD_LIBRARY_PATH'] = delim.join(
                   pa.get_library_dirs() + [
                       subprocess_env.get('DYLD_LIBRARY_PATH', '')
                   ]
               )
   ```
   Let me know if you want me to create a JIRA for this change or if you are ok with it being `MINOR`


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org