-
Notifications
You must be signed in to change notification settings - Fork 77
CMake followups #244
Comments
While working on various changes (#284 and #283), I experimented with replacing the current Are you interested in a PR with this (that is, replace Then (because the amount of code is reduced) it will also be easier to implement an option to switch between |
I built all components with Debug (x64) and ran all 189 tests successfully on MSVC 2017 v15.9.7. 👍 I also built with RelWithDebugInfo (x64) - that only discovered 186 tests, and of the 186 tests, these 6 tests failed: [26.02.2019 19.51.33 Informational] [ FAILED ] 6 tests, listed below: |
I'm happy to replace If v3.11 becomes a problem for people, we can (later) try to import the n.b. Our Travis config specifies Trusty which ships with CMake 3.9.2, so maybe switch (only) the CMake build to Xenial for now. |
See #296 for benchmark support |
I am interested to use opencensus in one of my projects, but I would need the following:
|
Experimental CMake support was added in PR #238 to address issue #86. This issue is to track followup work from the PR:
The text was updated successfully, but these errors were encountered: