tadeja commented on PR #50949: URL: https://github.com/apache/arrow/pull/50949#issuecomment-5414177285
So there are some temporary options here until #50194 gets completed - any preferences? a) Temporarily use [`target_link_options(arrow_s3fs PRIVATE "-Wl,-Bsymbolic-functions")`](https://github.com/apache/arrow/pull/50949/changes/bbefa63c99dd86a08c853c82fbe304481c20f7aa) and revert it together with #50194 (or open a separate "revert PR"). b) Temporarily use previous compiler flag directly with [CXXFLAGS: "-fvisibility-inlines-hidden"](https://github.com/apache/arrow/pull/50949/changes/BASE..361fa5e4c8dee73d0ebf0d83e5b096afad1aa596#diff-facaded51b7d1c9656ae43b449b69aa398fee9129321a0001d97048c00c8cc1cR295-R296) c) Temporarily use `export ARROW_CMAKE_ARGS=-DCMAKE_VISIBILITY_INLINES_HIDDEN=ON &&` for shared `&conda-cpp-command`. This is the [latest commit here](https://github.com/apache/arrow/pull/50949/changes/76eea109fe8c3cb03b95617405efc028874ce8ac) and [jobs pass](https://github.com/apache/arrow/pull/50949#issuecomment-5413052872). d) Preemptively remove `ARROW_S3_MODULE=ON \` from [ci/docker/conda-cpp.dockerfile instead in 50195](https://github.com/apache/arrow/pull/50195/changes#diff-77e83f3866a7b845e723c3eb031ba36dcafc14d833ace16baecaae1a2056cf9eL74) already in this PR, then the S3 module would no longer be tested. e) Do no temp changes, leave jobs `AMD64 Conda C++ AVX2`, `test-conda-cpp` and `test-conda-cpp-valgrind` failing until #50194. (...other options?) -- 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]
