Skip to content

Conversation

@wjones127
Copy link
Member

@wjones127 wjones127 commented Apr 7, 2022

Summary of Changes

  • Added instruction on C++ guide to unset CPATH if on MacOS and using Homebrew. If I did not do that, the build would prefer the header files of Arrow and GTest from homebrew while linking to the shared libraries from Conda, creating linking errors.
  • Added ccache.
  • Narrowed .clang-tidy checks to exclude the *, which I think selects everything? Or am I wrong about that?
  • Run clang-tidy regardless of compiler, including in CI, where it was not previously running.
  • Change Flight example port away from 5000 because Apple's Control Center process often occupies that port.

Steps to test on MacOS M1

# clear old environment if needed
conda env remove -n cookbook-cpp

conda create -n cookbook-cpp --file cpp/conda-osx-arm64.lock
conda activate cookbook-cpp

# If using homebrew, need to hide those include paths
unset CPATH

make cpp

@wjones127 wjones127 changed the title Improvements to support building on MacOS [C++] Lint in CI and various MacOS build improvements Apr 8, 2022
@wjones127 wjones127 closed this Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant