[coreboot] Guide lines: Prepend commit summary with component (was: New patch to review for coreboot: 0b41858 Clean up the mmu setup a bit)

Peter Stuge peter at stuge.se
Wed Jan 30 21:47:49 CET 2013


David Hendricks wrote:
> If you wish to focus only on a particular part of the codebase

It's more about getting an overview of what has changed.

It is impossible to discern whether "Fix MMU setup" is at all
relevant for my ThinkPad from the message alone. Maybe some setup
was wrong - what do I know. "armv7: Fix MMU setup" however makes it
clear what the change is.


> specify the directories/files you wish to look at when running git log.

It's the other way around, for creating an overview.

The commit message represents the commit. The commit contains a root
tree. Commits only affecting subsets of the code isn't really an
accurate representation of how the repository works.


> Actually, I think it would look kind of ugly to do "git log src/arch/x86"
> and see every commit cluttered with some cookie-cutter prefix.

Maybe - or maybe it is simply expected. Within x86 it's possible that
there will be several different and distinct parts.


//Peter



More information about the coreboot mailing list