Mirrors/cppbestpractices
1
0
Fork 0
mirror of https://github.com/cpp-best-practices/cppbestpractices.git synced 2025-12-17 11:14:35 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

Mention coverity scan and update info on appveyor

Browse Source
This commit is contained in:
Jason Turner
2015-06-08 09:56:46 -06:00
parent 525b526e94
commit 2de6bbce8c
1 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
02-Use_the_Tools_Available.md
View File

@@ -20,7 +20,7 @@ Use an industry standard widely accepted build tool. This prevents you from rein
* [Waf](https://waf.io/) * [Waf](https://waf.io/)
* [FASTBuild](http://www.fastbuild.org/) * [FASTBuild](http://www.fastbuild.org/)
* [Ninja](https://martine.github.io/ninja/) - can greatly improve the incremental build time of your larger projects. Can be used as a target for CMake. * [Ninja](https://martine.github.io/ninja/) - can greatly improve the incremental build time of your larger projects. Can be used as a target for CMake.
* [Bazel](http://bazel.io/) * [Bazel](http://bazel.io/) - Note: MacOS and Linux only.
Remember, it's not just a build tool, it's also a programming language. Try to maintain good clean build scripts and follow the recommended practices for the tool you are using. Remember, it's not just a build tool, it's also a programming language. Try to maintain good clean build scripts and follow the recommended practices for the tool you are using.
@@ -34,15 +34,16 @@ Continuous Integration (CI) tools automatically build the source code as changes
* works well with C++ * works well with C++
* designed for use with GitHub * designed for use with GitHub
* free for public repositories on GitHub * free for public repositories on GitHub
* [Hudson CI](http://hudson-ci.org/)
* [AppVeyor](http://www.appveyor.com/) * [AppVeyor](http://www.appveyor.com/)
* supports Windows and MSVC * supports Windows, MSVC and MinGW
* free for public repositoris on GitHub
* [Hudson CI](http://hudson-ci.org/)
* [Decent CI](https://github.com/lefticus/decent_ci) * [Decent CI](https://github.com/lefticus/decent_ci)
* simple ad-hoc continuous integration that posts results to GitHub * simple ad-hoc continuous integration that posts results to GitHub
* supports Windows, OS X, and Linux * supports Windows, OS X, and Linux
* used by [ChaiScript](http://chaiscript.com/ChaiScript-BuildResults/full_dashboard.html) * used by [ChaiScript](http://chaiscript.com/ChaiScript-BuildResults/full_dashboard.html)
If you have an open source, publicly-hosted project on GitHub, go enable travis-ci integration right now. We'll wait for you to come back. For a simple example of how to enable it for your C++ CMake-based application, see here: https://github.com/ChaiScript/ChaiScript/blob/master/.travis.yml If you have an open source, publicly-hosted project on GitHub, go enable travis-ci and AppVeyor integration right now. We'll wait for you to come back. For a simple example of how to enable it for your C++ CMake-based application, see here: https://github.com/ChaiScript/ChaiScript/blob/master/.travis.yml Also consider going an enabling [Coverity Scan](https://scan.coverity.com)
## Compilers ## Compilers
@@ -96,6 +97,10 @@ Consider using the *treat warnings as errors* setting. `/Wx` with MSVC, `-Werror
The best bet is the static analyzer that you can run as part of your automated build system. Cppcheck and clang meet that requirement for free options. The best bet is the static analyzer that you can run as part of your automated build system. Cppcheck and clang meet that requirement for free options.
### Coverity Scan
Coverity has a free (for open source) static analysis toolkit that can work on every commit in integration with [Travis CI](http://travis-ci.org) and [AppVeyor](http://www.appveyor.com/).
### Cppcheck ### Cppcheck
Cppcheck is free and open source. It strives for 0 false positives and does a good job at it. Therefore all warnings should be enabled: `-enable=all` Cppcheck is free and open source. It strives for 0 false positives and does a good job at it. Therefore all warnings should be enabled: `-enable=all`