search

project-hatohol / hatohol

A unified manager of monitoring software
http://www.hatohol.org/
Other
88 stars 29 forks source link

Add -Wsuggest-override #2512

Closed otakuto closed 7 years ago

otakuto commented 7 years ago

AX_CHECK_COMPILE_FLAGを使っているのでautoconf-archiveパッケージが必要になりますが問題ないでしょうか?

ashie commented 7 years ago

問題ないと思いますがそうやる場合は

が必要だと思います。

最近GitHub上で表示されなくなっているようですが、CentOSに関してはWerckerでCIが走ってます（ https://app.wercker.com/ ）。wercker.ymlも更新しないと失敗しそう。

依存性が増えるのが嫌であれば、ax_check_compile_flag.m4だけm4ディレクトリ以下に取り込んでしまう手もあります。

cosmo0920 commented 7 years ago

CentOS 6上ではビルドに失敗していますね。 ref: https://app.wercker.com/ashie/hatohol/runs/build/58a160b1f499ec0100b88241?step=58a160c2daa71200017e204c

masa0612 commented 7 years ago

今更感があるのですが，HatoholサーバーのRPMは for Cent7しか作成していないし， werkerのビルドテストは削除したほうが良いかもしれませんね。

cosmo0920 commented 7 years ago

WerckerのCIで用いるベースイメージをCentOS 7に変えるというのはどうでしょう？

masa0612 commented 7 years ago

私の書き方が悪かったです。 まさに

WerckerのCIで用いるベースイメージをCentOS 7に変えるというのはどうでしょう？

こういう意図を伝えようとしていましたが，改めて自分のコメント見ると全然そう見えない。。。

そうしましょうそうしましょう

otakuto commented 7 years ago

とりあえずこのPRはax_check_compile_flag.m4だけ取り込む方針で行きます.

masa0612 commented 7 years ago

OK. 話し逸らしてごめんなさい

cosmo0920 commented 7 years ago

@otakuto このPRでは -Wsuggest-override を使えるときは使うと言う変更だけにしませんか？ wercker.yml の変更は別PRにした方が良さそうな気がします。

otakuto commented 7 years ago

了解っす