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

Redundent interface build #1046

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

Conversation

yswhynot
Copy link

I was running into compilation error when doing catkin build with Ubuntu 14.04 and cmake 3.5.1.
The error shows

CMake Error at $CATKIN_WS/src/rtmros_common/rtmbuild/cmake/rtmbuild.cmake:265 (add_custom_target):
  add_custom_target cannot create target
  "RTMBUILD_openrtm_ros_bridge_MyService_genbridge" because another target
  with the same name already exists.  The existing target is a custom target
  created in source directory
  "$CATKIN_WS/src/rtmros_common/openrtm_ros_bridge".

I figured out it should be that there are two services from the same interface called MyService thus the rtmbuild tries to build it twice. Then it fails in the second time.
I think this is a fix for this problem.

@k-okada
Copy link
Member

k-okada commented Jun 26, 2018

@yswhynot Hi, I think the issue is similar to jsk-ros-pkg/jsk_common#1586, but I think this hould not happens if you uses catkin build. So, can you provide full output of your 'catkin build'

@yswhynot
Copy link
Author

yswhynot commented Jul 1, 2018

Hi, I've provided with the full log here: #1047
I think it's different jsk-ros-pkg/jsk_common#1586 since it's from the same package. I guess it might be a better solution if they build in the order that the parent interface is built first.

k-okada added a commit to k-okada/rtmros_common that referenced this pull request Jul 10, 2018
k-okada added a commit to k-okada/rtmros_common that referenced this pull request Jul 10, 2018
 as of 2018, we do not find the solution to fix this without breaking API (start-jsk#1046)...
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