Skip to content

Commit 356b0e1

Browse files
committed
docs: state C++14 requirement
Fix: jupp0r#610
1 parent e8b2805 commit 356b0e1

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ option(ENABLE_COMPRESSION "Enable gzip compression" ON)
2323
option(ENABLE_TESTING "Build tests" ON)
2424
option(USE_THIRDPARTY_LIBRARIES "Use 3rdParty submodules" ON)
2525
option(THIRDPARTY_CIVETWEB_WITH_SSL "Enable SSL support for embedded civetweb source code")
26-
option(OVERRIDE_CXX_STANDARD_FLAGS "Force building with -std=c++11 even if the CXXLFAGS are configured differently" ON)
26+
option(OVERRIDE_CXX_STANDARD_FLAGS "Force building with -std=c++14 even if the CXXLFAGS are configured differently" ON)
2727
option(GENERATE_PKGCONFIG "Generate and install pkg-config files" ${UNIX})
2828
option(RUN_IWYU "Run include-what-you-use" OFF)
2929

README.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ int main() {
9292

9393
## Requirements
9494

95-
Using `prometheus-cpp` requires a C++11 compliant compiler. It has been successfully tested with GNU GCC 7.4 on Ubuntu Bionic (18.04) and Visual Studio 2017 (but Visual Studio 2015 should work, too).
95+
Using `prometheus-cpp` requires a C++14 compliant compiler. It has been successfully tested with GNU GCC 7.4 on Ubuntu Bionic (18.04) and Visual Studio 2017.
9696

9797
## Building
9898

@@ -247,20 +247,19 @@ guidelines](https://chris.beams.io/posts/git-commit/).
247247

248248
You can check out this repo and build the library using
249249
``` bash
250-
bazel build //... # build everything
251-
bazel build //core //pull # build just the libraries
250+
bazel build --cxxopt=-std=c++14 //...
252251
```
253252

254253
Run the unit tests using
255254
```
256-
bazel test //...
255+
bazel test --cxxopt=-std=c++14 //...
257256
```
258257

259258
There is also an integration test that
260259
uses [telegraf](https://github.com/influxdata/telegraf) to scrape a
261260
sample server. With telegraf installed, it can be run using
262261
```
263-
bazel test //pull/tests/integration:scrape-test
262+
bazel test --cxxopt=-std=c++14 //pull/tests/integration:scrape-test
264263
```
265264

266265
## Benchmarks

0 commit comments

Comments
 (0)