That would be way easier on the eye, I think... although on the other hand it would be harder to understand for people unfamiliar with the syntax.

edit: Maybe we can counteract that problem by using the "align by default" version and telling people about the "unaligned" glyph in the error message. If it said something like

"<section name> is not aligned to an erase block (4K). If that is intentional (e.g. this section doesn't need to be erased/updated independently), add a '%' behind the section name/offset)."

then anyone who was unfamiliar with how this works and running into problems with it will understand what it does?

View Change

To view, visit change 37262. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I26b394590c28667a4afcd521c7caa2009b5b98a9
Gerrit-Change-Number: 37262
Gerrit-PatchSet: 5
Gerrit-Owner: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Thu, 28 Nov 2019 02:59:50 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment