Skip to content

Conversation

@konserw
Copy link
Contributor

@konserw konserw commented Sep 7, 2016

Download gmock/gtest from github as link used currently expired

@konserw
Copy link
Contributor Author

konserw commented Sep 7, 2016

Can you see why would most features fail because of tempering with gtest/gmock ?

@konserw
Copy link
Contributor Author

konserw commented Sep 7, 2016

ok I think I've got it - for some reason if GMOCK_ROOT is not a subdirectory it messes e2e tests

@konserw
Copy link
Contributor Author

konserw commented Sep 7, 2016

I've added support for gmock 1.8.0 to FindGmock but we have issue #121

@srbaker
Copy link

srbaker commented Sep 13, 2016

+1 on this. Would love to see this merged soon, as master won't even build currently. :(

@paoloambrosio
Copy link
Member

Nice one. Haven't gone through all lines changed, but I trust the CI pipeline.

I'd squash the first 4 commits into "change gmock/gtest source to github" and keep "support for gtest/gmock >= 1.8.0".

A couple of questions:

  • What about compiling with 1.8.0 in Travis / AppVeyor?
  • Why is 1.7.0 still the default?

@paoloambrosio
Copy link
Member

@srbaker it does compile if you specify GMOCK_PATH (see https://travis-ci.org/cucumber/cucumber-cpp/jobs/158838935). The CI pipeline doesn't currently run on master because it also tries to compile with the sources downloaded from GitHub.

@konserw
Copy link
Contributor Author

konserw commented Sep 14, 2016

I had some problems with gmock 1.8, see issue #121 - i'll need to verify what happened there :)

@paoloambrosio
Copy link
Member

If it doesn't automaticaly work with 1.8.0 we should split the two PRs and remove last commit from this one. In the one for 1.8.0 please add that to the CI build. After that and squashing I think it can be merged to close #122 (and finally merge #116).

@konserw
Copy link
Contributor Author

konserw commented Sep 17, 2016

@paoloambrosio - I've fixed building with gmock 1.8 and squashed it. It turned out to be hard to find but easy to fix issue.
I've also added third commit that adds gmock 1.8.0 to travis CI.
Please let me know if you still want gmock 1.8 support in separate PR.

@konserw
Copy link
Contributor Author

konserw commented Sep 17, 2016

all green :)

@paoloambrosio paoloambrosio merged commit 79c9684 into cucumber:master Sep 17, 2016
paoloambrosio added a commit that referenced this pull request Sep 17, 2016
@konserw konserw deleted the fixFindGmock branch September 17, 2016 12:19
@paoloambrosio
Copy link
Member

Nice one! Green build again 👍

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