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

appveyor で MingW ビルドする #309

Closed
m-tmatma opened this issue Jul 26, 2018 · 5 comments
Closed

appveyor で MingW ビルドする #309

m-tmatma opened this issue Jul 26, 2018 · 5 comments
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Milestone

Comments

@m-tmatma
Copy link
Member

appveyor で MingW ビルドする
https://www.appveyor.com/docs/build-environment/#mingw-msys-cygwin

#306 等で対応を進めているのでローカルビルドができるように
なったときに考える。

@berryzplus
Copy link
Contributor

#349(#351) で gcc ビルドできる条件が確認できたら
googletest の appveyor.yml を参考に MinGW でのビルドができないか検討。
実装の手間を考えると一旦は Makefile のまま組み込むのが良さそう。

#46 を発展させて #12 (CMake対応) を完成させる、が簡単にできるなら
先にそちらを済ませて cmake -G "MinGW Makefiles" して
MinGW向け一時フォルダを用意するのがいい気がする。
現状ではビルドできても使えないので、配布は考えなくて良さそう。

@berryzplus
Copy link
Contributor

クローズした #349 の話題はこちらに引き継ぎかな、と思っています。

m_tmatma

MinGW向け特化のCMakeLists.txtを作る →  #46 をベースにすることで意外と簡単にできた。

#378 で最新をベースに cmake 用の対応をあげました。

正確には、ここに引き継ぐのは
MinGW ビルドをどのように appveyor に組み込むか?
という話だと思っています。

MinGW 向けに CMakeLists.txt の完成度を上げておく話は別件かな?

単純に MinGW ビルドを appveyor に組み込む話だけでも、
本来はいろいろ考えることがあると思うんですが、
まずは「ビルドに組み込む」を最優先にしたいです。


ところで、MinGWビルドを appveyor に組み込む目的について話が出ていない気がします。
ぼくが主観的に「これが目的だ」と思っていることをちょっと書いておきます。

  1. Makefileの保守をやめる ←コードベースを身軽にする
  2. MakefileMakeの保守をやめる ←コードベースを身軽にする
  3. C++規格への準拠度を「できるだけ」高める ←コードベースの「良さ」を担保する

けっこう消極的な理由で進めている作業です。
大事か大事じゃないかっていうと結構大事なことだと思っていますが。

@m-tmatma
Copy link
Member Author

m-tmatma commented Nov 8, 2018

@berryzplus さん

これ残件は #591 だけかな?

@berryzplus
Copy link
Contributor

これ残件は #591 だけかな?

いったんそこで終わりでいいと思います。
本当はビルドさえできれば終わっていいと思っていたんですが、せっかくなのでテストを動かすとこまで出来るようにしておきたいです。

@m-tmatma
Copy link
Member Author

m-tmatma commented Nov 8, 2018

#591 をマージしたので閉じます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

2 participants