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

Release 315.1.8 #24

Closed
130s opened this issue Mar 3, 2014 · 9 comments
Closed

Release 315.1.8 #24

130s opened this issue Mar 3, 2014 · 9 comments
Assignees

Comments

@130s
Copy link
Contributor

130s commented Mar 3, 2014

There's an issue in hironx_ros_bridge that is kind of urgent and can get solved by this commit in hrpsys-base. I'm just waiting for "GO".

@k-okada
Copy link
Member

k-okada commented Mar 3, 2014

I'm still unclear if failure of following packages caused by hrpsys,, openhrp3 and openrtm, so I might want fix hrpsys/catkin.cmake again, but basiccaly it is ok. so please release as 315.1.8
https://travis-ci.org/start-jsk/rtmros_tutorials/builds/19807355
https://travis-ci.org/start-jsk/rtmros_gazebo/builds/19903816

@130s
Copy link
Contributor Author

130s commented Mar 3, 2014

I see.
This is currently blocked by #25 (comment) (we should refrain from releasing packages that don't even build locally).

Or I can try the following although I'm not sure if this works: roll back locally to around the point where the tag 315.1.8 was pushed on the upstream repo then release into release repo.

@k-okada
Copy link
Member

k-okada commented Mar 3, 2014

you don't have to roll back, let's wait for latest openrtm_aist is out.

@130s
Copy link
Contributor Author

130s commented Mar 5, 2014

openrtm_aist seems to have been build on shadow-fixed. Now blocked by #27

@130s 130s closed this as completed Mar 5, 2014
@130s
Copy link
Contributor Author

130s commented Mar 5, 2014

@130s
Copy link
Contributor Author

130s commented Mar 6, 2014

So, we need to re-release 315.1.8 to reflect #29.

@k-okada Does it require that we need to re-release openrtm_aist_core with start-jsk/openrtm_aist_core#21?

@130s 130s reopened this Mar 6, 2014
@k-okada
Copy link
Member

k-okada commented Mar 6, 2014

It is not require nor safe, but better to re-release with newer version. not urgent.

@130s
Copy link
Contributor Author

130s commented Mar 6, 2014

Ok then I'll re-release 315.1.8, for Hydro now as it passes pre-release as I described at the bottom of this.

@130s
Copy link
Contributor Author

130s commented Mar 6, 2014

Done for now.

@130s 130s closed this as completed Mar 6, 2014
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

No branches or pull requests

2 participants