This shows you the differences between two versions of the page.
Both sides previous revision Previous revision | Next revision Both sides next revision | ||
admin [2015/04/22 15:07] mcb30 |
admin [2015/04/22 15:09] mcb30 |
||
---|---|---|---|
Line 25: | Line 25: | ||
The iPXE policy for commit log messages is: | The iPXE policy for commit log messages is: | ||
- | * Messages start with a header line comprising a %%[subject]%% tag and a brief description of the change | + | * Messages start with a header line comprising a ''[subject]'' tag and a brief description of the change |
- | * The %%[subject]%% tag represents the driver or subsystem being modified. In most cases, this can be deduced by looking at git log for the file(s) being modified by this patch. | + | * The ''[subject]'' tag represents the driver or subsystem being modified. In most cases, this can be deduced by looking at git log for the file(s) being modified by this patch. |
* The brief description should be written in the imperative mood (e.g. "Add PCI device IDs for Intel I218-LM" rather than "Added PCI device IDs for..." or "Adds PCI device IDs for"), and should have no terminating punctuation. | * The brief description should be written in the imperative mood (e.g. "Add PCI device IDs for Intel I218-LM" rather than "Added PCI device IDs for..." or "Adds PCI device IDs for"), and should have no terminating punctuation. | ||
* Trivial patches or patches which simply add a new feature may have an empty log message body. | * Trivial patches or patches which simply add a new feature may have an empty log message body. | ||
* Non-trivial bugfix patches should have a log message explaining the problem and the chosen solution. | * Non-trivial bugfix patches should have a log message explaining the problem and the chosen solution. | ||
- | * All commits must include a %%Signed-off-by:%% line | + | * All commits must include a ''Signed-off-by:'' line |
- | * Credit should be given to all participants in the work leading to the patch, including bug reporters (%%Reported-by:%%), testers (%%Tested-by:%%), original authors of a completely reimplemented patch (%%Originally-implemented-by:%%) etc. Since many contributions come via IRC, the [[:nickmap]] page can be used to look up full names and e-mail addresses suitable for inclusion in the commit log. | + | * Credit should be given to all participants in the work leading to the patch, including bug reporters (''Reported-by:''), testers (''Tested-by:''), original authors of a completely reimplemented patch (''Originally-implemented-by:'') etc. Since many contributions come via IRC, the (private) [[:nickmap]] page can be used to look up full names and e-mail addresses suitable for inclusion in the commit log. |
* Patches which fix a regression should refer to the commit which introduced the regression. | * Patches which fix a regression should refer to the commit which introduced the regression. | ||