Closed wkentaro closed 7 years ago
Thanks for the update this looks good to me. @vrabaud can you take a quick look?
LGTM, please merge.
@tfoote Could u please merge this?
@tfoote Could this be released? Or are there any rules on period for releasing?
though it would be nice to have it released, it is far from being critical (it's a rare use case that you seem to be the only one to need :) ) so I guess we'll have to wait a bit.
I see.
2016年9月30日金曜日、Vincent Rabaudnotifications@github.comさんは書きました:
though it would be nice to have it released, it is far from being critical (it's a rare use case that you seem to be the only one to need :) ) so I guess we'll have to wait a bit.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ros/common_msgs/pull/91#issuecomment-250519983, or mute the thread https://github.com/notifications/unsubscribe-auth/AEHFk9an-r062cwcdXlsKASjyxtEXJDyks5qu-fcgaJpZM4J_nZK .
和田 健太郎 / Kentaro Wada http://wkentaro.com
We try to coordinate low level package releases due to the high cost of rebuilding as well as triggering downloads for the whole community. If we do it in batches we can share that cost across multiple packages.
This is for https://github.com/ros-perception/vision_opencv/pull/150, and suggested by @vrabaud at https://github.com/ros-perception/vision_opencv/pull/150#issuecomment-246205371 .