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

add RoomLight msg in jsk_robot_startup #1365

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

k-okada
Copy link
Member

@k-okada k-okada commented Jul 19, 2021

fixed version of #1330 ?

@k-okada
Copy link
Member Author

k-okada commented Jul 19, 2021

not sure why indigo fails...

  _______________________________________________________________________________
  Errors << jsk_nao_startup:cmake /github/home/ros/ws_jsk_robot/logs/jsk_nao_startup/build.cmake.000.log
  -- set EUSDIR to /opt/ros/indigo/share/euslisp/jskeus/eus
  -- set ARCHDIR to Linux64
  CMake Error at /github/home/ros/ws_jsk_robot/devel/share/jsk_robot_startup/cmake/jsk_robot_startupConfig.cmake:141 (message):
    Project 'jsk_nao_startup' tried to find library 'jsk_robot_startup'.  The
    library is neither a target nor built/installed properly.  Did you compile
    project 'jsk_robot_startup'? Did you find_package() it before the
    subdirectory containing its code is included?
  Call Stack (most recent call first):
    /opt/ros/indigo/share/roseus/cmake/roseus.cmake:93 (find_package)
    /opt/ros/indigo/share/roseus/cmake/roseus.cmake:174 (generate_all_roseus_messages)
    /opt/ros/indigo/share/roseus/cmake/roseusConfig.cmake:190 (include)
    /opt/ros/indigo/share/catkin/cmake/catkinConfig.cmake:76 (find_package)
    CMakeLists.txt:7 (find_package)

@knorth55
Copy link
Member

it seems jsk_nao_startup tried to generate euslisp message in jsk_robot_startup, but jsk_robot_startup is not build before jsk_nao_startup.
jsk_robot_startup is set in run_depend in jsk_nao_startup, but should it be build_depend?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants