Attention is currently required from: Thomas Heijligen, Angel Pons.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/70176 )
Change subject: README: Add information about meson and link build instructions ......................................................................
Patch Set 2:
(3 comments)
File README:
https://review.coreboot.org/c/flashrom/+/70176/comment/868047e1_4966e9b9 PS1, Line 49: Flashrom
nit: `flashrom` should not capitalised. […]
It is not painful to start with lowercase, so I did that :)
About rephrasing: I think the target audience would prefer direct sentences (I definitely do prefer direct). So leaving as is, just first letter lowercase.
https://review.coreboot.org/c/flashrom/+/70176/comment/dbc246e5_1f15bcaa PS1, Line 51: although not exactly : all of them. Full meson support is on the roadmap in the nearest future.
Hmmm, if meson support isn't full-featured yet, should we explicitly suggest using make whenever pos […]
Yes, your suggestion makes sense, I added one more sentence/paragraph to guide the readers that make is still default.
There is a category of developers/users who are building with meson already (for years) and those who know what they are doing will just continue. However, for the rest of people, I added clear guideline to use make for now, so that people are not confused.
https://review.coreboot.org/c/flashrom/+/70176/comment/6b399034_1b3fc64a PS1, Line 54: /Documentation/building.md
Should we convert the README into a README. […]
You will be delighted to know that on the last meeting of "One Build System Working Group" we decided we need to upgrade README to be fully relevant, up-to-date, readable, and crystal clear to the readers! :) This of course requires a real brainstorming session, open to everyone who is interested, where we can brainstorm the better README together. I am looking forward to it.
Meanwhile, this patch is meant to unblock RC1.