-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Docs: Explain the release build. And also how-to install Cppcheck #8044
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,20 +35,32 @@ GUIも利用する場合、Qtライブラリが必要です。 | |
| cmakeでCppcheckをコンパイルする例 | ||
|
|
||
| ```shell | ||
| mkdir build | ||
| cd build | ||
| cmake .. | ||
| cmake --build . | ||
| cmake -S . -B build | ||
| cmake --build build | ||
| ``` | ||
|
|
||
| デフォルトのビルドタイプは `Debug` です。`Release` ビルドが必要な場合は、ビルドフラグ (上記の最初の `cmake` コマンドで) を使用できます。 | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do you know japanese well?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Well, I'm wondering why you have a japense readme if you don't speak it 🤣? The Japanese readme is very outdated actually.
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't know any japanese. It has been updated now and then by a japanese guy. I don't want to have some auto translations that we don't know if they are proper or not. So the choice should be either to:
@fu7mu4 are you interested to review/update the file?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove what.. I think the cmake commands were outdated (like how to build it) so I would like to keep that to be honest.. but we can also remove the whole Japense readme file. |
||
| `-DCMAKE_BUILD_TYPE=Release` | ||
|
|
||
| リリース ビルドの場合は以下を使用することをお勧めします。 | ||
| `-DUSE_MATCHCOMPILER=ON` | ||
|
|
||
| C++標準を指定する必要がある場合次のオプションを指定します。 | ||
| -DCMAKE_CXX_STANDARD=11 | ||
| `-DCMAKE_CXX_STANDARD=11` | ||
|
|
||
| CppcheckのGUIが必要な場合次のフラグを指定します。 | ||
| -DBUILD_GUI=ON | ||
| `-DBUILD_GUI=ON` | ||
|
|
||
| pcreが必要になりますが、正規表現のルールサポートが必要な場合次のフラグを指定します。 | ||
| -DHAVE_RULES=ON | ||
| `-DHAVE_RULES=ON` | ||
|
|
||
| #### インストール | ||
|
|
||
| バイナリをコンパイルしたら、以下の cmake コマンドを実行して Cppcheck をインストールできます。 | ||
|
|
||
| ```shell | ||
| sudo cmake --install build | ||
| ``` | ||
|
|
||
| ### Visual Studio | ||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can use this change to explain what rules are and if one should enable it or not.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for information my idea is to get rid of pcre and use std::regex instead. the pcre library we use is end of life. there is work in progress..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep in mind that std::regex is very slow.
Do I need Rules support for the full functionality of cppcheck or only when I want to write custom rules?