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

Improve gtest finding, support local checkout. #774

Merged
merged 1 commit into from
Feb 11, 2018

Conversation

iphydf
Copy link
Member

@iphydf iphydf commented Feb 11, 2018

Also fix library dependency order for monolith test.


This change is Reviewable

@iphydf iphydf added this to the v0.2.0 milestone Feb 11, 2018
@iphydf iphydf force-pushed the find-gtest branch 6 times, most recently from 14e5c9a to 5fe9ed9 Compare February 11, 2018 00:52
@iphydf iphydf added the P0 Critical priority label Feb 11, 2018
@iphydf
Copy link
Member Author

iphydf commented Feb 11, 2018

Build is broken.

@iphydf
Copy link
Member Author

iphydf commented Feb 11, 2018

This PR fixes the build.

@Diadlo
Copy link

Diadlo commented Feb 11, 2018

cmake_minimum_required(VERSION 2.8.6)

set(CMAKE_C_STNADARD ...)

I'm not sure, but seems this feature is unavaliable on 2.8.8

@iphydf
Copy link
Member Author

iphydf commented Feb 11, 2018

Upped requirement to 3.1.0.

Also fix library dependency order for monolith test.
@robinlinden
Copy link
Member

:lgtm_strong:


Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@Diadlo
Copy link

Diadlo commented Feb 11, 2018

:lgtm_strong:


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@robinlinden robinlinden merged commit 2e74db5 into TokTok:master Feb 11, 2018
@iphydf iphydf deleted the find-gtest branch February 11, 2018 12:46
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 Critical priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants