s-nakaoka / choreonoid

An integrated graphical robotics application framework
http://choreonoid.org
Other
94 stars 58 forks source link

ChoreonoidConfig.cmakeにPYTHON_LIBRARIESの値を埋め込むようにした. #237

Closed y-masutani closed 4 years ago

y-masutani commented 4 years ago

OpenRTMプラグインをソースツリーの外でビルドする場合は,PYTHON_LIBRARIESの値が反映されないので,このようにしてはどうでしょうか?

s-nakaoka commented 4 years ago

修正ありがとうございます。ご指摘・修正いただいた点当方のミスでした。大変助かります。 (ちなみに大変恐縮ですが、コミットメッセージを英語にしただけると助かります。いちおうコミットメッセージは英語で統一するようにしておりますので…。)

y-masutani commented 4 years ago

コミットメッセージを日本語にしてしまい申し訳ありません.こちらのリポジトリで英語に直してみましたが,もうそれは手遅れですね… https://github.com/y-masutani/choreonoid/commit/6bb4e0a95a840377400cead53aa7cae38f3c342d 次からは英語にします.

s-nakaoka commented 4 years ago

ご理解いただきありがとうございます。 日本語になっていても英語環境で更新できなくなるようなことは無いと思うので、そこまで気になさらなくても大丈夫です。私もあまり深く考えずにマージしてしまったのですが、申し訳ありませんでした。 ただ確かGitではコミット後にも過去の変更を修正できるようですので、今回のメッセージについて修正できないか試してみます。

s-nakaoka commented 4 years ago

ただ確かGitではコミット後にも過去の変更を修正できるようですので、今回のメッセージについて修正できないか試してみます。

これについてgit rebase -i と git push -f を用いてできないことはないようですが、共有リポジトリに対してgit push -f を行うのは良くないようですので、修正は見送りことにします。私も今後マージの際には気をつけるようにいたします。