Skip to content

Adding cv_backports to package.xml#33

Closed
ugocupcic wants to merge 3 commits intoplasmodic:masterfrom
ugocupcic:patch-1
Closed

Adding cv_backports to package.xml#33
ugocupcic wants to merge 3 commits intoplasmodic:masterfrom
ugocupcic:patch-1

Conversation

@ugocupcic
Copy link

Otherwise I get the error:

CMake Error at /opt/ros/indigo/share/catkin/cmake/catkin_package.cmake:189 (message):
  catkin_package() the catkin package 'cv_backports' has been
  find_package()-ed but is not listed as a build dependency in the
  package.xml

Otherwise I get the error:

```
CMake Error at /opt/ros/indigo/share/catkin/cmake/catkin_package.cmake:189 (message):
  catkin_package() the catkin package 'cv_backports' has been
  find_package()-ed but is not listed as a build dependency in the
  package.xml
```
@vrabaud
Copy link
Member

vrabaud commented May 10, 2016

actually, it is not required on master anymore (which is Kinetic and above). So it should be removed from the CMake instead. Thx for updating your PR.
Now, I don't understand why that worked on the build farm ...

@ugocupcic
Copy link
Author

@vrabaud OK sorry! Updated the pr (if I understood correctly)

@vrabaud
Copy link
Member

vrabaud commented May 10, 2016

oh wait, that was supposed to work. Are you on Indigo by any chance ? You should checkout the Indigo branch then.

@ugocupcic
Copy link
Author

Ah! OK, you wouldn't have the equivalent of this rosinstall but for indigo by any chance?

@ugocupcic ugocupcic closed this May 10, 2016
@ugocupcic ugocupcic deleted the patch-1 branch May 10, 2016 08:51
@vrabaud
Copy link
Member

vrabaud commented May 10, 2016

good point, I did not update that rosinstall. Will do tonight.
For now, switching ecto_opencv to the indigo branch is the only thing to do I believe.

@vrabaud
Copy link
Member

vrabaud commented May 14, 2016

@ugocupcic , I fixed the rosinstall files here: http://wg-perception.github.io/object_recognition_core/install.html#rosinstall-file
Thx for reporting !

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

Successfully merging this pull request may close these issues.

2 participants