search

strands-project / scitos_drivers

Scitos G5 drivers that interface ROS to MIRA
3 stars 19 forks source link

Update indigo branch to use latest MIRA version. #91

Closed cburbridge closed 9 years ago

marc-hanheide commented 9 years ago

no surprise... this now fails for hydro :-(

marc-hanheide commented 9 years ago

also... in the current settings all pull requests are only checked for hydro...

cburbridge commented 9 years ago

ok, but if i merge this it wont break the hydro version even though jenkins is upset?

marc-hanheide commented 9 years ago

no... this is just the PR checking being a little messy... It checks all PR's against hydro... It should filter only PRs that are for the respective branch, but it currently doesn't. Feel free to ignore.

marc-hanheide commented 9 years ago

Oh... just spotted this one:

/tmp/test_repositories/src_repository/scitos_drivers/scitos_mira/src/ScitosModule.cpp:23:1140: warning: too many arguments for format [-Wformat-extra-args]
    ROS_WARN("Mira RPC error caught when setting parameter: ", e.what() );
cburbridge commented 9 years ago

Yes, will update this in a separate PR.