Thread (1 message) 1 message, 1 author, 2021-02-02

Re: [PATCH v2 2/6] doc hash-function-transition: use SHA-1 and SHA-256 consistently

From: Junio C Hamano <hidden>
Date: 2021-02-02 23:20:44

Junio C Hamano [off-list ref] writes:
"Thomas Ackermann via GitGitGadget" [off-list ref] writes:
quoted
From: Thomas Ackermann <redacted>

Use SHA-1 and SHA-256 instead of sha1 and sha256  when referring
to the hash type.
Ahh.  [1/6] was supposed to be only about formatting, and I found it
a bit irritating that it had some of these changes mixed in, as it
was not entirely clear to me that [1/6] covered all those lowercase
sha1 and sha256 instances, or just some of them.

Moving them from [1/6] to this step would help future readers by
reducing such irritation (I do not know if it is worth it until
I read through the series to the end).
It seems that only one hunk in 1/6 had premature conversion.
I've tried to locally move it around before queuing.

Thanks.

1:  eea107fb0e ! 1:  5df3cc249d doc hash-function-transition: fix asciidoc output
    @@ Documentation/technical/hash-function-transition.txt: network byte order):
      Loose object index
      ~~~~~~~~~~~~~~~~~~
     @@ Documentation/technical/hash-function-transition.txt: the following steps:
    -    they will be discarded.)
      3. convert to sha256: open a new (sha256) packfile. Read the topologically
         sorted list just generated. For each object, inflate its
    --   sha1-content, convert to sha256-content, and write it to the sha256
    +    sha1-content, convert to sha256-content, and write it to the sha256
     -   pack. Record the new sha1<->sha256 mapping entry for use in the idx.
    -+   SHA-1 content, convert to SHA-256 content, and write it to the SHA-256
    -+   pack. Record the new SHA-1<-->SHA-256 mapping entry for use in the idx.
    ++   pack. Record the new sha1<-->sha256 mapping entry for use in the idx.
      4. sort: reorder entries in the new pack to match the order of objects
         in the pack the server generated and include blobs. Write a sha256 idx
         file
2:  58934c8b43 ! 2:  8488d4f5f1 doc hash-function-transition: use SHA-1 and SHA-256 consistently
    @@ Documentation/technical/hash-function-transition.txt: the following steps:
     -3. convert to sha256: open a new (sha256) packfile. Read the topologically
     +3. convert to SHA-256: open a new SHA-256 packfile. Read the topologically
         sorted list just generated. For each object, inflate its
    -    SHA-1 content, convert to SHA-256 content, and write it to the SHA-256
    -    pack. Record the new SHA-1<-->SHA-256 mapping entry for use in the idx.
    +-   sha1-content, convert to sha256-content, and write it to the sha256
    +-   pack. Record the new sha1<-->sha256 mapping entry for use in the idx.
    ++   SHA-1 content, convert to SHA-256 content, and write it to the SHA-256
    ++   pack. Record the new SHA-1<-->SHA-256 mapping entry for use in the idx.
      4. sort: reorder entries in the new pack to match the order of objects
     -   in the pack the server generated and include blobs. Write a sha256 idx
     +   in the pack the server generated and include blobs. Write a SHA-256 idx
3:  4a710c8715 = 3:  454a9437cf doc hash-function-transition: use upper case consistently
4:  7e690524ac = 4:  b2c881b66b doc hash-function-transition: fix incomplete sentence
5:  80089fe818 = 5:  4ee4775ca3 doc hash-function-transition: move rationale upwards
6:  b221eae801 = 6:  c27c52ca0c doc: use https links
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help