git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 5/8] git-format-patch.txt: general rewordings and cleanups

From
Michael J Gruber <git@drmicha.warpmail.net>
Date
Apr 23, 2009, 12:14 UTC
Message-ID
<49F05B90.8050208@drmicha.warpmail.net>
In-Reply-To
<1240479482-31366-6-git-send-email-bebarino@gmail.com>
Stephen Boyd venit, vidit, dixit 23.04.2009 11:37:
Show 45 quoted lines
> Clarify --no-binary description using some words from the original
> commit 37c22a4b (add --no-binary, 2008-05-9). Cleanup --suffix
> description. Add --thread style option to synopsis and reorganize it a
> bit. Clarify renaming patches example and the configuration paragraph.
> 
> Signed-off-by: Stephen Boyd <bebarino@gmail.com>
> ---
>  Documentation/git-format-patch.txt |   38 ++++++++++++++++++------------------
>  1 files changed, 19 insertions(+), 19 deletions(-)
> 
> diff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt
> index 4a43c64..5ac9e33 100644
> --- a/Documentation/git-format-patch.txt
> +++ b/Documentation/git-format-patch.txt
> @@ -9,10 +9,10 @@ git-format-patch - Prepare patches for e-mail submission
>  SYNOPSIS
>  --------
>  [verse]
> -'git format-patch' [-k] [-o <dir> | --stdout] [--thread]
> -		   [--attach[=<boundary>] | --inline[=<boundary>] |
> -		     [--no-attach]]
> -		   [-s | --signoff] [<common diff options>]
> +'git format-patch' [-k] [(-o|--output-directory) <dir> | --stdout]
> +		   [--thread[=<style>]]
> +		   [(--attach|--inline)[=<boundary>] | --no-attach]
> +		   [-s | --signoff]
>  		   [-n | --numbered | -N | --no-numbered]
>  		   [--start-number <n>] [--numbered-files]
>  		   [--in-reply-to=Message-Id] [--suffix=.<sfx>]
> @@ -20,6 +20,7 @@ SYNOPSIS
>  		   [--subject-prefix=Subject-Prefix]
>  		   [--cc=<email>]
>  		   [--cover-letter]
> +		   [<common diff options>]
>  		   [ <since> | <revision range> ]
>  
>  DESCRIPTION
> @@ -170,18 +171,17 @@ if that is not set.
>  --suffix=.<sfx>::
>  	Instead of using `.patch` as the suffix for generated
>  	filenames, use specified suffix.  A common alternative is
> -	`--suffix=.txt`.
> +	`--suffix=.txt`.  Leaving this empty will remove the `.patch`
> +	suffix.
>  +

Those asciidoc constructs in diff make me always wonder "Huh? A merge diff -cc?"... It's fine, of course.

Show 5 quoted lines
> -Note that you would need to include the leading dot `.` if you
> -want a filename like `0001-description-of-my-change.patch`, and
> -the first letter does not have to be a dot.  Leaving it empty would
> -not add any suffix.
> +Note the leading character does not have to be a dot; for example, you
Maybe "Note that the" sounds and reads better.
Show 26 quoted lines
> +can use `--suffix=-patch` to get `0001-description-of-my-change-patch`.
>  
>  --no-binary::
> -	Don't output contents of changes in binary files, just take note
> -	that they differ.  Note that this disable the patch to be properly
> -	applied.  By default the contents of changes in those files are
> -	encoded in the patch.
> +	Do not output contents of changes in binary files, instead
> +	display a notice that those files changed.  Patches generated
> +	using this option cannot be applied properly, but they are
> +	still useful for code review.
>  
>  --root::
>  	Treat the revision argument as a <revision range>, even if it
> @@ -192,10 +192,10 @@ not add any suffix.
>  
>  CONFIGURATION
>  -------------
> -You can specify extra mail header lines to be added to each message
> -in the repository configuration, new defaults for the subject prefix
> -and file suffix, control attachments, and number patches when outputting
> -more than one.
> +You can specify extra mail header lines to be added to each message,
> +defaults for the subject prefix and file suffix, number patches when
> +outputting more than one patch, add "Cc:" headers, configure attachments,
> +and signoff patches with configuration variables.
"sign off"
Show 14 quoted lines
>  
>  ------------
>  [format]
> @@ -243,8 +243,8 @@ $ git format-patch -M -B origin
>  +
>  Additionally, it detects and handles renames and complete rewrites
>  intelligently to produce a renaming patch.  A renaming patch reduces
> -the amount of text output, and generally makes it easier to review it.
> -Note that the "patch" program does not understand renaming patches, so
> +the amount of text output, and generally makes it easier to review.
> +Note that non-git "patch" programs won't understand renaming patches, so
>  use it only when you know the recipient uses git to apply your patch.
>  
>  * Extract three topmost commits from the current branch and format them
Looks nice overall, especially the ones I'm not replying to ;)
Michael
Previous: Stephen BoydNext: Stephen Boyd
Message 10 of 14 in “Doc updates to config, format-patch, show-branch”
  1. 0/8 Doc updates to config, format-patch, show-branchStephen Boyd, Apr 23, 2009
  2. 1/8 config.txt: remove pointer to SubmittingPatchesStephen Boyd, Apr 23, 2009
  3. 2/8 config.txt: add missing format.{subjectprefix,cc,attach} variablesStephen Boyd, Apr 23, 2009
  4. 3/8 Documentation: use lowercase for shallow and deep threadingStephen Boyd, Apr 23, 2009
  5. 4/8 git-show-branch.txt: cleanup example descriptionStephen Boyd, Apr 23, 2009
  6. 5/8 git-format-patch.txt: general rewordings and cleanupsStephen Boyd, Apr 23, 2009
  7. 6/8 config.txt: add missing 'the's and make words pluralStephen Boyd, Apr 23, 2009
  8. 7/8 config.txt: clarify sentences in the configuration and syntax sectionsStephen Boyd, Apr 23, 2009
  9. 8/8 config.txt: Make configuration paragraph more consistentStephen Boyd, Apr 23, 2009
  10. Michael J GruberApr 23, 2009
  11. 5/8 git-format-patch.txt: general rewordings and cleanupsStephen Boyd, Apr 23, 2009
  12. Junio C HamanoApr 24, 2009
  13. Michael J GruberApr 23, 2009
  14. Stephen BoydApr 23, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.