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

[hrpsys_ros_bridge/test] remove reverse for samplerobot force-sensors #1108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Naoki-Hiraoka
Copy link
Contributor

jsk-ros-pkg/jsk_model_tools#236 のrelease後にmergeしてください.

euscolladaのバグに対応するためのテンポラリの修正として#1080 (comment) #1081 でテストに施された変更を、該当のバグがjsk-ros-pkg/jsk_model_tools#236 で修正されるため、戻します。

@pazeshun
Copy link
Collaborator

こちらですが、jsk-ros-pkg/jsk_model_tools#236 は0.4.4に含まれ、melodicではrelease済みです。
http://repositories.ros.org/status_page/melodic_default.html?q=euscollada
ただ、EOLのkineticに対してはもちろんreleaseされていないので、kinetic以下とmelodic以上でテストコードを分けることになるような気がしています。
http://repositories.ros.org/status_page/kinetic_default.html?q=euscollada
もしくは、kinetic以下にeuscollada 0.4.4をソースで入れるかですが、ビルドできるかなどは不明です。
ただし、現在rtmros_commonのmasterでは、melodicにおいてeuslispのテストを行っていないため、この問題は現状回避されています。
個人的には、euslispのテストとしては、現状のkineticまでのテストで当面は大丈夫な気がしていて、melodicのテストを入れる話は、#1107 をのんびり直しながらやっていこうと思っています。

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.

2 participants