git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/1] Documentation fix related to hash algorithm
@ 2020-05-26 18:37 Toon Claes
  2020-05-26 18:37 ` [PATCH v2 1/1] Documentation: correct hash environment variable Toon Claes
  0 siblings, 1 reply; 3+ messages in thread
From: Toon Claes @ 2020-05-26 18:37 UTC (permalink / raw)
  To: git; +Cc: Toon Claes

I was doing some research on the SHA256 implementation, and I discovered
a mismatch between the docs and the code. I have a patch attached that
fixes the docs.

This is version 2, which has a slightly nicer commit message as
suggested by Martin Ågren.

Martin Ågren also mentioned[0] this discrepancy is added on 2020-02-22
in 3c9331a129, which added both the documentation and the
implementation. As developers-only as this variable is, that commit was
not in v2.26.0, so this might be worth fixing before cutting v2.27.0.

[0] https://lore.kernel.org/git/20200525144138.1624272-1-toon@iotcl.com/T/#m637dc0692962e882928e035888435267b784ee17

Toon Claes (1):
  Documentation: correct hash environment variable

 Documentation/git.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.27.0.rc1.73.g31835d1b01

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH v2 1/1] Documentation: correct hash environment variable
  2020-05-26 18:37 [PATCH v2 0/1] Documentation fix related to hash algorithm Toon Claes
@ 2020-05-26 18:37 ` Toon Claes
  2020-05-27 17:05   ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Toon Claes @ 2020-05-26 18:37 UTC (permalink / raw)
  To: git; +Cc: Toon Claes

To set the default hash algorithm you can set the `GIT_DEFAULT_HASH`
environment variable. In the documentation this variable is named
`GIT_DEFAULT_HASH_ALGORITHM`, which is incorrect.

Signed-off-by: Toon Claes <toon@iotcl.com>
---
 Documentation/git.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/git.txt b/Documentation/git.txt
index 9d6769e95a..12890841c4 100644
--- a/Documentation/git.txt
+++ b/Documentation/git.txt
@@ -493,7 +493,7 @@ double-quotes and respecting backslash escapes. E.g., the value
 	details. This variable has lower precedence than other path
 	variables such as GIT_INDEX_FILE, GIT_OBJECT_DIRECTORY...
 
-`GIT_DEFAULT_HASH_ALGORITHM`::
+`GIT_DEFAULT_HASH`::
 	If this variable is set, the default hash algorithm for new
 	repositories will be set to this value. This value is currently
 	ignored when cloning; the setting of the remote repository
-- 
2.27.0.rc1.73.g31835d1b01


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2 1/1] Documentation: correct hash environment variable
  2020-05-26 18:37 ` [PATCH v2 1/1] Documentation: correct hash environment variable Toon Claes
@ 2020-05-27 17:05   ` Junio C Hamano
  0 siblings, 0 replies; 3+ messages in thread
From: Junio C Hamano @ 2020-05-27 17:05 UTC (permalink / raw)
  To: Toon Claes; +Cc: git

Toon Claes <toon@iotcl.com> writes:

> To set the default hash algorithm you can set the `GIT_DEFAULT_HASH`
> environment variable. In the documentation this variable is named
> `GIT_DEFAULT_HASH_ALGORITHM`, which is incorrect.

My reading stuttered while reading words around "to set ... you can
set"; I would have written

	The GIT_DEFAULT_HASH environment variable, which is the way
	to specify the default hash algorithm, was misspelled in the
	documentation.  Fix it.

but it's OK.

As you said, bc/sha-256-part-1-of-4 is new in this release, so let's
merge this fix down before the final.

Thanks.

> Signed-off-by: Toon Claes <toon@iotcl.com>
> ---
>  Documentation/git.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/git.txt b/Documentation/git.txt
> index 9d6769e95a..12890841c4 100644
> --- a/Documentation/git.txt
> +++ b/Documentation/git.txt
> @@ -493,7 +493,7 @@ double-quotes and respecting backslash escapes. E.g., the value
>  	details. This variable has lower precedence than other path
>  	variables such as GIT_INDEX_FILE, GIT_OBJECT_DIRECTORY...
>  
> -`GIT_DEFAULT_HASH_ALGORITHM`::
> +`GIT_DEFAULT_HASH`::
>  	If this variable is set, the default hash algorithm for new
>  	repositories will be set to this value. This value is currently
>  	ignored when cloning; the setting of the remote repository

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-05-27 17:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-26 18:37 [PATCH v2 0/1] Documentation fix related to hash algorithm Toon Claes
2020-05-26 18:37 ` [PATCH v2 1/1] Documentation: correct hash environment variable Toon Claes
2020-05-27 17:05   ` Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).