Cleanup #7668
closedremove custom googletest (gtest) copy from source tree
100%
Description
Remove the the potentially outdated gtest code from the source tree to keep it up-to-date:
Needed steps:
- since googletest is hosted via SVN [1], we need a copy of the repo in the github ceph project. Copy it from SVN, add documentation on how to keep it up-to-date
- verify and apply changes from current ceph git gtest to the new git repo
- remove code from ceph.git source tree
- add gtest to submodules
Updated by Greg Farnum about 10 years ago
I have no expertise here and am down with making this change if it works out better, but when we initially grabbed gtest a wholesale copy was the recommended approach (or so I was told).
Updated by Danny Al-Gaaf about 9 years ago
Can someone please create an empty gtest and gmock git repository in the ceph namespace so that we can move this code there (I have no permissions to do that)? I would be much easier to handle these dependencies at least in separate git repos and include it from there as submodule.
Updated by Danny Al-Gaaf about 9 years ago
Branch with solution can be found here: https://github.com/ceph/ceph/tree/wip-da-gmock-submodule
The problem is still: I need an empty gmock and gtest repo under the github ceph account!
Updated by Danny Al-Gaaf about 9 years ago
Pull request: https://github.com/ceph/ceph/pull/3846
Updated by Danny Al-Gaaf about 9 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 100
Updated by Danny Al-Gaaf about 9 years ago
- % Done changed from 100 to 80
There are issues with this merge on master. We need to fix gitbuilder and mirror the repos in the sepia git
Updated by Danny Al-Gaaf about 9 years ago
This depends on http://tracker.ceph.com/issues/11029 and https://github.com/ceph/autobuild-ceph/pull/24
Updated by Danny Al-Gaaf about 9 years ago
- Status changed from In Progress to Resolved
- % Done changed from 80 to 100