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

Call rosdep update on ws-create #69

Closed
wants to merge 1 commit into from

Conversation

reinzor
Copy link
Collaborator

@reinzor reinzor commented Sep 25, 2018

Fixes #59

@reinzor
Copy link
Collaborator Author

reinzor commented Oct 2, 2018

ping

1 similar comment
@reinzor
Copy link
Collaborator Author

reinzor commented Oct 12, 2018

ping

@Rayman
Copy link
Owner

Rayman commented Oct 15, 2018

If you create a new workspace, the environment variables of them will not have loaded. Does this rosdep update then work?

@reinzor
Copy link
Collaborator Author

reinzor commented Oct 17, 2018 via email

@Rayman
Copy link
Owner

Rayman commented Oct 18, 2018

I tried this in a fresh docker install, but I think that the rosdep update will not be called with the ROS_HOME variable of the new workspace, so it will use just the global ~/.ros/rosdep folder`.

The easiest options is to source the new workspace again and then rosdep update, but how can we automate that?

See below a log file of the result.

root@748dff50da26:~/ros# ros-get ws-create distro.yaml /opt/ros/kinetic/
...  (lots of lines)                                                                                
[build] Runtime: 1.5 seconds total.                                                                               
[INFO] root: symlink '/root/ros' => '/root/.config/ros-get/workspaces/ros'
[INFO] ros_get.workspace: saved the workspace as 'ros'
[INFO] ros_get.workspace: created first workspace, let's make it the default
[INFO] root: symlink 'workspaces/ros' => '/root/.config/ros-get/workspace'
reading in sources list data from /etc/ros/rosdep/sources.list.d
Warning: running 'rosdep update' as root is not recommended.
  You should run 'sudo rosdep fix-permissions' and invoke 'rosdep update' again without sudo.
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/base.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/python.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/ruby.yaml
Hit file:///etc/ros/rosdep/base.yaml
Query rosdistro index https://raw.githubusercontent.com/ros/rosdistro/master/index.yaml
Add distro "groovy"
Add distro "hydro"
Add distro "indigo"
Add distro "jade"
Add distro "kinetic"
Add distro "lunar"
Add distro "melodic"
updated cache in /root/.ros/rosdep/sources.cache

Now let's source the new workspace and try again:

root@748dff50da26:~/ros# source ~/.bashrc 
root@748dff50da26:~/ros# rosdep update
reading in sources list data from /etc/ros/rosdep/sources.list.d
Warning: running 'rosdep update' as root is not recommended.
  You should run 'sudo rosdep fix-permissions' and invoke 'rosdep update' again without sudo.
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/base.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/python.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/ruby.yaml
Hit file:///etc/ros/rosdep/base.yaml
Query rosdistro index file:///root/.ruvu/rosdistro/project-exr1.yaml
Add distro "kinetic"
updated cache in /root/ros/home/rosdep/sources.cache

You can see the last line updated cache in /root/ros/home/rosdep/sources.cache that is different.

@reinzor reinzor closed this Jan 29, 2019
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