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

Adds travis for linux #53

Merged
merged 6 commits into from
Aug 2, 2018
Merged

Adds travis for linux #53

merged 6 commits into from
Aug 2, 2018

Conversation

jlaura
Copy link
Collaborator

@jlaura jlaura commented Aug 1, 2018

Copy from the SensorUtils repo.

@jlaura jlaura requested a review from ihumphrey August 1, 2018 20:41
@jessemapel
Copy link
Contributor

Does this bring in the CSM dependency?

@jlaura
Copy link
Collaborator Author

jlaura commented Aug 1, 2018

The update does. Check the change to the CMakeLists.txt to see if that looks acceptable to hint (would be mainly used by Travis and Appveyor). I need to see why this PR is not travis testing though. I bet because no .travis.yml is present in the root.

@jlaura jlaura requested a review from jessemapel August 2, 2018 02:46
@jlaura
Copy link
Collaborator Author

jlaura commented Aug 2, 2018

This is getting to be a larger PR than I like. Apologies.

Adds the following:

  • Travis Stub (this PR needs to get merged to have travis fire - this PR does not necessarily have a fully passing travis build. Local builds are working on OS X and Ubuntu 18.04)
  • Addition of GTest as a submodule
  • Modification of root CMakeLists.txt to build GTest at compile time
  • Modification of tests/CMakeLists.txt to turn off GTest discovery. Since we build GTest, we do not need to discover the location.

This PR does not touch appveyor, so that failure is a false positive.

@celbal celbal merged commit 1293847 into DOI-USGS:master Aug 2, 2018
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