Document apt index refresh for rosdep in minimal ROS images - #828
Closed
yurekami wants to merge 1 commit into
Closed
Document apt index refresh for rosdep in minimal ROS images#828yurekami wants to merge 1 commit into
yurekami wants to merge 1 commit into
Conversation
The Jazzy README currently suggests a clone -> rosdep workflow that is correct on a host machine but can fail in minimal ROS Docker images after their APT lists have been cleared. Document the required apt-get update step and add the first-time contributor NOTICE entry expected by this repository. Constraint: rosdep still resolves curl to libcurl4-openssl-dev, and Ubuntu 24.04 apt does not accept virtual-only libcurl4-dev as an install candidate Rejected: Change ros/rosdistro curl mapping to libcurl4-dev | Noble apt-get -s install libcurl4-dev reports no installation candidate Rejected: Leave the issue to user workaround notes only | the main README currently omits the Docker-specific prerequisite in the build path Confidence: high Scope-risk: narrow Directive: Keep the Docker note adjacent to the generic build steps so future package-manager changes are updated in one place Tested: git diff --check; verified Noble apt simulation for libcurl4-openssl-dev and libcurl4-dev via WSL Ubuntu 24.04.3 Not-tested: upstream GitHub Actions; full micro_ros_setup build Signed-off-by: yurekami <yurekami@users.noreply.github.com>
Contributor
|
Tick the box to add this pull request to the merge queue (same as
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Document the
apt-get updatestep required beforerosdep installwhen buildingmicro_ros_setupin minimal ROS Docker images such asros:jazzy-ros-core-noble.Root cause
micro_ros_setupdepends on the rosdep keycurlrosdistrocorrectly resolves that key on Ubuntu tolibcurl4-openssl-dev/var/lib/apt/lists/*rosdep installwithout first refreshing APT indexes cannot locate the packageOn Ubuntu 24.04 Noble,
libcurl4-devis only a virtual package whilelibcurl4-openssl-devis the concrete installable package, so changing the rosdep mapping would be incorrect.Verification
git diff --checkThe
NOTICEentry follows this repository's first-time contributor requirement.Fixes #782