Re: [PATCH v2 2/3] glossary: improve description of SHA-1 related topics

2013-04-07 Thread Simon Ruderich
On Sun, Apr 07, 2013 at 05:10:29PM +0200, Thomas Ackermann wrote:
> Signed-off-by: Thomas Ackermann 
> ---
> [snip]
>
>  [[def_object_name]]object name::
> - The unique identifier of an <>. The <>
> - of the object's contents using the Secure Hash Algorithm
> - 1 and usually represented by the 40 character hexadecimal encoding of
> - the <> of the object.
> + The unique identifier of an <>.  The
> + object name is usually represented by a 40 character
> + hexadecimal string.  Also colloquially called <>
^
Trailing . missing.

Regards
Simon
-- 
+ privacy is necessary
+ using gnupg http://gnupg.org
+ public key id: 0x92FEFDB7E44C32F9
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 2/3] glossary: improve description of SHA-1 related topics

2013-04-07 Thread Junio C Hamano
Thomas Ackermann  writes:

> Signed-off-by: Thomas Ackermann 
> ---

The readers of the history may still want to see "improve how"
explained in the log.

The name of the hash function is "SHA-1", not "SHA1".

Also to people who look up "object name" in the glossary,
the details of which hash function is applied on what to
compute "object name" is not important (the fact that the
name is meant to be an unique identifier for the contents
stored in the object is).

or something?

(no review comment below this line; patch kept for others' reference)

>  Documentation/glossary-content.txt | 20 ++--
>  1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/Documentation/glossary-content.txt 
> b/Documentation/glossary-content.txt
> index 5a7a486..4dd0a52 100644
> --- a/Documentation/glossary-content.txt
> +++ b/Documentation/glossary-content.txt
> @@ -153,7 +153,7 @@ to point at the new commit.
>   created. Configured via the `.git/info/grafts` file.
>  
>  [[def_hash]]hash::
> - In Git's context, synonym to <>.
> + In Git's context, synonym for <>.
>  
>  [[def_head]]head::
>   A <> to the <> at the tip 
> of a
> @@ -225,7 +225,7 @@ This commit is referred to as a "merge commit", or 
> sometimes just a
>  
>  [[def_object]]object::
>   The unit of storage in Git. It is uniquely identified by the
> - <> of its contents. Consequently, an
> + <> of its contents. Consequently, an
>   object can not be changed.
>  
>  [[def_object_database]]object database::
> @@ -237,10 +237,9 @@ This commit is referred to as a "merge commit", or 
> sometimes just a
>   Synonym for <>.
>  
>  [[def_object_name]]object name::
> - The unique identifier of an <>. The <>
> - of the object's contents using the Secure Hash Algorithm
> - 1 and usually represented by the 40 character hexadecimal encoding of
> - the <> of the object.
> + The unique identifier of an <>.  The
> + object name is usually represented by a 40 character
> + hexadecimal string.  Also colloquially called <>
>  
>  [[def_object_type]]object type::
>   One of the identifiers "<>",
> @@ -376,7 +375,7 @@ should not be combined with other pathspec.
>   to the result.
>  
>  [[def_ref]]ref::
> - A 40-byte hex representation of a <> or a name that
> + A 40-byte hex representation of a <> or a name that
>   denotes a particular <>. They may be stored in
>   a file under `$GIT_DIR/refs/` directory, or
>   in the `$GIT_DIR/packed-refs` file.
> @@ -432,8 +431,9 @@ should not be combined with other pathspec.
>  [[def_SCM]]SCM::
>   Source code management (tool).
>  
> -[[def_SHA1]]SHA1::
> - Synonym for <>.
> +[[def_SHA1]]SHA-1::
> + "Secure Hash Algorithm 1"; a cryptographic hash function. 
> + In the context of Git used as a synonym for < name>>.
>  
>  [[def_shallow_repository]]shallow repository::
>   A shallow <> has an incomplete
> @@ -447,7 +447,7 @@ should not be combined with other pathspec.
>   its history can be later deepened with linkgit:git-fetch[1].
>  
>  [[def_symref]]symref::
> - Symbolic reference: instead of containing the <>
> + Symbolic reference: instead of containing the <>
>   id itself, it is of the format 'ref: refs/some/thing' and when
>   referenced, it recursively dereferences to this reference.
>   '<>' is a prime example of a symref. Symbolic
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html