Skip to content
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

[WIP] cmake 対応の実験 #46

Closed

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jun 3, 2018

[WIP] cmake 対応の実験をしました。
cmake を使ってビルドができるようになりました。

cmake をインストールして build-by-cmake.bat を実行したらビルドできます。

@m-tmatma m-tmatma mentioned this pull request Jun 3, 2018
@kobake
Copy link
Member

kobake commented Jun 3, 2018

これがうまく使えるようであれば Makefile は不要になる、という認識で合ってますか?

@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 3, 2018

これがうまく使えるようであれば Makefile は不要になる、という認識で合ってますか?

はい。

CMake は MinGW に対応しているみたいなので。
http://d.hatena.ne.jp/simpg/20120714/1342220586

(preBuild.bat は組み込んでていない)
- add_custom_command で PRE_BUILD を設定し、GitHash 更新用のバッチファイルを実行
- cmake の最低バージョンを 3.0.2 に変更 (add_custom_command を使うため)
- mkdir に失敗したときバッチフアイルをエラー終了させる
@m-tmatma m-tmatma force-pushed the feature/WIP-cmake-experiement branch from 62dad55 to 0fcd899 Compare June 3, 2018 12:31
@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 3, 2018

現状の問題点

以下を実行しても、Funccode_enum.h と Funccode_define.h を消してくれない。

cmake --build . --target clean

@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 3, 2018

これがうまく使えるようであれば Makefile は不要になる、という認識で合ってますか?

CMake は MinGW に対応しているみたいなので。
http://d.hatena.ne.jp/simpg/20120714/1342220586

ただし、CMake の設定ファイルでコンパイルオプションとかを VC 用に設定しているので
コンパイルオプション(precompiled header 関連など)が、MinGW と互換性がないと思うので
MinGW 対応の作業は必要だと思います。

@m-tmatma
Copy link
Member Author

この PR はだいぶん古くなったので #378 で更新版をあげました。

@m-tmatma m-tmatma added the Won't Fix 対応しない【ChangeLog除外】 label Sep 30, 2018
@m-tmatma
Copy link
Member Author

#378 があるのでクローズ

@m-tmatma m-tmatma closed this Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Won't Fix 対応しない【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants