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.

Sign up for GitHub

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

Jump to bottom

fix iob for downstream packages #597

Merged
merged 8 commits into from
Apr 22, 2015
Merged

fix iob for downstream packages #597

merged 8 commits into from
Apr 22, 2015

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Apr 22, 2015

  • [lib/io/iob{cpp,h},rtc/RobotHardware/{robot.cpp,robot.h,RobotHardwareService_impl.cpp,CMakeList.txt] add ROBOT_IOB_VERSION for backword compatibility

k-okada and others added 8 commits April 22, 2015 14:57
@k-okada
[.travis.yml, .travis.sh] enable hrpsys with 315.1.10
124b1e9 
[.travis.sh] RULE CHANGED adding new function to iob.h is ok

[.travis.sh] do not install test/share/samples/src of old hrpsys, use sample/test/launch of latest hrpsys

[.travis.sh] display test results when failure
@chiwunau @k-okada
[sample/RampleRobot,sample/environment] install pyhton scripts under …
d5c54e5 
…deval
@k-okada
[hrpsys_config.py] #567 is wrong, do not need to decode
436287b
@k-okada
[RobotHardware, iob, BodyRTC] fix to compatible with old API
3b78894
@k-okada
[test-colcheck.test] load modelfiles
55c1d28
@k-okada
[test-robot-hardware.test] add test to check robot-hardware service call
46b27de
@k-okada
[.travis.sh] hot fix for start-jsk/rtmros_hironx#358
56805f5
@k-okada
[lib/io/iob{cpp,h},rtc/RobotHardware/{robot.cpp,robot.h,RobotHardware…
c07ca55 
…Service_impl.cpp,CMakeList.txt] add ROBOT_IOB_VERSION for backword compatibility
@k-okada
Copy link
Contributor Author

k-okada commented Apr 22, 2015

川下の方から悲鳴が上がっているので対応しました．
start-jsk/rtmros_gazebo#183

read_batteryを使いたい場合は明示的に

add_definitions(-DROBOT_IOB_VERSION=2)

とするものです．

@k-okada k-okada mentioned this pull request Apr 22, 2015
Open
@k-okada
Copy link
Contributor Author

k-okada commented Apr 22, 2015

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

@k-okada
Copy link
Contributor Author

k-okada commented Apr 22, 2015

こちらも大丈夫だと思います．差分は最後のコミットのみです．

fkanehiro added a commit that referenced this pull request Apr 22, 2015
@fkanehiro
Merge pull request #597 from k-okada/fix_iob2
3dff336 
fix iob for downstream packages
@fkanehiro fkanehiro merged commit 3dff336 into fkanehiro:master Apr 22, 2015
@k-okada k-okada deleted the fix_iob2 branch May 2, 2015 02:25
@k-okada k-okada mentioned this pull request Dec 17, 2016
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@k-okada @fkanehiro @chiwunau