Visual Studio 2022でのCMakeの警告に対応 #1848

dep5 · 2022-05-31T12:40:29Z

PR の目的

2>CMake Warning (dev) in CMakeLists.txt:
2>  No project() command is present.  The top-level CMakeLists.txt file must
2>  contain a literal, direct call to the project() command.  Add a line of
2>  code such as
2>
2>    project(ProjectName)
2>
2>  near the top of the file, but after cmake_minimum_required().
2>
2>  CMake is pretending there is a "project(Project)" command on the first
2>  line.
2>This warning is for project developers.  Use -Wno-dev to suppress it.

ビルド・リビルド時にこのような警告が出るので対応します。

2>  Manually-specified variables were not used by the project:
2>
2>    CMAKE_C_COMPILER

新たに別の警告が出るので
これにも対応します。

カテゴリ

不具合修正
ビルド関連
- ビルド手順
- ローカルビルド

PR の背景

Visual Studio 2017では警告は出ませんが、Visual Studio 2022だと出るようになりました。
2019でも出ていたような気がします…

PR のメリット

PR のデメリット (トレードオフとかあれば)

仕様・動作説明

https://cmake.org/cmake/help/latest/command/project.html
警告の内容に従って、~~project()~~ -Wno-devを使っています。

PR の影響範囲

Visual Studio 2017でも問題ないようです。

ビルドしてテストしてみました。

テスト内容

テスト1

手順

参考資料

dep5 · 2022-05-31T12:43:03Z

この警告の有無でVSがどのバージョンか見分けていましたが、
対応することにしました。

サクラエディタ全体のCMakeではないと思うので、関数名を書いておきましたが
PROJECT_NAMEはこれでいいのでしょうか？

"-DCMAKE_C_COMPILER=%CL_COMPILER%"
は残しておいたほうがいいのでしょうか？

AppVeyorBot · 2022-05-31T13:24:25Z

❌ Build sakura 1.0.4138 failed (commit 1a043c6d51 by @dep5)

sanomari · 2022-06-01T23:41:36Z

対処方法が誤っているように見えました。

2>CMake Warning (dev) in CMakeLists.txt:
2> No project() command is present. The top-level CMakeLists.txt file must
2> contain a literal, direct call to the project() command. Add a line of
2> code such as
2>
2> project(ProjectName)
2>
2> near the top of the file, but after cmake_minimum_required().
2>
2> CMake is pretending there is a "project(Project)" command on the first
2> line.
2>This warning is for project developers. Use -Wno-dev to suppress it.

正しい対処は最後の一文「Use -Wno-dev to suppress it.」です。

ここのCMakeLists.txtは成果物を作らないので、projectを呼ばなくてもよいです。
「呼ばなくても良い」を分かったうえでprojectしたい場合、格納フォルダ名を指定してprojectします。
このPRでは違う名前を付けようとしてるので、そこも違う気がします。

バッチの修正は、明らかに使ってないので問題ないです。

dep5 · 2022-06-09T13:03:08Z

sanomari さん
コメントありがとうございます。
アドバイスに従って書き直してみたところ、
"-DCMAKE_C_COMPILER=%CL_COMPILER%"があっても警告が出なくなったので
元に戻しておきました。
必要な警告まで消えてしまうのではないかと心配ですが、
もともと警告を消したいというPRでしたね…