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

Install tags missed #79

Open
ipa-nhg opened this issue Feb 5, 2016 · 2 comments
Open

Install tags missed #79

ipa-nhg opened this issue Feb 5, 2016 · 2 comments
Assignees

Comments

@ipa-nhg
Copy link
Member

ipa-nhg commented Feb 5, 2016

Please review the packages , I saw that you have defined a lot of folders inside each packages that are not listed on the CMakeLists.

Take a look at http://wiki.ros.org/catkin/CMakeLists.txt the section 10. Optional Step: Specifying Installable Targets . It is optional but if you dont define the folders like data in squirrel_object_perception , this folder will be ignored when you release your package.
A simple example can be found on squirrel_robotino for meshes and urdf folders https://github.com/squirrel-project/squirrel_robotino/blob/indigo_dev/robotino_description/CMakeLists.txt#L12

@bajo
Copy link
Contributor

bajo commented Feb 5, 2016

@ipa-nhg Thank you, I did not know that.
So I need to add folders like squirrel_object_perception/data/octomaps/ and squirrel_perception/rosdep as an Installable Target ?

@ipa-nhg
Copy link
Member Author

ipa-nhg commented Feb 5, 2016

yes, with squirrel_object_perception/data is enough.

You can see the result with the command catkin_make install , the install folder that will be generated with this command, is the version that you will generate when you release your package.

Actually, I usually source the catkin_ws/install/setup.bash to use the release version and be sure that nothing is missed.

PS: This issue is not critical

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