Skip to content

Install rosdep with apt#6542

Open
ahcorde wants to merge 1 commit into
rollingfrom
ahcorde/rolling/rosdep_apt
Open

Install rosdep with apt#6542
ahcorde wants to merge 1 commit into
rollingfrom
ahcorde/rolling/rosdep_apt

Conversation

@ahcorde

@ahcorde ahcorde commented May 4, 2026

Copy link
Copy Markdown
Collaborator

@ahcorde
ahcorde requested a review from sloretz May 4, 2026 16:28
@ahcorde ahcorde self-assigned this May 4, 2026
@ahcorde ahcorde mentioned this pull request May 4, 2026
Closed
@github-actions

github-actions Bot commented May 4, 2026 •
edited
Loading

Copy link
Copy Markdown

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/25331153316/artifacts/6788684398.

To view the resulting site:

  1. Click on the above link to download the artifacts archive
  2. Extract it
  3. Open html-artifacts-6542/index.html in your favorite browser

christophebedard
christophebedard approved these changes May 4, 2026
View reviewed changes

@christophebedard christophebedard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. We install it from apt in the normal Ubuntu installation tutorial, although now it's installed through sudo apt install ros-dev-tools.

@ahcorde @christophebedard
Install rosdep with atp
d875005 
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@christophebedard
christophebedard force-pushed the ahcorde/rolling/rosdep_apt branch from 248f310 to d875005 Compare May 4, 2026 16:44
christophebedard
christophebedard requested changes May 4, 2026
View reviewed changes

@christophebedard christophebedard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, wait, the pip install command is intentional, we should keep it. See the note above it.

@christophebedard christophebedard added the backport-all backport at reviewers discretion; from rolling to all versions label May 4, 2026
fujitatomoya
fujitatomoya reviewed May 11, 2026
View reviewed changes
Comment thread source/Tutorials/Intermediate/Rosdep.rst
.. code-block:: console

$ pip install rosdep
$ sudo apt-get install python3-rosdep

@fujitatomoya fujitatomoya May 11, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@christophebedard @ahcorde what we should do here is to add --break-system-package instead?

@christophebedard christophebedard May 11, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that would probably be useful, yeah

@dlanov

dlanov commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Current rolling already uses sudo apt install python3-rosdep for Ubuntu while retaining the intentional pip install rosdep command for the PyPI case, so this PR appears to have been superseded by later documentation changes.

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

Reviewers

@christophebedard christophebedard christophebedard requested changes
@fujitatomoya fujitatomoya fujitatomoya left review comments
@sloretz sloretz Awaiting requested review from sloretz

Requested changes must be addressed to merge this pull request.

Assignees

@ahcorde ahcorde

Labels

backport-all backport at reviewers discretion; from rolling to all versions

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ahcorde @dlanov @christophebedard @fujitatomoya