[Starlingx-discuss] Proposed rules regarding patch files

Jim Somerville jim.somerville at windriver.com
Tue Nov 26 18:48:47 UTC 2019


Hi Folks,

I'm tired of tripping over the same nonsense again and again regarding 
patch files in our source repos.  Can we consider getting this captured 
in our StarlingX developer how-to documentation/training materials?

Also the build-pkgs tool should have an alternate "--edit" option that 
applies patches via "git am", making the actual source it produces in a 
git repo useful, instead of the mangled commit headers it creates currently.

-Jim

Patch files should always be generated via git format-patch and contain 
a meaningful commit header, thus being "git am friendly".  Patches are 
not to be generated via quick and dirty diff commands.  The commit 
header should also contain your sign off if you've done more than just a 
context refresh or simple rebase of the patch.

Patch files should never be directly edited.  The temptation may be 
there for you to be lazy and just edit a patch file directly.  Do not do 
this, work in git to correct the patch and then regenerate it.

The name of a patch file is generated automatically by git format-patch 
from the shortlog ie. first line of the commit header.  The shortlog is 
very important in the grand scheme of the commit header, put a lot of 
thought into writing that one line to make it *meaningful*.  Many 
developers use "git log --oneline" to display lists of commits, and that 
displays the shortlog only.  Again, make it succinct and meaningful, I 
cannot emphasize this enough.

Patch files should never be renamed eg. via the mv command, after they 
are generated.  If you don't like the name of the generated patch file, 
change the commit shortlog and regenerate it.

Do NOT do this crime (example):

git format-patch <arguments>
produces
fix-some-problem-description.patch
mv fix-some-problem-description.patch some-description-i-prefer.patch



More information about the Starlingx-discuss mailing list