[setup] Rewrite Ubuntu install_prereqs from the ground up #22055
Conversation
This file contains 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
jwnimmer-tri
force-pushed
the
setup-txt-accuracy
branch
from
6b41139
to
a11e740
Compare
jwnimmer-tri
changed the title
jwnimmer-tri
force-pushed
the
setup-txt-accuracy
branch
from
a11e740
to
709113b
Compare
jwnimmer-tri
force-pushed
the
setup-txt-accuracy
branch
2 times, most recently
from
bbf37ee
to
ef6931b
Compare
The existing from-source setup scripts are deprecated and will be removed after 2025-06-01: - setup/ubuntu/install_prereqs.sh - setup/ubuntu/source_distribution/install_prereqs_user_environment.sh Instead, users and developers should run setup/install_prereqs (not as root). Usability changes: - Fine-tuned log output, with option for --verbose (e.g., in CI). - Skip unnecessary steps; incremental runs are typically sub-second. Maintainability changes: - Always use files, not heredocs. (The pkg-config etc in a heredoc were documented as unwanted when using Drake's Debian packages, but that was wrong; we always want those deps even in Debian packages.) Website changes: - Major updates to from_source that explain how to install from CMake. A few cleanups macOS come along for the ride: - Remove useless binary_distribution_called_update shenanigans. - Remove vestigial i386 rcfile. - Abandon all hope of supporting 'brew install drake'.
jwnimmer-tri
force-pushed
the
setup-txt-accuracy
branch
from
ef6931b
to
94a95dc
Compare
jwnimmer-tri
commented
Nov 18, 2024
There was a problem hiding this comment.
Reviewed 13 of 15 files at r1, 28 of 30 files at r2, 2 of 3 files at r3, 2 of 3 files at r4.
Reviewable status: 2 unresolved discussions, needs platform reviewer assigned, needs at least two assigned reviewers, commits need curation (https://drake.mit.edu/reviewable.html#curated-commits), missing label for release notes
doc/_pages/from_source.md line 42 at r5 (raw file):
require a working display server. Most personal computers will have this already built in, but some cloud or docker environments may require extra setup steps.
Working
Refresh our advice re: display servers now that we use EGL by default.
doc/_pages/installation.md line 40 at r5 (raw file):
require a working display server. Most personal computers will have this already built in, but some cloud or docker environments may require extra setup steps.
Working
Refresh our advice re: display servers now that we use EGL by default.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This change is