Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

- Enable C++11 for GoogleTest external project for sake of consistency with the project. #290

Merged
merged 2 commits into from
Feb 19, 2020

Conversation

HamedSabri-adsk
Copy link
Contributor

@HamedSabri-adsk HamedSabri-adsk commented Feb 18, 2020

I also bumped the CMake Min version to 3.12.0 and removed CMAKE_MACOSX_RPATH flag as is no longer needed.

…y with the project.

- I also pumped the CMake Min version to 3.12.0 and removed CMAKE_MACOSX_RPATH flag as is no longer needed.
@kxl-adsk kxl-adsk added the do-not-merge-yet Development is not finished, PR not ready for merge label Feb 19, 2020
@kxl-adsk kxl-adsk removed the do-not-merge-yet Development is not finished, PR not ready for merge label Feb 19, 2020
@kxl-adsk kxl-adsk merged commit c75e696 into dev Feb 19, 2020
@kxl-adsk kxl-adsk deleted the sabrih/enable_c++11_gtest_external_project branch February 19, 2020 22:09
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.

3 participants