Activate C++ 11 for choreonoid 1.7

start-jsk / rtmros_choreonoid

using chreonoid for simulator with hrpsys and other ros system

9 stars 43 forks source link

Activate C++ 11 for choreonoid 1.7 #277

Closed epsilonkei closed 6 years ago

epsilonkei commented 6 years ago

Fix CMake for #276

YoheiKakiuchi commented 6 years ago

このPR自体は、hotfixなんだと思うんだけど、一般的に、c++11でないとエラーになるようなヘッダ（だよね悪さしてるのは）のpkg-configや.cmakeは cflagsにフラグを入れるのか、今回のようにユーザーが対応するものなのだろうか？

ishiguroJSK commented 6 years ago

僕はcmake歴浅いので相場が分かりませんが，最初はこれで普通にbugfix完了だと思ってました．確かにpkg-config choreonoid-body-plugin --cflagsで-std=c++11が返ってくるようにしたほうがいいという説もありますね． (現状それはchoreonoid/CMakeLists.txtに書かれているようですが) Choreonoid側にissue立てて訊いてみますか？

YoheiKakiuchi commented 6 years ago

そうですね、僕もどうするのがマナーなのかわからないので、ちょっと聞いてみました。

ヘッダーにc++11が必要な記述が入ると、リンクして使っている側にc++11の設定が必要になる。 c++11の設定がないときにコンパイルできないなら、pkg-configでも対応してもらえるとありがたい。（これは、一般的なマナーかな？よく知らない）

Choreonoid側にissue立てて訊いてみますか？

なので聞いてみましょう。