Attention is currently required from: Martin L Roth, Paul Menzel.
Juan José García-Castro Crespo has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80722?usp=email )
Change subject: Documentation/tutorial/part1.md: Install libssl-dev and pkg-config on Debian ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/80722/comment/75a3e014_931be479 : PS1, Line 7: Update Debian packages :
You could be more specific: […]
Done
https://review.coreboot.org/c/coreboot/+/80722/comment/ec141da5_10669f7b : PS1, Line 10: Added
We normally use present tense in commit messages: Add …
Done
https://review.coreboot.org/c/coreboot/+/80722/comment/673c37e2_76143281 : PS1, Line 12: Also converted multiline package list to one-line code markup.
As written, this should be a separate change-set/commit.
80749: Documentation/tutorial/part1.md: Reformat Redhat package description | https://review.coreboot.org/c/coreboot/+/80749
File Documentation/tutorial/part1.md:
https://review.coreboot.org/c/coreboot/+/80722/comment/3700a9fc_811d0fbc : PS1, Line 39: `sudo dnf install git make gcc-gnat flex bison xz bzip2 gcc g++ : ncurses-devel wget zlib-devel patch`
This seems unrelated to the change at hand. Please split this into a separate patch.
Moved to a separate patch.
80749: Documentation/tutorial/part1.md: Reformat Redhat package description | https://review.coreboot.org/c/coreboot/+/80749