Re: [PATCH] Documentation: Use 'First Paragraph' instead of 'First Line'.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jeremy White <jwhite@xxxxxxxxxxxxxxx> writes:

> The discussion of email subject throughout the documentation is
> misleading; it indicates that the first line will become the subject.
> In fact, the first and second and third lines will become the subject,
> up until the first full blank line.  Describing it as the first paragraph
> is more accurate.
>
> Signed-off-by: Jeremy White <jwhite@xxxxxxxxxxxxxxx>
> ---
>  Documentation/git-commit.txt       |    2 +-
>  Documentation/git-for-each-ref.txt |    2 +-
>  Documentation/git-format-patch.txt |    8 +++++---
>  Documentation/git-shortlog.txt     |    2 +-
>  Documentation/gitcore-tutorial.txt |    2 +-
>  Documentation/gittutorial.txt      |    2 +-
>  Documentation/user-manual.txt      |    2 +-
>  7 files changed, 11 insertions(+), 9 deletions(-)
>
> diff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt
> index e99bb14..a61bca9 100644
> --- a/Documentation/git-commit.txt
> +++ b/Documentation/git-commit.txt
> @@ -349,7 +349,7 @@ DISCUSSION
>  Though not required, it's a good idea to begin the commit message
>  with a single short (less than 50 character) line summarizing the
>  change, followed by a blank line and then a more thorough description.
> -Tools that turn commits into email, for example, use the first line
> +Tools that turn commits into email, for example, use the first paragraph
>  on the Subject: line and the rest of the commit in the body.

This is a good change, as the previous sentence says the norm for
the first paragraph is a single line.

> diff --git a/Documentation/git-for-each-ref.txt b/Documentation/git-for-each-ref.txt
> index 7e83288..499c26a 100644
> --- a/Documentation/git-for-each-ref.txt
> +++ b/Documentation/git-for-each-ref.txt
> @@ -100,7 +100,7 @@ Fields that have name-email-date tuple as its value (`author`,
>  `committer`, and `tagger`) can be suffixed with `name`, `email`,
>  and `date` to extract the named component.
>  
> -The first line of the message in a commit and tag object is
> +The first paragraph of the message in a commit and tag object is
>  `subject`, the remaining lines are `body`.  The whole message
>  is `contents`.

This may need a bit more explanation.  Just saying "First Paragraph"
without saying that for all commits in a sane project it is the same
as "First Line" will make the reader wonder "What happens to the
second and subsequent lines?" (the answer is that the first paragraph 
is made into a single looooooong line).

> diff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt
> index 9674f9d..e6f6d0e 100644
> --- a/Documentation/git-format-patch.txt
> +++ b/Documentation/git-format-patch.txt
> @@ -57,10 +57,12 @@ output, unless the `--stdout` option is specified.
>  If `-o` is specified, output files are created in <dir>.  Otherwise
>  they are created in the current working directory.
>  
> -By default, the subject of a single patch is "[PATCH] First Line" and
> +By default, the subject of a single patch is "[PATCH] First Paragraph" and
>  the subject when multiple patches are output is "[PATCH n/m] First
> +Paragraph". Note that "First Paragraph" consists of text in the commit message
> +prior to the first completely blank line (see the DISCUSSION section
> +in linkgit:git-commit[1]).  To force 1/1 to be added for a single patch,
> +use `-n`.  To omit patch numbers from the subject, use `-N`.

Ditto.

> diff --git a/Documentation/git-shortlog.txt b/Documentation/git-shortlog.txt
> index dfd4d0c..9464932 100644
> --- a/Documentation/git-shortlog.txt
> +++ b/Documentation/git-shortlog.txt
> @@ -15,7 +15,7 @@ DESCRIPTION
>  -----------
>  Summarizes 'git log' output in a format suitable for inclusion
>  in release announcements. Each commit will be grouped by author and
> -the first line of the commit message will be shown.
> +the first paragraph of the commit message will be shown.

Ditto.

> diff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt
> index f7815e9..92f97e6 100644
> --- a/Documentation/gitcore-tutorial.txt
> +++ b/Documentation/gitcore-tutorial.txt
> @@ -956,7 +956,7 @@ $ git show-branch --topo-order --more=1 master mybranch
>  ------------------------------------------------
>  
>  The first two lines indicate that it is showing the two branches
> -and the first line of the commit log message from their
> +and the first paragraph of the commit log message from their
>  top-of-the-tree commits, you are currently on `master` branch
>  (notice the asterisk `\*` character), and the first column for
>  the later output lines is used to show commits contained in the

Ditto.

> diff --git a/Documentation/gittutorial.txt b/Documentation/gittutorial.txt
> index 1c16066..a1bc56c 100644
> --- a/Documentation/gittutorial.txt
> +++ b/Documentation/gittutorial.txt
> @@ -139,7 +139,7 @@ A note on commit messages: Though not required, it's a good idea to
>  begin the commit message with a single short (less than 50 character)
>  line summarizing the change, followed by a blank line and then a more
>  thorough description.  Tools that turn commits into email, for
> -example, use the first line on the Subject: line and the rest of the
> +example, use the first paragraph on the Subject: line and the rest of the
>  commit in the body.

This is a good change, as it is clear that you are expected to write
a single line in the first paragraph in the previous sentence.

> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt
> index fe6fb72..dd2f985 100644
> --- a/Documentation/user-manual.txt
> +++ b/Documentation/user-manual.txt
> @@ -1126,7 +1126,7 @@ Though not required, it's a good idea to begin the commit message
>  with a single short (less than 50 character) line summarizing the
>  change, followed by a blank line and then a more thorough
>  description.  Tools that turn commits into email, for example, use
> -the first line on the Subject line and the rest of the commit in the
> +the first paragraph on the Subject line and the rest of the commit in the
>  body.

Ditto.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]