Skip to content

fixes 'catkin_make install' problem #25

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

teevr
Copy link

@teevr teevr commented Feb 2, 2022

I wanted to install/run ocam with catkin_make install but failed as followings;

teevr@teevr-NUC7i7BNH:~/ws$ catkin_make install
......
teevr@teevr-NUC7i7BNH:~/ws$ . install/setup.sh
teevr@teevr-NUC7i7BNH:~/ws$ roslaunch ocam ocam_ros.launch
RLException: [ocam_ros.launch] is neither a launch file in package [ocam] nor is [ocam] a launch file name
The traceback for the exception was written to the log file
1 teevr@teevr-NUC7i7BNH:~/ws$

This PR makes catkin_make install work properly, i guess.

@zeropk
Copy link
Contributor

zeropk commented Feb 3, 2022

first, check if the ros package is added
$ rospack list | grep ocam

if ros package is added

unnamed

$ source devel/setup.bash
$ roslaunch ocam ocam_ros.launch

if ros package is not added try build again
check the link for oCam ROS Package
https://github.com/withrobot/oCam/tree/master/Software/oCam_ROS_Package

@teevr
Copy link
Author

teevr commented Feb 3, 2022

@zeropk oh, i was able to run with commands catkin_make, source devel/setup.sh and roslaunch ocam ocam_ros.launch.
but failed with catkin_make install, source install/setup.sh and roslaunch ocam ocam_ros.launch (this should work for packaging or something, i think).

@zeropk
Copy link
Contributor

zeropk commented Feb 3, 2022

oCam ROS Package does not include setup for catkin_make install, we recommend using catkin_make.
If you want to use you edit ocam/CMakeLists.txt.

@teevr
Copy link
Author

teevr commented Feb 3, 2022

ok. what i'm saying is this PR can solve the 'install' problem above.
i've tested on ROS noetic.
so i want this PR merged (iff there's no flaw).

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