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 eusgazebo package again #93

Merged
merged 5 commits into from
Jul 9, 2014

Conversation

mmurooka
Copy link
Member

@mmurooka mmurooka commented Jul 4, 2014

This is new PR same as old PR, #41.

@k-okada
Copy link
Member

k-okada commented Jul 4, 2014

  Could not find a configuration file for package atlas_description.
  Set atlas_description_DIR to the directory containing a CMake configuration
  file for atlas_description.  The file will have one of the following names:
    atlas_descriptionConfig.cmake
    atlas_description-config.cmake
Call Stack (most recent call first):
  rtmros_gazebo/hrpsys_gazebo_atlas/catkin.cmake:5 (find_package)
  rtmros_gazebo/hrpsys_gazebo_atlas/CMakeLists.txt:2 (include)

you need to add more package at https://github.com/start-jsk/rtmros_gazebo/blob/master/.travis.yml#L31 ( I thought drcsim-hydro installs atlas_description ...)

@mmurooka
Copy link
Member Author

mmurooka commented Jul 4, 2014

( I thought drcsim-hydro installs atlas_description ...)

Yes, I thought so too.

I just added eusgazebo package, which is not dependent on atlas_description.
Only hrpsys_gazebo_atlas package depends on atlas_description and I don't add any change to this package, so I don't know why the error about atlas_description is occurring here.
I need to investigate the problem more.
origin/master pass travis ( #94 ).

@k-okada
Copy link
Member

k-okada commented Jul 5, 2014

yes, this is strange, I restarted https://travis-ci.org/start-jsk/rtmros_gazebo/builds/29022623 to see what happens

@mmurooka
Copy link
Member Author

mmurooka commented Jul 5, 2014

Now, I know what the problem is, but I don't know how to solve, so please help.

I need to add gazebo_msgs as <run_depend> and <build_depend>,
because roseus message of gazebo_msgs is not generated and get the following error if I don't add.

eustf roseus_c_util [ERROR] [1404549652.990504301]: Could not find roseus messages under (/home/travis/.ros/roseus/hydro /home/travis/ros/ws_rtmros_gazebo/devel/share/roseus/ros /opt/ros/hydro/share/roseus/ros)
try rosrun roseus generate-all-msg-srv.sh gazebo_msgs

I can not add gazebo_msgs as <run_depend> and <build_depend>,
because ros-hydro-gazebo-msg is installed and drcsim-hydro is removed by rosdep, and get the following error.
( Please see #94 (comment) )

  Could not find a configuration file for package atlas_description.
  Set atlas_description_DIR to the directory containing a CMake configuration
  file for atlas_description.  The file will have one of the following names:
    atlas_descriptionConfig.cmake
    atlas_description-config.cmake

So, I'd like to know the way that ros-hydro-gazebo-msg is not installed by rosdep but roseus message for gazebo_msgs are generated.
ros-hydro-gazebo3-msg is already installed when installing drcsim-hydro, so no more rosdep is necessary.

One idea is that add rosrun roseus generate-all-msg-srv.sh gazebo_msgs to travis
and remove <run_depend> and <build_depend> from package.xml.
But this seems not good solution.

@k-okada
Copy link
Member

k-okada commented Jul 5, 2014

how about add some file under /etc/apt/preferences.d/ and set hydro-gazebo-msg to be pinned, add something like following code around https://github.com/start-jsk/rtmros_gazebo/blob/master/.travis.yml#L27

sudo bash -c 'cat << EOF > /etc/apt/preferencesd.d/conflict-with-gazebo3
Package: ros-hydro-gazebo-msgs
Pin: version 0.0
Pin-Priority: -1
EOF'

@mmurooka
Copy link
Member Author

mmurooka commented Jul 5, 2014

travis regards the situation not finding apt package as error.
mmurooka@74532f9/10ba57a8cc346ab5aad3a8013732e62d2bbeb116
https://travis-ci.org/start-jsk/rtmros_gazebo/builds/29196417

executing command [sudo apt-get install -y ros-hydro-gazebo-msgs]
Reading package lists... Done
Building dependency tree       
Reading state information... Done
Package ros-hydro-gazebo-msgs is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'ros-hydro-gazebo-msgs' has no installation candidate

...


ERROR: the following rosdeps failed to install
  apt: command [sudo apt-get install -y ros-hydro-gazebo-msgs] failed
  apt: Failed to detect successful installation of [ros-hydro-gazebo-msgs]
+++error

@mmurooka
Copy link
Member Author

mmurooka commented Jul 5, 2014

I removed <run_depend> and <build_depend> from package.xml,
and added execute_process(COMMAND rosrun roseus generate-all-msg-srv.sh gazebo_msgs) to catkin.cmake,
It worked both for use_deb=TRUE/FALSE.

mmurooka@ba172cb
https://travis-ci.org/start-jsk/rtmros_gazebo/builds/29190727

Not the best but one solution.

@mmurooka
Copy link
Member Author

mmurooka commented Jul 6, 2014

When apt package is not found, apt-get regards it as error, and aptitude does not.
rosdep uses apt-get, so we can't avoid error by this strategy.

murooka@murooka-ThinkPad-T430:~$ LANG=C sudo apt-get install ros-hydro-gazebo-msgs
Reading package lists... Done
Building dependency tree       
Reading state information... Done
Package ros-hydro-gazebo-msgs is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'ros-hydro-gazebo-msgs' has no installation candidate
murooka@murooka-ThinkPad-T430:~$ echo $?
100

murooka@murooka-ThinkPad-T430:~$ LANG=C sudo aptitude install ros-hydro-gazebo-msgs
No candidate version found for ros-hydro-gazebo-msgs
No candidate version found for ros-hydro-gazebo-msgs
No packages will be installed, upgraded, or removed.
0 packages upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
Need to get 0 B of archives. After unpacking 0 B will be used.
murooka@murooka-ThinkPad-T430:~$ echo $?
0

@k-okada
Copy link
Member

k-okada commented Jul 8, 2014

did you check with rosdep install we run rosdep install with -r (Continue installing despite errors.) in .travis.yml

@mmurooka
Copy link
Member Author

mmurooka commented Jul 9, 2014

After jsk-ros-pkg/jsk_travis#3 is merged and reflected to this repository,
I'll revert the last commit and travis should pass.

@mmurooka
Copy link
Member Author

mmurooka commented Jul 9, 2014

Passed!!!

k-okada added a commit that referenced this pull request Jul 9, 2014
@k-okada k-okada merged commit 8d6fceb into start-jsk:master Jul 9, 2014
@mmurooka mmurooka deleted the add-eusgazebo-package branch July 9, 2014 11:04
robograffitti pushed a commit to robograffitti/rtmros_gazebo that referenced this pull request Jun 24, 2015
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