-
Notifications
You must be signed in to change notification settings - Fork 346
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
[doc] Update doc with cross-rs
and musl
builds
#2621
Conversation
PTAL, thanks! |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2621 +/- ##
=======================================
Coverage 65.88% 65.88%
=======================================
Files 133 133
Lines 16819 16819
=======================================
+ Hits 11081 11082 +1
+ Misses 5738 5737 -1 |
|
||
#### Debian, Ubuntu and related distributions | ||
```console | ||
$ sudo apt-get install \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure that we need all these dependencies.
IIUC we don't need neither of libsystemd-dev
, libelf-dev
, libclang-dev
or libssl-dev
.
Are they needed by any of the tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. I often forget to update the dependencies. A good way to check is to delete them from there and attempt our CI. When all CI are passed, it means they aren't necessary.
https://github.com/containers/youki/blob/656ccc3f69329c2ed10129e046d0474e6c3dee13/justfile#L153-L160
Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good
This PR updates the mdbook documentation to take into account the changes introduced in #2536