documentation: update prerequisites #7256
Conversation
The following commit updates the `prerequisites` for Linux to include `sudo apt-get install build-essential`. The change was first described in the following comment #2174 (comment) but was never included. It was also useful in resolving the issue #7255. Signed-off-by: vince-fugnitto <[email protected]>
There was a problem hiding this comment.
@vince-fugnitto We add the build-essential in the documentation, but what about the apps, it is set for some docker images but not all, Do you know why?
If we have it for theia-full-docker, why it is not set for theia-docker or theia-go-docker ?
@lmcbout they might be using different base images which already handle For example, |
There was a problem hiding this comment.
Is it blocking this pull-request?
No it is not a blocking issue, just curious why
Looks good
Thanks @vince-fugnitto
What it does
The following pull-request updates the prerequisites for Linux to include build-essential.
The additional prerequisite was useful in the resolution of the following two issues (but the document was never updated):
How to test
The command was useful to resolve two issues regarding
nsfw.Review checklist
Reminder for reviewers
Signed-off-by: vince-fugnitto [email protected]