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

Blacklist image tools py #232

Merged
merged 2 commits into from
May 23, 2019
Merged

Blacklist image tools py #232

merged 2 commits into from
May 23, 2019

Conversation

clalancette
Copy link
Contributor

@clalancette clalancette commented Jan 2, 2019

Blacklist image_tools_py on Xenial, since there is no OpenCV python bindings for Python3 there. Also install python3-opencv on non-Xenial Linux targets.

connects to ros2/demos#85

@clalancette clalancette added the in progress Actively being worked on (Kanban column) label Jan 2, 2019
@clalancette clalancette added in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Jan 2, 2019
@cottsay
Copy link
Member

cottsay commented Jan 28, 2019

@clalancette - this appears to have been reviewed. Is it ready to be merged?

@clalancette
Copy link
Contributor Author

clalancette commented Jan 29, 2019

@cottsay I was waiting to merge this until ros2/demos#107 is approved/merged, since it is not necessary until then. But I'm fine to put it in now if you think we should.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette
Copy link
Contributor Author

I'm going to merge this one in, so we'll be ready for ros2/demos#107 when it is approved and merged.

@clalancette clalancette merged commit eafb314 into master May 23, 2019
@clalancette clalancette deleted the blacklist-image-tools-py branch May 23, 2019 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in review Waiting for review (Kanban column)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants