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

fix flip of logic on openrtm major version detection #446

Merged
merged 4 commits into from
Jan 17, 2015

Conversation

yosuke
Copy link
Contributor

@yosuke yosuke commented Jan 12, 2015

「1.1.1はマイナーチェンジなので1.1.0と同様に処理したい」という意図の設定だと思うのですが、逆に機能していたので直しました。

@k-okada
Copy link
Contributor

k-okada commented Jan 12, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1723/

Build Log
last 10 lines

[...truncated 4143 lines...]
=====
$ tar -xvzf /tmp/hrpsys-446-qnx-2015-01-13; cd hrpsys-source-446-2015-01-13/build; #make install
+ grep \*\*\* /tmp/robot-compile-hrpsys-446.log
make[2]: *** [ec/hrpEC/CMakeFiles/hrpEC.dir/hrpEC.o] Error 1
make[1]: *** [ec/hrpEC/CMakeFiles/hrpEC.dir/all] Error 2
make: *** [all] Error 2
+ exit 1
Build step 'Execute shell' marked build as failure
Sending e-mails to: [email protected] [email protected] [email protected] [email protected]

Test FAILed.

@yosuke
Copy link
Contributor Author

yosuke commented Jan 12, 2015

逆に1.1.0の場合がダメになってしまったようです。regexで書き直してみます。

@k-okada
Copy link
Contributor

k-okada commented Jan 13, 2015

QNXでコンパイルが通らなくなっているようです.
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1723/console

@fkanehiro
Copy link
Owner

26798e0 を見ると、むしろ1.1.0と1.1.1で処理を分けたいという意図のようです。

@yosuke
Copy link
Contributor Author

yosuke commented Jan 15, 2015

情報ありがとうございます。OpenRTMのSVNを見るとtrunkの方がECの格納にRTM_expというネームスペースを使っていてRELENG-1-1ブランチのほうがRTMネームスペースのままのようで、これが私が1.1.1を使った時のコンパイルエラーの原因のようです。
問題なければこの部分だけネームスペースを戻したいと思うのですが、こうなった経緯をなにかご存知でしょうか?

@yosuke
Copy link
Contributor Author

yosuke commented Jan 15, 2015

度々すみません。変更全体を見てみるとこのコミットは1.1.1というよりはtrunkバージョン(おそらく1.2なのだと思うのですが、configure.ac上では1.1.1になっている)を意図しているもののようですね。その後、1.1.1が本当に出てしまったので不具合が起こっているようです。
1.1.x系列にはRTObjectStateMachineファイルがないので、そちらの方でcmakeにスイッチさせることを考えてみます。

@k-okada
Copy link
Contributor

k-okada commented Jan 17, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1729/
Test PASSed.

fkanehiro added a commit that referenced this pull request Jan 17, 2015
fix flip of logic on openrtm major version detection
@fkanehiro fkanehiro merged commit 71879d6 into fkanehiro:master Jan 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants